On 10/18/2024, Marek Vasut wrote: > On 10/18/24 8:48 AM, Liu Ying wrote: >> Add a HDMI connector to connect with i.MX8MP HDMI TX output. >> This is a preparation for making the i.MX8MP LCDIF driver use >> drm_bridge_connector which requires the DRM_BRIDGE_ATTACH_NO_CONNECTOR >> flag. With that flag, the DW HDMI bridge core driver would >> try to attach the next bridge which is the HDMI connector. >> >> Signed-off-by: Liu Ying <victor.liu@xxxxxxx> >> --- >> .../dts/freescale/imx8mp-msc-sm2s-ep1.dts | 19 +++++++++++++++++++ >> 1 file changed, 19 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dts b/arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dts >> index 83194ea7cb81..b776646a258a 100644 >> --- a/arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dts >> +++ b/arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dts >> @@ -15,6 +15,17 @@ / { >> "avnet,sm2s-imx8mp-14N0600E", "avnet,sm2s-imx8mp", >> "fsl,imx8mp"; >> + hdmi-connector { >> + compatible = "hdmi-connector"; >> + type = "a"; > Shouldn't this also have a 'label' property ? 'label' property is not required by hdmi-connector.yaml and there are in-tree hdmi-connector nodes that haven't got it. I tried to find schematics for the board online, but failed. I don't have the board to see the label printed in silk layer, either. If anyone can provide a valid label name, I may add it. -- Regards, Liu Ying