Hi Chris, Thank you for the patch. On Tue, Jun 11, 2024 at 12:42:38AM +0100, Christopher Obbard wrote: > Enable the HDMI output on the Debix Model A SBC, using the HDMI encoder > present in the i.MX8MP SoC. > > This has been tested with a generic 1080p HDMI display, along with a > smaller 1280x800 HDMI display. > > Signed-off-by: Christopher Obbard <chris.obbard@xxxxxxxxxxxxx> I've been running with this patch in my tree for a while, Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> Tested-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> Shawn, could you merge this for v6.11 ? > --- > > Changes in v2: > - Do not set SION (Software Input ON) bit in pinmux pin config register. > - Small changes to commit log/cover letter. > - v1: https://lore.kernel.org/linux-arm-kernel/20240415114135.25473-1-chris.obbard@xxxxxxxxxxxxx/ > > .../dts/freescale/imx8mp-debix-model-a.dts | 47 +++++++++++++++++++ > 1 file changed, 47 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-debix-model-a.dts b/arch/arm64/boot/dts/freescale/imx8mp-debix-model-a.dts > index 9b8f97a84e61..af02af9e5334 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mp-debix-model-a.dts > +++ b/arch/arm64/boot/dts/freescale/imx8mp-debix-model-a.dts > @@ -20,6 +20,18 @@ chosen { > stdout-path = &uart2; > }; > > + hdmi-connector { > + compatible = "hdmi-connector"; > + label = "hdmi"; > + type = "a"; > + > + port { > + hdmi_connector_in: endpoint { > + remote-endpoint = <&hdmi_tx_out>; > + }; > + }; > + }; > + > leds { > compatible = "gpio-leds"; > pinctrl-names = "default"; > @@ -94,6 +106,28 @@ ethphy0: ethernet-phy@0 { /* RTL8211E */ > }; > }; > > +&hdmi_pvi { > + status = "okay"; > +}; > + > +&hdmi_tx { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_hdmi>; > + status = "okay"; > + > + ports { > + port@1 { > + hdmi_tx_out: endpoint { > + remote-endpoint = <&hdmi_connector_in>; > + }; > + }; > + }; > +}; > + > +&hdmi_tx_phy { > + status = "okay"; > +}; > + > &i2c1 { > clock-frequency = <400000>; > pinctrl-names = "default"; > @@ -239,6 +273,10 @@ &i2c6 { > status = "okay"; > }; > > +&lcdif3 { > + status = "okay"; > +}; > + > &snvs_pwrkey { > status = "okay"; > }; > @@ -356,6 +394,15 @@ MX8MP_IOMUXC_NAND_READY_B__GPIO3_IO16 0x19 > >; > }; > > + pinctrl_hdmi: hdmigrp { > + fsl,pins = < > + MX8MP_IOMUXC_HDMI_DDC_SCL__HDMIMIX_HDMI_SCL 0x1c3 > + MX8MP_IOMUXC_HDMI_DDC_SDA__HDMIMIX_HDMI_SDA 0x1c3 > + MX8MP_IOMUXC_HDMI_HPD__HDMIMIX_HDMI_HPD 0x19 > + MX8MP_IOMUXC_HDMI_CEC__HDMIMIX_HDMI_CEC 0x19 > + >; > + }; > + > pinctrl_i2c1: i2c1grp { > fsl,pins = < > MX8MP_IOMUXC_I2C1_SCL__I2C1_SCL 0x400001c2 -- Regards, Laurent Pinchart
