Devicetree
 help / color / mirror / Atom feed
* [PATCH v5 0/2] Add DSS support for AM62L SoC
@ 2026-09-23 19:26 Swamil Jain
  2026-09-23 19:26 ` [PATCH v5 1/2] arm64: dts: ti: k3-am62l-main: Add DSS & DSI related nodes Swamil Jain
  2026-09-23 19:26 ` [PATCH v5 2/2] arm64: dts: ti: k3-am62l3-evm: Add HDMI support Swamil Jain
  0 siblings, 2 replies; 8+ messages in thread
From: Swamil Jain @ 2026-09-23 19:26 UTC (permalink / raw)
  To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt, tomi.valkeinen
  Cc: devarsht, sebin.francis, praneeth, u-kumar1, linux-arm-kernel,
	devicetree, linux-kernel, s-jain1

This series:
- Adds base support to enable Display on TI AM62L[1] SoCs
- Adds base support for DSI output
- Enables HDMI output out-of-box

[1]: https://www.ti.com/product/AM62L
---
Test-log:
https://gist.github.com/jainswamil/c21b0d7c5a3b5b2fc2a3f79101a296ce
---
Changelog:
v4->v5:
- Move pinctrl from sii9022 to dss
- Add bootph-all property to all display related nodes
- Pick R-by tag

Link to v4: https://lore.kernel.org/all/20260811130822.2324438-1-s-jain1@ti.com/

v3->v4:
- Change pin configuration from PIN_OUTPUT to PIN_INPUT for
  VOUT_EXTPCLKIN

Link to v3: https://lore.kernel.org/all/20260723102519.3581762-1-s-jain1@ti.com/

v2->v3:
- Split patches for SoC and EVM related changes
- Add dsi ports node
- Correct styling issues
- Drop GIC

Link to v2: https://lore.kernel.org/all/20260528133529.3476499-1-s-jain1@ti.com/

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@ti.com/
---
Devarsh Thakkar (2):
  arm64: dts: ti: k3-am62l-main: Add DSS & DSI related nodes
  arm64: dts: ti: k3-am62l3-evm: Add HDMI support

 arch/arm64/boot/dts/ti/k3-am62l-main.dtsi |  60 ++++++++++++
 arch/arm64/boot/dts/ti/k3-am62l3-evm.dts  | 106 ++++++++++++++++++++++
 2 files changed, 166 insertions(+)


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH v5 1/2] arm64: dts: ti: k3-am62l-main: Add DSS & DSI related nodes
  2026-09-23 19:26 [PATCH v5 0/2] Add DSS support for AM62L SoC Swamil Jain
@ 2026-09-23 19:26 ` Swamil Jain
  2026-09-24  9:44   ` Kumar, Udit
  2026-09-23 19:26 ` [PATCH v5 2/2] arm64: dts: ti: k3-am62l3-evm: Add HDMI support Swamil Jain
  1 sibling, 1 reply; 8+ messages in thread
From: Swamil Jain @ 2026-09-23 19:26 UTC (permalink / raw)
  To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt, tomi.valkeinen
  Cc: devarsht, sebin.francis, praneeth, u-kumar1, linux-arm-kernel,
	devicetree, linux-kernel, s-jain1

From: Devarsh Thakkar <devarsht@ti.com>

Add DSS, DSI and DPHY nodes to add base display support on AM62L[1].
AM62L shares a single video port for DPI and DSI output.

[1]: https://www.ti.com/product/AM62L

Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Signed-off-by: Devarsh Thakkar <devarsht@ti.com>
Signed-off-by: Swamil Jain <s-jain1@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am62l-main.dtsi | 60 +++++++++++++++++++++++
 1 file changed, 60 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..a3199a2455c3 100644
--- a/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi
@@ -544,6 +544,66 @@ elm0: ecc@25010000 {
 		status = "disabled";
 	};
 
+	dphy_tx0: phy@301c0000 {
+		compatible = "ti,j721e-dphy";
+		reg = <0x00 0x301c0000 0x00 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";
+	};
+
+	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>;
+		};
+	};
+
+	dsi0: dsi@30500000 {
+		compatible = "ti,j721e-dsi";
+		reg = <0x00 0x30500000 0x00 0x100000>,
+		      <0x00 0x30270000 0x00 0x100>;
+		clocks = <&scmi_clk 155>, <&scmi_clk 158>;
+		clock-names = "dsi_p_clk", "dsi_sys_clk";
+		power-domains = <&scmi_pds 38>;
+		interrupts = <GIC_SPI 57 IRQ_TYPE_LEVEL_HIGH>;
+		phys = <&dphy_tx0>;
+		phy-names = "dphy";
+		status = "disabled";
+
+		dsi0_ports: ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+
+			port@0 {
+				reg = <0>;
+			};
+
+			port@1 {
+				reg = <1>;
+			};
+		};
+	};
+
 	gpmc0: memory-controller@3b000000 {
 		compatible = "ti,am64-gpmc";
 		power-domains = <&scmi_pds 37>;

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH v5 2/2] arm64: dts: ti: k3-am62l3-evm: Add HDMI support
  2026-09-23 19:26 [PATCH v5 0/2] Add DSS support for AM62L SoC Swamil Jain
  2026-09-23 19:26 ` [PATCH v5 1/2] arm64: dts: ti: k3-am62l-main: Add DSS & DSI related nodes Swamil Jain
@ 2026-09-23 19:26 ` Swamil Jain
  2026-09-23 19:40   ` sashiko-bot
  2026-09-24  9:43   ` Kumar, Udit
  1 sibling, 2 replies; 8+ messages in thread
From: Swamil Jain @ 2026-09-23 19:26 UTC (permalink / raw)
  To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt, tomi.valkeinen
  Cc: devarsht, sebin.francis, praneeth, u-kumar1, linux-arm-kernel,
	devicetree, linux-kernel, s-jain1

From: Devarsh Thakkar <devarsht@ti.com>

Add the SII9022 HDMI bridge and HDMI connector nodes, along with the
DPI pinmux configuration connecting VOUT0 to the bridge. Enable the DSS
and hook up its DPI output port to the SII9022 bridge input, and the
bridge output to the HDMI connector.

Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Signed-off-by: Devarsh Thakkar <devarsht@ti.com>
Signed-off-by: Swamil Jain <s-jain1@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 106 +++++++++++++++++++++++
 1 file changed, 106 insertions(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
index 84a0cfaaf196..8ff7c4baaadb 100644
--- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
@@ -44,6 +44,22 @@ usr: button-usr {
 		};
 	};
 
+	hdmi0: connector-hdmi {
+		compatible = "hdmi-connector";
+		label = "hdmi";
+		type = "a";
+		bootph-all;
+
+		port {
+			bootph-all;
+
+			hdmi_connector_in: endpoint {
+				remote-endpoint = <&sii9022_out>;
+				bootph-all;
+			};
+		};
+	};
+
 	leds {
 		compatible = "gpio-leds";
 		pinctrl-names = "default";
@@ -192,6 +208,41 @@ exp2: gpio@23 {
 		bootph-all;
 	};
 
+	sii9022: bridge-hdmi@3b {
+		compatible = "sil,sii9022";
+		reg = <0x3b>;
+		interrupt-parent = <&exp1>;
+		interrupts = <20 IRQ_TYPE_EDGE_FALLING>;
+		#sound-dai-cells = <0>;
+		sil,i2s-data-lanes = < 0 >;
+		bootph-all;
+
+		ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			bootph-all;
+
+			port@0 {
+				reg = <0>;
+				bootph-all;
+
+				sii9022_in: endpoint {
+					remote-endpoint = <&dpi_out>;
+					bootph-all;
+				};
+			};
+
+			port@1 {
+				reg = <1>;
+				bootph-all;
+
+				sii9022_out: endpoint {
+					remote-endpoint = <&hdmi_connector_in>;
+					bootph-all;
+				};
+			};
+		};
+	};
 };
 
 &i2c2 {
@@ -221,6 +272,41 @@ usb_con_hs: endpoint {
 };
 
 &pmx0 {
+	dpi_pins_default: dpi-default-pins {
+		pinctrl-single,pins = <
+			AM62LX_IOPAD(0x00e4, PIN_OUTPUT, 1) /* (L20) GPMC0_CSn0.VOUT0_VSYNC */
+			AM62LX_IOPAD(0x00dc, PIN_OUTPUT, 1) /* (N21) GPMC0_WPn.VOUT0_HSYNC */
+			AM62LX_IOPAD(0x00e8, PIN_OUTPUT, 1) /* (L19) GPMC0_CSn1.VOUT0_PCLK */
+			AM62LX_IOPAD(0x00e0, PIN_OUTPUT, 1) /* (M21) GPMC0_DIR.VOUT0_DE */
+			AM62LX_IOPAD(0x0078, PIN_OUTPUT, 1) /* (L22) GPMC0_AD0.VOUT0_DATA0 */
+			AM62LX_IOPAD(0x007c, PIN_OUTPUT, 1) /* (L23) GPMC0_AD1.VOUT0_DATA1 */
+			AM62LX_IOPAD(0x0080, PIN_OUTPUT, 1) /* (K22) GPMC0_AD2.VOUT0_DATA2 */
+			AM62LX_IOPAD(0x0084, PIN_OUTPUT, 1) /* (J23) GPMC0_AD3.VOUT0_DATA3 */
+			AM62LX_IOPAD(0x0088, PIN_OUTPUT, 1) /* (K23) GPMC0_AD4.VOUT0_DATA4 */
+			AM62LX_IOPAD(0x008c, PIN_OUTPUT, 1) /* (H22) GPMC0_AD5.VOUT0_DATA5 */
+			AM62LX_IOPAD(0x0090, PIN_OUTPUT, 1) /* (H23) GPMC0_AD6.VOUT0_DATA6 */
+			AM62LX_IOPAD(0x0094, PIN_OUTPUT, 1) /* (J22) GPMC0_AD7.VOUT0_DATA7 */
+			AM62LX_IOPAD(0x0098, PIN_OUTPUT, 1) /* (H19) GPMC0_AD8.VOUT0_DATA8 */
+			AM62LX_IOPAD(0x009c, PIN_OUTPUT, 1) /* (H20) GPMC0_AD9.VOUT0_DATA9 */
+			AM62LX_IOPAD(0x00a0, PIN_OUTPUT, 1) /* (H21) GPMC0_AD10.VOUT0_DATA10 */
+			AM62LX_IOPAD(0x00a4, PIN_OUTPUT, 1) /* (H18) GPMC0_AD11.VOUT0_DATA11 */
+			AM62LX_IOPAD(0x00a8, PIN_OUTPUT, 1) /* (G23) GPMC0_AD12.VOUT0_DATA12 */
+			AM62LX_IOPAD(0x00ac, PIN_OUTPUT, 1) /* (G22) GPMC0_AD13.VOUT0_DATA13 */
+			AM62LX_IOPAD(0x00b0, PIN_OUTPUT, 1) /* (F22) GPMC0_AD14.VOUT0_DATA14 */
+			AM62LX_IOPAD(0x00b4, PIN_OUTPUT, 1) /* (F23) GPMC0_AD15.VOUT0_DATA15 */
+			AM62LX_IOPAD(0x00b8, PIN_OUTPUT, 1) /* (L21) GPMC0_CLK.VOUT0_DATA16 */
+			AM62LX_IOPAD(0x00c0, PIN_OUTPUT, 1) /* (N19) GPMC0_ADVn_ALE.VOUT0_DATA17 */
+			AM62LX_IOPAD(0x00c4, PIN_OUTPUT, 1) /* (N20) GPMC0_OEn_REn.VOUT0_DATA18 */
+			AM62LX_IOPAD(0x00c8, PIN_OUTPUT, 1) /* (M19) GPMC0_WEn.VOUT0_DATA19 */
+			AM62LX_IOPAD(0x00cc, PIN_OUTPUT, 1) /* (P23) GPMC0_BE0n_CLE.VOUT0_DATA20 */
+			AM62LX_IOPAD(0x00d0, PIN_OUTPUT, 1) /* (P22) GPMC0_BE1n.VOUT0_DATA21 */
+			AM62LX_IOPAD(0x00d4, PIN_OUTPUT, 1) /* (N23) GPMC0_WAIT0.VOUT0_DATA22 */
+			AM62LX_IOPAD(0x00d8, PIN_OUTPUT, 1) /* (N22) GPMC0_WAIT1.VOUT0_DATA23 */
+			AM62LX_IOPAD(0x00ec, PIN_INPUT,  5) /* (M23) GPMC0_CSn2.VOUT0_EXTPCLKIN */
+		>;
+		bootph-all;
+	};
+
 	gpio0_ioexp_intr_pins_default: gpio0-ioexp-intr-default-pins {
 		pinctrl-single,pins = <
 			AM62LX_IOPAD(0x01b0, PIN_INPUT, 7) /* (B12) SPI0_D1.GPIO0_91 */
@@ -321,6 +407,26 @@ AM62LX_IOPAD(0x0004, PIN_OUTPUT, 0) /* (AA23) WKUP_UART0_TXD */
 	};
 };
 
+&dss {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&dpi_pins_default>;
+	bootph-all;
+};
+
+&dss_ports {
+	/* VP1: DPI Output */
+	port@0 {
+		reg = <0>;
+		bootph-all;
+
+		dpi_out: endpoint {
+			remote-endpoint = <&sii9022_in>;
+			bootph-all;
+		};
+	};
+};
+
 &sdhci0 {
 	/* eMMC */
 	pinctrl-names = "default";

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH v5 2/2] arm64: dts: ti: k3-am62l3-evm: Add HDMI support
  2026-09-23 19:26 ` [PATCH v5 2/2] arm64: dts: ti: k3-am62l3-evm: Add HDMI support Swamil Jain
@ 2026-09-23 19:40   ` sashiko-bot
  2026-09-24  9:43   ` Kumar, Udit
  1 sibling, 0 replies; 8+ messages in thread
From: sashiko-bot @ 2026-09-23 19:40 UTC (permalink / raw)
  To: Swamil Jain; +Cc: devicetree, robh, conor+dt

Thank you for your contribution! Sashiko AI review found 2 potential issue(s) to consider:
- [Medium] dts: missing reset-gpios in sii9022 HDMI bridge node
- [Medium] dts: unmanaged VOUT0_FET_SEL0 multiplexer signal
--

commit 4ee6007c0b1e46ee93bc1bd222e57649c25a3e88
Author: Devarsh Thakkar <devarsht@ti.com>

arm64: dts: ti: k3-am62l3-evm: Add HDMI support

Add the SII9022 HDMI bridge and HDMI connector nodes, along with the
DPI pinmux configuration connecting VOUT0 to the bridge. Enable the DSS
and hook up its DPI output port to the SII9022 bridge input, and the
bridge output to the HDMI connector.

> diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> index 84a0cfaaf196a..8ff7c4baaadba 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts

[ ... ]

> @@ -192,6 +208,41 @@ exp2: gpio@23 {
>  		bootph-all;
>  	};
>  
> +	sii9022: bridge-hdmi@3b {
> +		compatible = "sil,sii9022";
> +		reg = <0x3b>;
> +		interrupt-parent = <&exp1>;
> +		interrupts = <20 IRQ_TYPE_EDGE_FALLING>;

[Severity: Medium]
The exp2 IO expander defines GPIO_HDMI_RSTn on this board, which defaults to a
high-impedance input on power-up. Should this node define the reset-gpios
property to map and drive that line?

Without explicitly mapping and driving this line via reset-gpios, the HDMI
bridge may remain in reset or an undefined state. This could prevent the driver
from satisfying the hardware's reset timing requirements, potentially causing
I2C probe failures or non-functional HDMI output.

> +		#sound-dai-cells = <0>;
> +		sil,i2s-data-lanes = < 0 >;
> +		bootph-all;

[ ... ]

> @@ -221,6 +272,41 @@ usb_con_hs: endpoint {
>  };
>  
>  &pmx0 {
> +	dpi_pins_default: dpi-default-pins {
> +		pinctrl-single,pins = <
> +			AM62LX_IOPAD(0x00e4, PIN_OUTPUT, 1) /* (L20) GPMC0_CSn0.VOUT0_VSYNC */
> +			AM62LX_IOPAD(0x00dc, PIN_OUTPUT, 1) /* (N21) GPMC0_WPn.VOUT0_HSYNC */

[ ... ]

> +			AM62LX_IOPAD(0x00ec, PIN_INPUT,  5) /* (M23) GPMC0_CSn2.VOUT0_EXTPCLKIN */
> +		>;
> +		bootph-all;
> +	};

[Severity: Medium]
The exp2 IO expander defines a VOUT0_FET_SEL0 signal, which controls the FET
switch for routing the DSS VOUT0/DPI signals on the board. Should this patch
configure that pin (e.g., via a gpio-hog or within a display controller node)?

Since the TCA6424 IO expander defaults to high-impedance inputs, the FET switch
is currently left unmanaged. If the default or floating hardware state does not
route the signals to the HDMI bridge, the hardware path is physically
disconnected and the display will remain blank despite software succeeding.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260923192643.1362891-1-s-jain1@ti.com?part=2

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v5 2/2] arm64: dts: ti: k3-am62l3-evm: Add HDMI support
  2026-09-23 19:26 ` [PATCH v5 2/2] arm64: dts: ti: k3-am62l3-evm: Add HDMI support Swamil Jain
  2026-09-23 19:40   ` sashiko-bot
@ 2026-09-24  9:43   ` Kumar, Udit
  2026-09-29 15:38     ` Nishanth Menon
  1 sibling, 1 reply; 8+ messages in thread
From: Kumar, Udit @ 2026-09-24  9:43 UTC (permalink / raw)
  To: Swamil Jain, nm, vigneshr, kristo, robh, krzk+dt, conor+dt,
	tomi.valkeinen
  Cc: devarsht, sebin.francis, praneeth, linux-arm-kernel, devicetree,
	linux-kernel, u-kumar1



On 9/24/2026 12:56 AM, Swamil Jain wrote:
> From: Devarsh Thakkar <devarsht@ti.com>
> 
> Add the SII9022 HDMI bridge and HDMI connector nodes, along with the
> DPI pinmux configuration connecting VOUT0 to the bridge. Enable the DSS
> and hook up its DPI output port to the SII9022 bridge input, and the
> bridge output to the HDMI connector.
> 
> Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
> Signed-off-by: Devarsh Thakkar <devarsht@ti.com>
> Signed-off-by: Swamil Jain <s-jain1@ti.com>
> ---
>  arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 106 +++++++++++++++++++++++
>  1 file changed, 106 insertions(+)
> 
> [..]>  
> +&dss {
> +	status = "okay";
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&dpi_pins_default>;
> +	bootph-all;

nit pick : child node's bootph property is carried forward to parent.

Reviewed-by: Udit Kumar <u-kumar1@ti.com>

> +};


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v5 1/2] arm64: dts: ti: k3-am62l-main: Add DSS & DSI related nodes
  2026-09-23 19:26 ` [PATCH v5 1/2] arm64: dts: ti: k3-am62l-main: Add DSS & DSI related nodes Swamil Jain
@ 2026-09-24  9:44   ` Kumar, Udit
  0 siblings, 0 replies; 8+ messages in thread
From: Kumar, Udit @ 2026-09-24  9:44 UTC (permalink / raw)
  To: Swamil Jain, nm, vigneshr, kristo, robh, krzk+dt, conor+dt,
	tomi.valkeinen
  Cc: devarsht, sebin.francis, praneeth, linux-arm-kernel, devicetree,
	linux-kernel, u-kumar1



On 9/24/2026 12:56 AM, Swamil Jain wrote:
> From: Devarsh Thakkar <devarsht@ti.com>
> 
> Add DSS, DSI and DPHY nodes to add base display support on AM62L[1].
> AM62L shares a single video port for DPI and DSI output.
> 
> [1]: https://www.ti.com/product/AM62L
> 
> Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
> Signed-off-by: Devarsh Thakkar <devarsht@ti.com>
> Signed-off-by: Swamil Jain <s-jain1@ti.com>
> ---
>  arch/arm64/boot/dts/ti/k3-am62l-main.dtsi | 60 +++++++++++++++++++++++
>  1 file changed, 60 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..a3199a2455c3 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi
> @@ -544,6 +544,66 @@ elm0: ecc@25010000 {
>  		status = "disabled";
>  	};
>  
> +	dphy_tx0: phy@301c0000 {
> +		compatible = "ti,j721e-dphy";
> [..]
Reviewed-by: Udit Kumar <u-kumar1@ti.com>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v5 2/2] arm64: dts: ti: k3-am62l3-evm: Add HDMI support
  2026-09-24  9:43   ` Kumar, Udit
@ 2026-09-29 15:38     ` Nishanth Menon
  2026-10-07 11:40       ` Swamil Jain
  0 siblings, 1 reply; 8+ messages in thread
From: Nishanth Menon @ 2026-09-29 15:38 UTC (permalink / raw)
  To: Kumar, Udit
  Cc: Swamil Jain, vigneshr, kristo, robh, krzk+dt, conor+dt,
	tomi.valkeinen, devarsht, sebin.francis, praneeth,
	linux-arm-kernel, devicetree, linux-kernel

On 15:13-20260924, Kumar, Udit wrote:
> 
> 
> On 9/24/2026 12:56 AM, Swamil Jain wrote:
> > From: Devarsh Thakkar <devarsht@ti.com>
> > 
> > Add the SII9022 HDMI bridge and HDMI connector nodes, along with the
> > DPI pinmux configuration connecting VOUT0 to the bridge. Enable the DSS
> > and hook up its DPI output port to the SII9022 bridge input, and the
> > bridge output to the HDMI connector.
> > 
> > Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
> > Signed-off-by: Devarsh Thakkar <devarsht@ti.com>
> > Signed-off-by: Swamil Jain <s-jain1@ti.com>
> > ---
> >  arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 106 +++++++++++++++++++++++
> >  1 file changed, 106 insertions(+)
> > 
> > [..]>  
> > +&dss {
> > +	status = "okay";
> > +	pinctrl-names = "default";
> > +	pinctrl-0 = <&dpi_pins_default>;
> > +	bootph-all;
> 
> nit pick : child node's bootph property is carried forward to parent.

drop the property please.

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D
https://ti.com/opensource

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v5 2/2] arm64: dts: ti: k3-am62l3-evm: Add HDMI support
  2026-09-29 15:38     ` Nishanth Menon
@ 2026-10-07 11:40       ` Swamil Jain
  0 siblings, 0 replies; 8+ messages in thread
From: Swamil Jain @ 2026-10-07 11:40 UTC (permalink / raw)
  To: Nishanth Menon, Kumar, Udit
  Cc: vigneshr, kristo, robh, krzk+dt, conor+dt, tomi.valkeinen,
	devarsht, sebin.francis, praneeth, linux-arm-kernel, devicetree,
	linux-kernel



On 9/29/26 21:08, Nishanth Menon wrote:
> On 15:13-20260924, Kumar, Udit wrote:
>>
>>
>> On 9/24/2026 12:56 AM, Swamil Jain wrote:
>>> From: Devarsh Thakkar <devarsht@ti.com>
>>>
>>> Add the SII9022 HDMI bridge and HDMI connector nodes, along with the
>>> DPI pinmux configuration connecting VOUT0 to the bridge. Enable the DSS
>>> and hook up its DPI output port to the SII9022 bridge input, and the
>>> bridge output to the HDMI connector.
>>>
>>> Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
>>> Signed-off-by: Devarsh Thakkar <devarsht@ti.com>
>>> Signed-off-by: Swamil Jain <s-jain1@ti.com>
>>> ---
>>>   arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 106 +++++++++++++++++++++++
>>>   1 file changed, 106 insertions(+)
>>>
>>> [..]>
>>> +&dss {
>>> +	status = "okay";
>>> +	pinctrl-names = "default";
>>> +	pinctrl-0 = <&dpi_pins_default>;
>>> +	bootph-all;
>>
>> nit pick : child node's bootph property is carried forward to parent.
> 
> drop the property please.
> 

Sent v6[1], avoiding redundant use of "bootph-all".
[1]: https://lore.kernel.org/all/20261007100707.1065564-1-s-jain1@ti.com/


Regards,
Swamil.

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2026-10-07 11:41 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-23 19:26 [PATCH v5 0/2] Add DSS support for AM62L SoC Swamil Jain
2026-09-23 19:26 ` [PATCH v5 1/2] arm64: dts: ti: k3-am62l-main: Add DSS & DSI related nodes Swamil Jain
2026-09-24  9:44   ` Kumar, Udit
2026-09-23 19:26 ` [PATCH v5 2/2] arm64: dts: ti: k3-am62l3-evm: Add HDMI support Swamil Jain
2026-09-23 19:40   ` sashiko-bot
2026-09-24  9:43   ` Kumar, Udit
2026-09-29 15:38     ` Nishanth Menon
2026-10-07 11:40       ` Swamil Jain

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox