Re: [PATCH v2] arm64: dts: ti: k3-am62l: Add HDMI & DSI support

[Swamil Jain]

Hi Udit,

On 5/29/26 14:28, Kumar, Udit wrote:
> On 5/28/2026 7:05 PM, Swamil Jain wrote:
> > From: Devarsh Thakkar <devarsht@xxxxxx>
> >
> > Add DSS, DSI, DPHY, and HDMI bridge nodes to enable DPI-to-HDMI output
> > on AM62L[1]. The DSS uses parallel DPI interface with pinctrl
> > configuration to drive an external SiI9022 HDMI bridge.
> >
> > DSI and DPHY nodes are added but remain disabled, as the AM62L shares a
> > single video port for DPI and DSI output.
> >
> > DSI functionality can be enabled via overlays when required.
> >
> > [1]: https://www.ti.com/product/AM62L
> >
> > Signed-off-by: Devarsh Thakkar <devarsht@xxxxxx>
> > Signed-off-by: Swamil Jain <s-jain1@xxxxxx>
> > ---
> > Changelog:
> > v1->v2:
> > - Update the commit message to elaborate more about the patch.
> > - Add dpi-pinctrl configuration.
> > - Update the interrupt routing index for HDMI interrupt line
> >
> > Link to v1: https://lore.kernel.org/all/20260513210942.692269-1-s-jain1@xxxxxx/
> > ---
> >   arch/arm64/boot/dts/ti/k3-am62l-main.dtsi | 47 +++++++++++
> >   arch/arm64/boot/dts/ti/k3-am62l3-evm.dts  | 95 +++++++++++++++++++++++
> >   2 files changed, 142 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi
> > index 80615ca1e01a..1d255c9138db 100644
> > --- a/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi
> > +++ b/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi
> > @@ -584,4 +584,51 @@ scmi_shmem: sram@0 {
> >   			bootph-all;
> >   		};
> >   	};
> > +
> > +	dss: dss@30200000 {
> > +		compatible = "ti,am62l-dss";
> > +		reg = <0x00 0x30200000 0x00 0x1000>, /* common */
> > +		      <0x00 0x30202000 0x00 0x1000>, /* vidl1 */
> > +		      <0x00 0x30207000 0x00 0x1000>, /* ovr1 */
> > +		      <0x00 0x3020a000 0x00 0x1000>, /* vp1 */
> > +		      <0x00 0x30201000 0x00 0x1000>; /* common1 */
> > +		reg-names = "common", "vidl1", "ovr1", "vp1", "common1";
> > +		power-domains = <&scmi_pds 39>;
> > +		clocks = <&scmi_clk 162>,
> > +			 <&scmi_clk 161>;
> > +		clock-names = "fck", "vp1";
> > +		interrupts = <GIC_SPI 60 IRQ_TYPE_LEVEL_HIGH>;
> > +		status = "disabled";
> > +
> > +		dss_ports: ports {
> > +			#address-cells = <1>;
> > +			#size-cells = <0>;
> > +		};
> > +	};
> > +
> > +	dphy_tx0: phy@301c0000 {
> > +		compatible = "ti,j721e-dphy";
> > +		reg = <0x0 0x301c0000 0x0 0x1000>;
> > +		clocks = <&scmi_clk 348>, <&scmi_clk 343>;
> > +		clock-names = "psm", "pll_ref";
> > +		#phy-cells = <0>;
> > +		power-domains = <&scmi_pds 86>;
> > +		assigned-clocks = <&scmi_clk 343>;
> > +		assigned-clock-parents = <&scmi_clk 341>;
> > +		assigned-clock-rates = <25000000>;
> > +		status = "disabled";
> > +	};
> > +
> > +	dsi0: dsi@30500000 {
> > +		compatible = "ti,j721e-dsi";
> > +		reg = <0x0 0x30500000 0x0 0x100000>, <0x0 0x30270000 0x0 0x100>;
> > +		clocks = <&scmi_clk 155>, <&scmi_clk 158>;
> > +		clock-names = "dsi_p_clk", "dsi_sys_clk";
> > +		power-domains = <&scmi_pds 38>;
> > +		interrupt-parent = <&gic500>;
> > +		interrupts = <GIC_SPI 57 IRQ_TYPE_LEVEL_HIGH>;
> > +		phys = <&dphy_tx0>;
> > +		phy-names = "dphy";
> > +		status = "disabled";
>
>
> I think you should add port information for dsi as well
> see DSI node in other SOC (J721E, J721S2, J784s4) for reference.

Sure, will add ports info. in the next rev.

Regards,
Swamil.


> > +	};
> >   };
> > diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> > index a1af4571a815..9b04fa6d2cba 100644
> > --- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> > +++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> > @@ -119,6 +119,18 @@ vcc_1v8: regulator-3 {
> >   		regulator-always-on;
> >   		regulator-boot-on;
> >   	};
> > +
> [..]