Re: [PATCH] arm64: dts: ti: k3-am62l: Enable HDMI & DSI display

[Swamil Jain]

On 7/13/26 13:37, Vignesh Raghavendra wrote:
> On Thu, 14 May 2026 02:39:42 +0530, Swamil Jain <s-jain1@xxxxxx> wrote:
> > Enable DSS, DSI and HDMI related nodes and pinmuxes.
>
> Don't see any Pinmux node added? did you miss them?

Added in v2: 
https://lore.kernel.org/all/20260528133529.3476499-1-s-jain1@xxxxxx/

> Also, DSI still remains disabled?

Keeping HDMI out-of-box, dsi can be enabled using panel overlays.

> > 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 {
>
> New nodes need to added in sorted order as per their addresses in this
> dtsi file.

Yeah, sure.

> > [ ... skip 17 lines ... ]
> > +		};
> > +	};
> > +
> > +	dphy_tx0: phy@301c0000 {
> > +		compatible = "ti,j721e-dphy";
> > +		reg = <0x0 0x301c0000 0x0 0x1000>;
>
> reg = <0x00 0x301c0000 0x00 0x1000>;

Ack.

> > +		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>;
>
> reg = <0x00 0x30500000 0x00 0x100000>,
> 		      <0x00 0x30270000 0x00 0x100>;

Ack.

> > +		clocks = <&scmi_clk 155>, <&scmi_clk 158>;
> > +		clock-names = "dsi_p_clk", "dsi_sys_clk";
> > +		power-domains = <&scmi_pds 38>;
> > +		interrupt-parent = <&gic500>;
>
> GIC is implied, drop.

Will drop in next rev.

> > +		interrupts = <GIC_SPI 57 IRQ_TYPE_LEVEL_HIGH>;
> > +		phys = <&dphy_tx0>;
> > +		phy-names = "dphy";
> > +		status = "disabled";
>
> Don't you need dsi*_ports ?
>
> >

Yes, Udit pointed in v2: 
https://lore.kernel.org/all/cb4c1b80-40c0-4e63-bfa8-65a68c577999@xxxxxx/
Will add ports info. in next rev.

> > diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> > index a1af4571a815..7462921e2180 100644
> > --- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> > +++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> > @@ -192,6 +204,36 @@ exp2: gpio@23 {
> >   		bootph-all;
> >   	};
> >   
> > +	sii9022: bridge-hdmi@3b {
> > +		compatible = "sil,sii9022";
> > +		reg = <0x3b>;
> > +		interrupt-parent = <&exp1>;
>
> Is there a reset line to the chip, I see GPIO_HDMI_RSTn in exp2 GPIO
> expander, does that need to be hooked up somewhere?

It is optional, HDMI output works fine without reset-gpios.

Regards,
Swamil.