Re: [PATCH v3 08/11] drm/bridge: imx8mp-hdmi-tx-connector-fixup: add an hdmi-connector when missing using a DT overlay at boot time

[Luca Ceresoli]

Hello Liu,

On Fri Apr 3, 2026 at 11:28 AM CEST, Liu Ying wrote:

>> +++ b/drivers/gpu/drm/bridge/imx/imx8mp-hdmi-tx-connector-fixup.dtso
>> @@ -0,0 +1,33 @@
>> +// SPDX-License-Identifier: GPL-2.0+
>> +/*
>> + * DTS overlay adding an hdmi-connector node to boards using the imx8mp hdmi_tx
>> + *
>> + * Copyright (C) 2026 GE HealthCare
>> + * Author: Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx>
>> + */
>> +
>> +/dts-v1/;
>> +/plugin/;
>> +
>> +&{/} {
>> +	#address-cells = <2>;
>> +	#size-cells = <2>;
>> +
>> +	fixup-hdmi-connector {
>> +		compatible = "hdmi-connector";
>> +		label = "HDMI";
>> +		type = "a";
>> +
>> +		port {
>> +			fixup_hdmi_connector_in: endpoint {
>> +				remote-endpoint = <&hdmi_tx_out>;
>> +			};
>> +		};
>> +	};
>> +};
>> +
>> +&{/soc@0/bus@32c00000/hdmi@32fd8000/ports/port@1} {
>> +	hdmi_tx_out: endpoint {
>> +		remote-endpoint = <&fixup_hdmi_connector_in>;
>> +	};
>> +};
>>
>
> There is a build warning(W=1):
>
>   DTC     drivers/gpu/drm/bridge/imx/imx8mp-hdmi-tx-connector-fixup.dtbo
> drivers/gpu/drm/bridge/imx/imx8mp-hdmi-tx-connector-fixup.dtso:12.6-27.3: Warning (avoid_unnecessary_addr_size): /fragment@0/__overlay__: unnecessary #address-cells/#size-cells without "ranges", "dma-ranges" or child "reg" or "ranges" property

Ouch, my bad, forgot to run W=1 after the last edit. Fix incoming v4,
toghether with the other improvements you have suggested.

Luca

--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com