* [PATCH v2 1/7] arm64: dts: ti: k3-j784s4-j742s2-main-common: add DSI & DSI PHY
2025-06-24 8:26 [PATCH v2 0/7] Add DSI display support for TI's Jacinto platforms Jayesh Choudhary
@ 2025-06-24 8:26 ` Jayesh Choudhary
2025-06-26 6:26 ` Vignesh Raghavendra
2025-06-24 8:26 ` [PATCH v2 2/7] arm64: dts: ti: k3-j784s4-j742s2-evm-common: Enable DisplayPort-1 Jayesh Choudhary
` (6 subsequent siblings)
7 siblings, 1 reply; 18+ messages in thread
From: Jayesh Choudhary @ 2025-06-24 8:26 UTC (permalink / raw)
To: nm, vigneshr, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1, j-choudhary
Add DT nodes for DPI to DSI Bridge and DSI Phy.
The DSI bridge is Cadence DSI and the PHY is a
Cadence DPHY with TI wrapper.
Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
---
.../dts/ti/k3-j784s4-j742s2-main-common.dtsi | 37 +++++++++++++++++++
1 file changed, 37 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi
index 363d68fec387..2413c4913a8b 100644
--- a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi
@@ -2517,6 +2517,43 @@ watchdog18: watchdog@2550000 {
status = "reserved";
};
+ dphy_tx0: phy@4480000 {
+ compatible = "ti,j721e-dphy";
+ reg = <0x0 0x04480000 0x0 0x1000>;
+ clocks = <&k3_clks 402 20>, <&k3_clks 402 3>;
+ clock-names = "psm", "pll_ref";
+ #phy-cells = <0>;
+ power-domains = <&k3_pds 402 TI_SCI_PD_EXCLUSIVE>;
+ assigned-clocks = <&k3_clks 402 3>;
+ assigned-clock-parents = <&k3_clks 402 4>;
+ assigned-clock-rates = <19200000>;
+ status = "disabled";
+ };
+
+ dsi0: dsi@4800000 {
+ compatible = "ti,j721e-dsi";
+ reg = <0x0 0x04800000 0x0 0x100000>, <0x0 0x04710000 0x0 0x100>;
+ clocks = <&k3_clks 215 2>, <&k3_clks 215 5>;
+ clock-names = "dsi_p_clk", "dsi_sys_clk";
+ power-domains = <&k3_pds 215 TI_SCI_PD_EXCLUSIVE>;
+ interrupt-parent = <&gic500>;
+ interrupts = <GIC_SPI 600 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>;
+ };
+ };
+ };
+
mhdp: bridge@a000000 {
compatible = "ti,j721e-mhdp8546";
reg = <0x0 0xa000000 0x0 0x30a00>,
--
2.34.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH v2 1/7] arm64: dts: ti: k3-j784s4-j742s2-main-common: add DSI & DSI PHY
2025-06-24 8:26 ` [PATCH v2 1/7] arm64: dts: ti: k3-j784s4-j742s2-main-common: add DSI & DSI PHY Jayesh Choudhary
@ 2025-06-26 6:26 ` Vignesh Raghavendra
2025-07-15 9:46 ` Jayesh Choudhary
0 siblings, 1 reply; 18+ messages in thread
From: Vignesh Raghavendra @ 2025-06-26 6:26 UTC (permalink / raw)
To: Jayesh Choudhary, nm, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1
Hi
On 24/06/25 13:56, Jayesh Choudhary wrote:
> Add DT nodes for DPI to DSI Bridge and DSI Phy.
> The DSI bridge is Cadence DSI and the PHY is a
> Cadence DPHY with TI wrapper.
>
> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
> ---
> .../dts/ti/k3-j784s4-j742s2-main-common.dtsi | 37 +++++++++++++++++++
> 1 file changed, 37 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi
> index 363d68fec387..2413c4913a8b 100644
> --- a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi
> @@ -2517,6 +2517,43 @@ watchdog18: watchdog@2550000 {
> status = "reserved";
> };
>
> + dphy_tx0: phy@4480000 {
> + compatible = "ti,j721e-dphy";
> + reg = <0x0 0x04480000 0x0 0x1000>;
Follow the convention of the file. Use:
reg = <0x00 0x04480000 0x00 0x1000>;
Please fix throughout the series.
> + clocks = <&k3_clks 402 20>, <&k3_clks 402 3>;
> + clock-names = "psm", "pll_ref";
> + #phy-cells = <0>;
> + power-domains = <&k3_pds 402 TI_SCI_PD_EXCLUSIVE>;
> + assigned-clocks = <&k3_clks 402 3>;
> + assigned-clock-parents = <&k3_clks 402 4>;
> + assigned-clock-rates = <19200000>;
> + status = "disabled";
> + };
> +
> + dsi0: dsi@4800000 {
> + compatible = "ti,j721e-dsi";
> + reg = <0x0 0x04800000 0x0 0x100000>, <0x0 0x04710000 0x0 0x100>;
> + clocks = <&k3_clks 215 2>, <&k3_clks 215 5>;
> + clock-names = "dsi_p_clk", "dsi_sys_clk";
> + power-domains = <&k3_pds 215 TI_SCI_PD_EXCLUSIVE>;
> + interrupt-parent = <&gic500>;
This is implied and can be dropped.
> + interrupts = <GIC_SPI 600 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>;
> + };
> + };
> + };
> +
> mhdp: bridge@a000000 {
> compatible = "ti,j721e-mhdp8546";
> reg = <0x0 0xa000000 0x0 0x30a00>,
--
Regards
Vignesh
https://ti.com/opensource
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2 1/7] arm64: dts: ti: k3-j784s4-j742s2-main-common: add DSI & DSI PHY
2025-06-26 6:26 ` Vignesh Raghavendra
@ 2025-07-15 9:46 ` Jayesh Choudhary
0 siblings, 0 replies; 18+ messages in thread
From: Jayesh Choudhary @ 2025-07-15 9:46 UTC (permalink / raw)
To: Vignesh Raghavendra, nm, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1
Hello Vignesh,
On 26/06/25 11:56, Vignesh Raghavendra wrote:
> Hi
>
> On 24/06/25 13:56, Jayesh Choudhary wrote:
>> Add DT nodes for DPI to DSI Bridge and DSI Phy.
>> The DSI bridge is Cadence DSI and the PHY is a
>> Cadence DPHY with TI wrapper.
>>
>> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
>> ---
>> .../dts/ti/k3-j784s4-j742s2-main-common.dtsi | 37 +++++++++++++++++++
>> 1 file changed, 37 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi
>> index 363d68fec387..2413c4913a8b 100644
>> --- a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi
>> @@ -2517,6 +2517,43 @@ watchdog18: watchdog@2550000 {
>> status = "reserved";
>> };
>>
>> + dphy_tx0: phy@4480000 {
>> + compatible = "ti,j721e-dphy";
>> + reg = <0x0 0x04480000 0x0 0x1000>;
>
> Follow the convention of the file. Use:
>
> reg = <0x00 0x04480000 0x00 0x1000>;
Okay I will add padding.
Will add it to 4th field as well as I am seeing it in some
nodes:
reg = <0x00 0x04480000 0x00 0x00001000>;
>
> Please fix throughout the series.
Okay.
>
>> + clocks = <&k3_clks 402 20>, <&k3_clks 402 3>;
>> + clock-names = "psm", "pll_ref";
>> + #phy-cells = <0>;
>> + power-domains = <&k3_pds 402 TI_SCI_PD_EXCLUSIVE>;
>> + assigned-clocks = <&k3_clks 402 3>;
>> + assigned-clock-parents = <&k3_clks 402 4>;
>> + assigned-clock-rates = <19200000>;
>> + status = "disabled";
>> + };
>> +
>> + dsi0: dsi@4800000 {
>> + compatible = "ti,j721e-dsi";
>> + reg = <0x0 0x04800000 0x0 0x100000>, <0x0 0x04710000 0x0 0x100>;
>> + clocks = <&k3_clks 215 2>, <&k3_clks 215 5>;
>> + clock-names = "dsi_p_clk", "dsi_sys_clk";
>> + power-domains = <&k3_pds 215 TI_SCI_PD_EXCLUSIVE>;
>
>> + interrupt-parent = <&gic500>;
>
> This is implied and can be dropped.
Will drop.
Warm Regards,
Jayesh
>
>> + interrupts = <GIC_SPI 600 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>;
>> + };
>> + };
>> + };
>> +
>> mhdp: bridge@a000000 {
>> compatible = "ti,j721e-mhdp8546";
>> reg = <0x0 0xa000000 0x0 0x30a00>,
>
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v2 2/7] arm64: dts: ti: k3-j784s4-j742s2-evm-common: Enable DisplayPort-1
2025-06-24 8:26 [PATCH v2 0/7] Add DSI display support for TI's Jacinto platforms Jayesh Choudhary
2025-06-24 8:26 ` [PATCH v2 1/7] arm64: dts: ti: k3-j784s4-j742s2-main-common: add DSI & DSI PHY Jayesh Choudhary
@ 2025-06-24 8:26 ` Jayesh Choudhary
2025-06-24 8:26 ` [PATCH v2 3/7] arm64: dts: ti: k3-j721s2-main: add DSI & DSI PHY Jayesh Choudhary
` (5 subsequent siblings)
7 siblings, 0 replies; 18+ messages in thread
From: Jayesh Choudhary @ 2025-06-24 8:26 UTC (permalink / raw)
To: nm, vigneshr, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1, j-choudhary
Enable DSI display for J784S4 EVM.
Add DT node for DSI-to-eDP bridge. The DSI to eDP bridge is
SN65DSI86 on the board.
Add the endpoint nodes to describe connection from:
DSS => DSI => SN65DSI86 bridge => DisplayPort-1
Set status for all required nodes for display as 'okay'.
Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
---
.../dts/ti/k3-j784s4-j742s2-evm-common.dtsi | 117 +++++++++++++++++-
1 file changed, 116 insertions(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-evm-common.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-evm-common.dtsi
index fa656b7b13a1..63ff3eaedd4a 100644
--- a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-evm-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-evm-common.dtsi
@@ -301,6 +301,52 @@ codec_audio: sound {
clock-names = "cpb-mcasp-auxclk", "cpb-mcasp-auxclk-48000",
"cpb-codec-scki", "cpb-codec-scki-48000";
};
+
+ vsys_io_1v8: regulator-vsys-io-1v8 {
+ compatible = "regulator-fixed";
+ regulator-name = "vsys_io_1v8";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ vsys_io_1v2: regulator-vsys-io-1v2 {
+ compatible = "regulator-fixed";
+ regulator-name = "vsys_io_1v2";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <1200000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ edp1_refclk: clock-edp1-refclk {
+ #clock-cells = <0>;
+ compatible = "fixed-clock";
+ clock-frequency = <19200000>;
+ };
+
+ dp1_pwr_3v3: regulator-dp1-prw {
+ compatible = "regulator-fixed";
+ regulator-name = "dp1-pwr";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
+ gpio = <&exp4 1 GPIO_ACTIVE_HIGH>;
+ enable-active-high;
+ };
+
+ dp1: connector-dp1 {
+ compatible = "dp-connector";
+ label = "DP1";
+ type = "full-size";
+ dp-pwr-supply = <&dp1_pwr_3v3>;
+
+ port {
+ dp1_connector_in: endpoint {
+ remote-endpoint = <&dp1_out>;
+ };
+ };
+ };
};
&wkup_gpio0 {
@@ -1340,12 +1386,24 @@ &mhdp {
};
&dss_ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
/* DP */
- port {
+ port@0 {
+ reg = <0>;
dpi0_out: endpoint {
remote-endpoint = <&dp0_in>;
};
};
+
+ /* DSI */
+ port@2 {
+ reg = <2>;
+ dpi2_out: endpoint {
+ remote-endpoint = <&dsi0_in>;
+ };
+ };
};
&main_i2c4 {
@@ -1360,6 +1418,63 @@ exp4: gpio@20 {
gpio-controller;
#gpio-cells = <2>;
};
+
+ dsi_edp_bridge: dsi-edp-bridge@2c {
+ compatible = "ti,sn65dsi86";
+ reg = <0x2c>;
+
+ clock-names = "refclk";
+ clocks = <&edp1_refclk>;
+
+ enable-gpios = <&exp4 2 GPIO_ACTIVE_HIGH>;
+
+ vpll-supply = <&vsys_io_1v8>;
+ vccio-supply = <&vsys_io_1v8>;
+ vcca-supply = <&vsys_io_1v2>;
+ vcc-supply = <&vsys_io_1v2>;
+
+ dsi_edp_bridge_ports: ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ port@0 {
+ reg = <0>;
+ dp1_in: endpoint {
+ remote-endpoint = <&dsi0_out>;
+ };
+ };
+
+ port@1 {
+ reg = <1>;
+ dp1_out: endpoint {
+ remote-endpoint = <&dp1_connector_in>;
+ };
+ };
+ };
+ };
+};
+
+&dsi0_ports {
+ port@0 {
+ reg = <0>;
+ dsi0_out: endpoint {
+ remote-endpoint = <&dp1_in>;
+ };
+ };
+
+ port@1 {
+ reg = <1>;
+ dsi0_in: endpoint {
+ remote-endpoint = <&dpi2_out>;
+ };
+ };
+};
+
+&dphy_tx0 {
+ status = "okay";
+};
+
+&dsi0 {
+ status = "okay";
};
&dp0_ports {
--
2.34.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v2 3/7] arm64: dts: ti: k3-j721s2-main: add DSI & DSI PHY
2025-06-24 8:26 [PATCH v2 0/7] Add DSI display support for TI's Jacinto platforms Jayesh Choudhary
2025-06-24 8:26 ` [PATCH v2 1/7] arm64: dts: ti: k3-j784s4-j742s2-main-common: add DSI & DSI PHY Jayesh Choudhary
2025-06-24 8:26 ` [PATCH v2 2/7] arm64: dts: ti: k3-j784s4-j742s2-evm-common: Enable DisplayPort-1 Jayesh Choudhary
@ 2025-06-24 8:26 ` Jayesh Choudhary
2025-06-26 6:27 ` Vignesh Raghavendra
2025-06-24 8:26 ` [PATCH v2 4/7] arm64: dts: ti: k3-j721s2-common-proc-board: Add main_i2c4 instance Jayesh Choudhary
` (4 subsequent siblings)
7 siblings, 1 reply; 18+ messages in thread
From: Jayesh Choudhary @ 2025-06-24 8:26 UTC (permalink / raw)
To: nm, vigneshr, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1, j-choudhary
From: Rahul T R <r-ravikumar@ti.com>
Add DT nodes for DPI to DSI Bridge and DSI Phy.
The DSI bridge is Cadence DSI and the PHY is a
Cadence DPHY with TI wrapper.
Signed-off-by: Rahul T R <r-ravikumar@ti.com>
[j-choudhary@ti.com: disable dsi and dphy nodes, rename dphy node]
Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
---
arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi | 37 ++++++++++++++++++++++
1 file changed, 37 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
index 83cf0adb2cb7..e17fffc36248 100644
--- a/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
@@ -1795,6 +1795,43 @@ main_spi7: spi@2170000 {
status = "disabled";
};
+ dphy_tx0: phy@4480000 {
+ compatible = "ti,j721e-dphy";
+ reg = <0x0 0x04480000 0x0 0x1000>;
+ clocks = <&k3_clks 363 8>, <&k3_clks 363 14>;
+ clock-names = "psm", "pll_ref";
+ #phy-cells = <0>;
+ power-domains = <&k3_pds 363 TI_SCI_PD_EXCLUSIVE>;
+ assigned-clocks = <&k3_clks 363 14>;
+ assigned-clock-parents = <&k3_clks 363 15>;
+ assigned-clock-rates = <19200000>;
+ status = "disabled";
+ };
+
+ dsi0: dsi@4800000 {
+ compatible = "ti,j721e-dsi";
+ reg = <0x0 0x04800000 0x0 0x100000>, <0x0 0x04710000 0x0 0x100>;
+ clocks = <&k3_clks 154 4>, <&k3_clks 154 1>;
+ clock-names = "dsi_p_clk", "dsi_sys_clk";
+ power-domains = <&k3_pds 154 TI_SCI_PD_EXCLUSIVE>;
+ interrupt-parent = <&gic500>;
+ interrupts = <GIC_SPI 600 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>;
+ };
+ };
+ };
+
dss: dss@4a00000 {
compatible = "ti,j721e-dss";
reg = <0x00 0x04a00000 0x00 0x10000>, /* common_m */
--
2.34.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH v2 3/7] arm64: dts: ti: k3-j721s2-main: add DSI & DSI PHY
2025-06-24 8:26 ` [PATCH v2 3/7] arm64: dts: ti: k3-j721s2-main: add DSI & DSI PHY Jayesh Choudhary
@ 2025-06-26 6:27 ` Vignesh Raghavendra
0 siblings, 0 replies; 18+ messages in thread
From: Vignesh Raghavendra @ 2025-06-26 6:27 UTC (permalink / raw)
To: Jayesh Choudhary, nm, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1
On 24/06/25 13:56, Jayesh Choudhary wrote:
> From: Rahul T R <r-ravikumar@ti.com>
>
> Add DT nodes for DPI to DSI Bridge and DSI Phy.
> The DSI bridge is Cadence DSI and the PHY is a
> Cadence DPHY with TI wrapper.
>
> Signed-off-by: Rahul T R <r-ravikumar@ti.com>
> [j-choudhary@ti.com: disable dsi and dphy nodes, rename dphy node]
> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
> ---
> arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi | 37 ++++++++++++++++++++++
> 1 file changed, 37 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
> index 83cf0adb2cb7..e17fffc36248 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
> @@ -1795,6 +1795,43 @@ main_spi7: spi@2170000 {
> status = "disabled";
> };
>
> + dphy_tx0: phy@4480000 {
> + compatible = "ti,j721e-dphy";
> + reg = <0x0 0x04480000 0x0 0x1000>;
> + clocks = <&k3_clks 363 8>, <&k3_clks 363 14>;
> + clock-names = "psm", "pll_ref";
> + #phy-cells = <0>;
> + power-domains = <&k3_pds 363 TI_SCI_PD_EXCLUSIVE>;
> + assigned-clocks = <&k3_clks 363 14>;
> + assigned-clock-parents = <&k3_clks 363 15>;
> + assigned-clock-rates = <19200000>;
> + status = "disabled";
> + };
> +
> + dsi0: dsi@4800000 {
> + compatible = "ti,j721e-dsi";
> + reg = <0x0 0x04800000 0x0 0x100000>, <0x0 0x04710000 0x0 0x100>;
> + clocks = <&k3_clks 154 4>, <&k3_clks 154 1>;
> + clock-names = "dsi_p_clk", "dsi_sys_clk";
> + power-domains = <&k3_pds 154 TI_SCI_PD_EXCLUSIVE>;
> + interrupt-parent = <&gic500>;
> + interrupts = <GIC_SPI 600 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>;
> + };
Messed up indentation
> + };
> + };
> +
> dss: dss@4a00000 {
> compatible = "ti,j721e-dss";
> reg = <0x00 0x04a00000 0x00 0x10000>, /* common_m */
--
Regards
Vignesh
https://ti.com/opensource
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v2 4/7] arm64: dts: ti: k3-j721s2-common-proc-board: Add main_i2c4 instance
2025-06-24 8:26 [PATCH v2 0/7] Add DSI display support for TI's Jacinto platforms Jayesh Choudhary
` (2 preceding siblings ...)
2025-06-24 8:26 ` [PATCH v2 3/7] arm64: dts: ti: k3-j721s2-main: add DSI & DSI PHY Jayesh Choudhary
@ 2025-06-24 8:26 ` Jayesh Choudhary
2025-06-26 10:26 ` Kumar, Udit
2025-06-24 8:26 ` [PATCH v2 5/7] arm64: dts: ti: k3-j721s2-som-p0: add DSI to eDP Jayesh Choudhary
` (3 subsequent siblings)
7 siblings, 1 reply; 18+ messages in thread
From: Jayesh Choudhary @ 2025-06-24 8:26 UTC (permalink / raw)
To: nm, vigneshr, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1, j-choudhary
Add dt node for main_i2c4 instance along with required pinmuxing.
Also add the gpio expander 'exp4' required by display connector.
Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
---
.../dts/ti/k3-j721s2-common-proc-board.dts | 24 +++++++++++++++++++
1 file changed, 24 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
index e2fc1288ed07..793d50344fad 100644
--- a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
@@ -148,6 +148,13 @@ J721S2_IOPAD(0x060, PIN_INPUT_PULLUP, 13) /* (AC27) MCASP2_AXR1.I2C3_SDA */
>;
};
+ main_i2c4_pins_default: main-i2c4-default-pins {
+ pinctrl-single,pins = <
+ J721S2_IOPAD(0x014, PIN_INPUT_PULLUP, 8) /* (AD25) I2C4_SCL */
+ J721S2_IOPAD(0x010, PIN_INPUT_PULLUP, 8) /* (AF28) I2C4_SDA */
+ >;
+ };
+
main_i2c5_pins_default: main-i2c5-default-pins {
pinctrl-single,pins = <
J721S2_IOPAD(0x01c, PIN_INPUT, 8) /* (Y24) MCAN15_TX.I2C5_SCL */
@@ -370,6 +377,23 @@ exp2: gpio@22 {
};
};
+&main_i2c4 {
+ status = "okay";
+ pinctrl-names = "default";
+ pinctrl-0 = <&main_i2c4_pins_default>;
+ clock-frequency = <400000>;
+
+ exp4: gpio@20 {
+ compatible = "ti,tca6408";
+ reg = <0x20>;
+ gpio-controller;
+ #gpio-cells = <2>;
+ gpio-line-names = "DP0_PWR_SW_EN", "DP1_PWR_SW_EN", "UB981_PDB",
+ "UB981_GPIO0", "UB981_GPIO1", "UB981_GPIO2",
+ "UB981_GPIO3", "PWR_SW_CNTL_DSI0#";
+ };
+};
+
&main_i2c5 {
pinctrl-names = "default";
pinctrl-0 = <&main_i2c5_pins_default>;
--
2.34.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH v2 4/7] arm64: dts: ti: k3-j721s2-common-proc-board: Add main_i2c4 instance
2025-06-24 8:26 ` [PATCH v2 4/7] arm64: dts: ti: k3-j721s2-common-proc-board: Add main_i2c4 instance Jayesh Choudhary
@ 2025-06-26 10:26 ` Kumar, Udit
0 siblings, 0 replies; 18+ messages in thread
From: Kumar, Udit @ 2025-06-26 10:26 UTC (permalink / raw)
To: Jayesh Choudhary, nm, vigneshr, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1
On 6/24/2025 1:56 PM, Jayesh Choudhary wrote:
> Add dt node for main_i2c4 instance along with required pinmuxing.
> Also add the gpio expander 'exp4' required by display connector.
>
> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
> ---
> .../dts/ti/k3-j721s2-common-proc-board.dts | 24 +++++++++++++++++++
> 1 file changed, 24 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
> index e2fc1288ed07..793d50344fad 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
> +++ b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
> @@ -148,6 +148,13 @@ J721S2_IOPAD(0x060, PIN_INPUT_PULLUP, 13) /* (AC27) MCASP2_AXR1.I2C3_SDA */
> >;
> };
>
> + main_i2c4_pins_default: main-i2c4-default-pins {
> + pinctrl-single,pins = <
> + J721S2_IOPAD(0x014, PIN_INPUT_PULLUP, 8) /* (AD25) I2C4_SCL */
Please check once for i2c clock, pin should be in input or output pull
up mode.
> + J721S2_IOPAD(0x010, PIN_INPUT_PULLUP, 8) /* (AF28) I2C4_SDA */
> + >;
> + };
> +
> main_i2c5_pins_default: main-i2c5-default-pins {
> pinctrl-single,pins = <
> J721S2_IOPAD(0x01c, PIN_INPUT, 8) /* (Y24) MCAN15_TX.I2C5_SCL */
> @@ -370,6 +377,23 @@ exp2: gpio@22 {
> };
> };
>
> +&main_i2c4 {
> + status = "okay";
> + pinctrl-names = "default";
> + pinctrl-0 = <&main_i2c4_pins_default>;
> + clock-frequency = <400000>;
> +
> + exp4: gpio@20 {
> + compatible = "ti,tca6408";
> + reg = <0x20>;
> + gpio-controller;
> + #gpio-cells = <2>;
> + gpio-line-names = "DP0_PWR_SW_EN", "DP1_PWR_SW_EN", "UB981_PDB",
> + "UB981_GPIO0", "UB981_GPIO1", "UB981_GPIO2",
> + "UB981_GPIO3", "PWR_SW_CNTL_DSI0#";
> + };
> +};
> +
> &main_i2c5 {
> pinctrl-names = "default";
> pinctrl-0 = <&main_i2c5_pins_default>;
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v2 5/7] arm64: dts: ti: k3-j721s2-som-p0: add DSI to eDP
2025-06-24 8:26 [PATCH v2 0/7] Add DSI display support for TI's Jacinto platforms Jayesh Choudhary
` (3 preceding siblings ...)
2025-06-24 8:26 ` [PATCH v2 4/7] arm64: dts: ti: k3-j721s2-common-proc-board: Add main_i2c4 instance Jayesh Choudhary
@ 2025-06-24 8:26 ` Jayesh Choudhary
2025-06-24 8:26 ` [PATCH v2 6/7] arm64: dts: ti: k3-j721s2-common-proc-board: Enable DisplayPort-1 Jayesh Choudhary
` (2 subsequent siblings)
7 siblings, 0 replies; 18+ messages in thread
From: Jayesh Choudhary @ 2025-06-24 8:26 UTC (permalink / raw)
To: nm, vigneshr, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1, j-choudhary
From: Rahul T R <r-ravikumar@ti.com>
Add DT nodes for DSI to eDP bridge. The DSI to eDP
bridge is SN65DSI86 on SOM.
Signed-off-by: Rahul T R <r-ravikumar@ti.com>
Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
---
arch/arm64/boot/dts/ti/k3-j721s2-som-p0.dtsi | 52 ++++++++++++++++++++
1 file changed, 52 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-som-p0.dtsi b/arch/arm64/boot/dts/ti/k3-j721s2-som-p0.dtsi
index 54fc5c4f8c3f..b4fc669a36ae 100644
--- a/arch/arm64/boot/dts/ti/k3-j721s2-som-p0.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721s2-som-p0.dtsi
@@ -152,6 +152,30 @@ transceiver0: can-phy0 {
#phy-cells = <0>;
max-bitrate = <5000000>;
};
+
+ vsys_io_1v8: regulator-vsys-io-1v8 {
+ compatible = "regulator-fixed";
+ regulator-name = "vsys_io_1v8";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ vsys_io_1v2: regulator-vsys-io-1v2 {
+ compatible = "regulator-fixed";
+ regulator-name = "vsys_io_1v2";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <1200000>;
+ regulator-always-on;
+ regulator-boot-on;
+ };
+
+ edp1_refclk: clock-edp1-refclk {
+ #clock-cells = <0>;
+ compatible = "fixed-clock";
+ clock-frequency = <19200000>;
+ };
};
&wkup_pmx0 {
@@ -630,3 +654,31 @@ &c71_1 {
memory-region = <&c71_1_dma_memory_region>,
<&c71_1_memory_region>;
};
+
+&main_i2c4 {
+ dsi_edp_bridge: dsi-edp-bridge@2c {
+ compatible = "ti,sn65dsi86";
+ reg = <0x2c>;
+
+ clock-names = "refclk";
+ clocks = <&edp1_refclk>;
+
+ enable-gpios = <&exp_som 5 0>;
+
+ vpll-supply = <&vsys_io_1v8>;
+ vccio-supply = <&vsys_io_1v8>;
+ vcca-supply = <&vsys_io_1v2>;
+ vcc-supply = <&vsys_io_1v2>;
+
+ dsi_edp_bridge_ports: ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ port@0 {
+ reg = <0>;
+ };
+ port@1 {
+ reg = <1>;
+ };
+ };
+ };
+};
--
2.34.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v2 6/7] arm64: dts: ti: k3-j721s2-common-proc-board: Enable DisplayPort-1
2025-06-24 8:26 [PATCH v2 0/7] Add DSI display support for TI's Jacinto platforms Jayesh Choudhary
` (4 preceding siblings ...)
2025-06-24 8:26 ` [PATCH v2 5/7] arm64: dts: ti: k3-j721s2-som-p0: add DSI to eDP Jayesh Choudhary
@ 2025-06-24 8:26 ` Jayesh Choudhary
2025-06-26 10:34 ` Kumar, Udit
2025-06-24 8:26 ` [PATCH v2 7/7] arm64: dts: ti: k3-am68-sk: Enable DSI on DisplayPort-0 Jayesh Choudhary
2025-06-26 6:15 ` [PATCH v2 0/7] Add DSI display support for TI's Jacinto platforms Vignesh Raghavendra
7 siblings, 1 reply; 18+ messages in thread
From: Jayesh Choudhary @ 2025-06-24 8:26 UTC (permalink / raw)
To: nm, vigneshr, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1, j-choudhary
Enable DSI display for J721S2 EVM.
Add the endpoint nodes to describe connection from:
DSS => DSI Bridge => DSI to eDP bridge => DisplayPort-1
Set status for all required nodes for DisplayPort-1 as 'okay'.
Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
---
.../dts/ti/k3-j721s2-common-proc-board.dts | 89 +++++++++++++++++++
1 file changed, 89 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
index 793d50344fad..efe857a50bb1 100644
--- a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
@@ -93,6 +93,29 @@ vdd_sd_dv: gpio-regulator-TLV71033 {
<3300000 0x1>;
};
+ dp1_pwr_3v3: regulator-dp1-prw {
+ compatible = "regulator-fixed";
+ regulator-name = "dp1-pwr";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
+ gpio = <&exp4 1 GPIO_ACTIVE_HIGH>; /* P1 - DP1_PWR_SW_EN */
+ enable-active-high;
+ regulator-always-on;
+ };
+
+ dp1: connector-dp1 {
+ compatible = "dp-connector";
+ label = "DP1";
+ type = "full-size";
+ dp-pwr-supply = <&dp1_pwr_3v3>;
+
+ port {
+ dp1_connector_in: endpoint {
+ remote-endpoint = <&dp1_out>;
+ };
+ };
+ };
+
transceiver1: can-phy1 {
compatible = "ti,tcan1043";
#phy-cells = <0>;
@@ -563,3 +586,69 @@ &main_mcan5 {
pinctrl-0 = <&main_mcan5_pins_default>;
phys = <&transceiver4>;
};
+
+&dss {
+ /*
+ * DSS on J721S2-EVM supports DP on VP0 and DSI on VP2.
+ * These clock assignments are chosen to enable the following outputs:
+ * VP0 - DisplayPort SST
+ * VP2 - DSI
+ */
+ status = "okay";
+ assigned-clocks = <&k3_clks 158 2>,
+ <&k3_clks 158 14>;
+ assigned-clock-parents = <&k3_clks 158 3>,
+ <&k3_clks 158 16>;
+};
+
+&dss_ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ port@2 {
+ reg = <2>;
+ dpi2_out: endpoint {
+ remote-endpoint = <&dsi0_in>;
+ };
+ };
+};
+
+&dsi0_ports {
+ port@0 {
+ reg = <0>;
+ dsi0_out: endpoint {
+ remote-endpoint = <&dp1_in>;
+ };
+ };
+
+ port@1 {
+ reg = <1>;
+ dsi0_in: endpoint {
+ remote-endpoint = <&dpi2_out>;
+ };
+ };
+};
+
+&dsi_edp_bridge_ports {
+ port@0 {
+ reg = <0>;
+ dp1_in: endpoint {
+ remote-endpoint = <&dsi0_out>;
+ };
+ };
+
+ port@1 {
+ reg = <1>;
+ dp1_out: endpoint {
+ remote-endpoint = <&dp1_connector_in>;
+ };
+ };
+};
+
+&dphy_tx0 {
+ status = "okay";
+};
+
+&dsi0 {
+ status = "okay";
+};
--
2.34.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH v2 6/7] arm64: dts: ti: k3-j721s2-common-proc-board: Enable DisplayPort-1
2025-06-24 8:26 ` [PATCH v2 6/7] arm64: dts: ti: k3-j721s2-common-proc-board: Enable DisplayPort-1 Jayesh Choudhary
@ 2025-06-26 10:34 ` Kumar, Udit
2025-07-16 5:42 ` Jayesh Choudhary
0 siblings, 1 reply; 18+ messages in thread
From: Kumar, Udit @ 2025-06-26 10:34 UTC (permalink / raw)
To: Jayesh Choudhary, nm, vigneshr, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1
On 6/24/2025 1:56 PM, Jayesh Choudhary wrote:
> Enable DSI display for J721S2 EVM.
>
> Add the endpoint nodes to describe connection from:
> DSS => DSI Bridge => DSI to eDP bridge => DisplayPort-1
>
> Set status for all required nodes for DisplayPort-1 as 'okay'.
>
> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
> ---
> .../dts/ti/k3-j721s2-common-proc-board.dts | 89 +++++++++++++++++++
> 1 file changed, 89 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
> index 793d50344fad..efe857a50bb1 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
> +++ b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
> @@ -93,6 +93,29 @@ vdd_sd_dv: gpio-regulator-TLV71033 {
> <3300000 0x1>;
> };
>
> + dp1_pwr_3v3: regulator-dp1-prw {
> + compatible = "regulator-fixed";
> + regulator-name = "dp1-pwr";
> + regulator-min-microvolt = <3300000>;
> + regulator-max-microvolt = <3300000>;
> + gpio = <&exp4 1 GPIO_ACTIVE_HIGH>; /* P1 - DP1_PWR_SW_EN */
> + enable-active-high;
> + regulator-always-on;
Please check once, if this regulator falls under regulator-always-on case,
I can imagine a case, where DP1 is not used and still regulator is kept on
> + };
> +
> + dp1: connector-dp1 {
> + compatible = "dp-connector";
> + label = "DP1";
> + type = "full-size";
> + dp-pwr-supply = <&dp1_pwr_3v3>;
> +
> + port {
> + dp1_connector_in: endpoint {
> + remote-endpoint = <&dp1_out>;
> + };
> + };
> + };
> +
> transceiver1: can-phy1 {
> compatible = "ti,tcan1043";
> #phy-cells = <0>;
> @@ -563,3 +586,69 @@ &main_mcan5 {
> pinctrl-0 = <&main_mcan5_pins_default>;
> phys = <&transceiver4>;
> };
> +
> +&dss {
> + /*
> + * DSS on J721S2-EVM supports DP on VP0 and DSI on VP2.
> + * These clock assignments are chosen to enable the following outputs:
> + * VP0 - DisplayPort SST
> + * VP2 - DSI
> + */
> + status = "okay";
> + assigned-clocks = <&k3_clks 158 2>,
> + <&k3_clks 158 14>;
> + assigned-clock-parents = <&k3_clks 158 3>,
> + <&k3_clks 158 16>;
> +};
> +
> +&dss_ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + port@2 {
> + reg = <2>;
> + dpi2_out: endpoint {
> + remote-endpoint = <&dsi0_in>;
> + };
> + };
> +};
> +
> +&dsi0_ports {
> + port@0 {
> + reg = <0>;
> + dsi0_out: endpoint {
> + remote-endpoint = <&dp1_in>;
> + };
> + };
> +
> + port@1 {
> + reg = <1>;
> + dsi0_in: endpoint {
> + remote-endpoint = <&dpi2_out>;
> + };
> + };
> +};
> +
> +&dsi_edp_bridge_ports {
> + port@0 {
> + reg = <0>;
> + dp1_in: endpoint {
> + remote-endpoint = <&dsi0_out>;
> + };
> + };
> +
> + port@1 {
> + reg = <1>;
> + dp1_out: endpoint {
> + remote-endpoint = <&dp1_connector_in>;
> + };
> + };
> +};
> +
> +&dphy_tx0 {
> + status = "okay";
> +};
> +
> +&dsi0 {
> + status = "okay";
> +};
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2 6/7] arm64: dts: ti: k3-j721s2-common-proc-board: Enable DisplayPort-1
2025-06-26 10:34 ` Kumar, Udit
@ 2025-07-16 5:42 ` Jayesh Choudhary
0 siblings, 0 replies; 18+ messages in thread
From: Jayesh Choudhary @ 2025-07-16 5:42 UTC (permalink / raw)
To: Kumar, Udit, nm, vigneshr, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht
Hello Udit,
On 26/06/25 16:04, Kumar, Udit wrote:
>
> On 6/24/2025 1:56 PM, Jayesh Choudhary wrote:
>> Enable DSI display for J721S2 EVM.
>>
>> Add the endpoint nodes to describe connection from:
>> DSS => DSI Bridge => DSI to eDP bridge => DisplayPort-1
>>
>> Set status for all required nodes for DisplayPort-1 as 'okay'.
>>
>> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
>> ---
>> .../dts/ti/k3-j721s2-common-proc-board.dts | 89 +++++++++++++++++++
>> 1 file changed, 89 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
>> b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
>> index 793d50344fad..efe857a50bb1 100644
>> --- a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
>> @@ -93,6 +93,29 @@ vdd_sd_dv: gpio-regulator-TLV71033 {
>> <3300000 0x1>;
>> };
>> + dp1_pwr_3v3: regulator-dp1-prw {
>> + compatible = "regulator-fixed";
>> + regulator-name = "dp1-pwr";
>> + regulator-min-microvolt = <3300000>;
>> + regulator-max-microvolt = <3300000>;
>> + gpio = <&exp4 1 GPIO_ACTIVE_HIGH>; /* P1 - DP1_PWR_SW_EN */
>> + enable-active-high;
>> + regulator-always-on;
>
> Please check once, if this regulator falls under regulator-always-on case,
>
> I can imagine a case, where DP1 is not used and still regulator is kept on
Yes it is not required.
I will remove this property from j721s2-common-proc-board and am68-sk
board dts.
Thanks,
Jayesh
>
>
>> + };
>> +
>> + dp1: connector-dp1 {
>> + compatible = "dp-connector";
[...]
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v2 7/7] arm64: dts: ti: k3-am68-sk: Enable DSI on DisplayPort-0
2025-06-24 8:26 [PATCH v2 0/7] Add DSI display support for TI's Jacinto platforms Jayesh Choudhary
` (5 preceding siblings ...)
2025-06-24 8:26 ` [PATCH v2 6/7] arm64: dts: ti: k3-j721s2-common-proc-board: Enable DisplayPort-1 Jayesh Choudhary
@ 2025-06-24 8:26 ` Jayesh Choudhary
2025-06-26 6:34 ` Vignesh Raghavendra
2025-06-26 6:15 ` [PATCH v2 0/7] Add DSI display support for TI's Jacinto platforms Vignesh Raghavendra
7 siblings, 1 reply; 18+ messages in thread
From: Jayesh Choudhary @ 2025-06-24 8:26 UTC (permalink / raw)
To: nm, vigneshr, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1, j-choudhary
Enable DSI support for AM68-SK platform.
Add DT node for DSI2eDP bridge. The DSI to eDP bridge is sn65dsi86
on the board.
Add the endpoint nodes to describe connection from:
DSS => DSI => SN65DSI86 bridge => DisplayPort-0
Set status for all required nodes for DisplayPort-0 as 'okay'.
Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
---
.../boot/dts/ti/k3-am68-sk-base-board.dts | 96 +++++++++++++++++++
1 file changed, 96 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
index 5fa70a874d7b..aef63ae2994c 100644
--- a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
@@ -135,6 +135,35 @@ transceiver4: can-phy3 {
max-bitrate = <5000000>;
};
+ edp0_refclk: clock-edp0-refclk {
+ #clock-cells = <0>;
+ compatible = "fixed-clock";
+ clock-frequency = <19200000>;
+ };
+
+ dp0_pwr_3v3: fixedregulator-dp0-pwr {
+ compatible = "regulator-fixed";
+ regulator-name = "dp0-pwr";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
+ gpio = <&exp2 2 GPIO_ACTIVE_HIGH>; /*P0 - DP0_3V3 _EN */
+ enable-active-high;
+ regulator-always-on;
+ };
+
+ dp0: dp0-connector {
+ compatible = "dp-connector";
+ label = "DP0";
+ type = "full-size";
+ dp-pwr-supply = <&dp0_pwr_3v3>;
+
+ port {
+ dp0_connector_in: endpoint {
+ remote-endpoint = <&dp0_out>;
+ };
+ };
+ };
+
connector-hdmi {
compatible = "hdmi-connector";
label = "hdmi";
@@ -605,6 +634,39 @@ exp2: gpio@20 {
gpio-line-names = "HDMI_PDn","HDMI_LS_OE",
"DP0_3V3_EN","eDP_ENABLE";
};
+
+ dsi_edp_bridge: dsi-edp-bridge@2c {
+ compatible = "ti,sn65dsi86";
+ reg = <0x2c>;
+ clock-names = "refclk";
+ clocks = <&edp0_refclk>;
+ enable-gpios = <&exp2 3 GPIO_ACTIVE_HIGH>;
+ vpll-supply = <&vsys_io_1v8>;
+ vccio-supply = <&vsys_io_1v8>;
+ vcca-supply = <&vsys_io_1v2>;
+ vcc-supply = <&vsys_io_1v2>;
+
+ dsi_edp_bridge_ports: ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ port@0 {
+ reg = <0>;
+
+ dp0_in: endpoint {
+ remote-endpoint = <&dsi0_out>;
+ };
+ };
+
+ port@1 {
+ reg = <1>;
+
+ dp0_out: endpoint {
+ remote-endpoint = <&dp0_connector_in>;
+ };
+ };
+ };
+ };
};
&main_sdhci1 {
@@ -699,6 +761,15 @@ dpi_out0: endpoint {
remote-endpoint = <&tfp410_in>;
};
};
+
+ /* DSI */
+ port@2 {
+ reg = <2>;
+
+ dpi0_out: endpoint {
+ remote-endpoint = <&dsi0_in>;
+ };
+ };
};
&serdes_ln_ctrl {
@@ -756,3 +827,28 @@ &usb0 {
phys = <&serdes0_usb_link>;
phy-names = "cdns3,usb3-phy";
};
+
+&dphy_tx0 {
+ status = "okay";
+};
+
+&dsi0 {
+ status = "okay";
+};
+
+&dsi0_ports {
+
+ port@0 {
+ reg = <0>;
+ dsi0_out: endpoint {
+ remote-endpoint = <&dp0_in>;
+ };
+ };
+
+ port@1 {
+ reg = <1>;
+ dsi0_in: endpoint {
+ remote-endpoint = <&dpi0_out>;
+ };
+ };
+};
--
2.34.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH v2 7/7] arm64: dts: ti: k3-am68-sk: Enable DSI on DisplayPort-0
2025-06-24 8:26 ` [PATCH v2 7/7] arm64: dts: ti: k3-am68-sk: Enable DSI on DisplayPort-0 Jayesh Choudhary
@ 2025-06-26 6:34 ` Vignesh Raghavendra
2025-07-15 9:48 ` Jayesh Choudhary
0 siblings, 1 reply; 18+ messages in thread
From: Vignesh Raghavendra @ 2025-06-26 6:34 UTC (permalink / raw)
To: Jayesh Choudhary, nm, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1
On 24/06/25 13:56, Jayesh Choudhary wrote:
> Enable DSI support for AM68-SK platform.
>
> Add DT node for DSI2eDP bridge. The DSI to eDP bridge is sn65dsi86
> on the board.
>
> Add the endpoint nodes to describe connection from:
> DSS => DSI => SN65DSI86 bridge => DisplayPort-0
>
> Set status for all required nodes for DisplayPort-0 as 'okay'.
>
> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
> ---
> .../boot/dts/ti/k3-am68-sk-base-board.dts | 96 +++++++++++++++++++
> 1 file changed, 96 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
> index 5fa70a874d7b..aef63ae2994c 100644
> --- a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
> @@ -135,6 +135,35 @@ transceiver4: can-phy3 {
> max-bitrate = <5000000>;
> };
>
> + edp0_refclk: clock-edp0-refclk {
> + #clock-cells = <0>;
> + compatible = "fixed-clock";
> + clock-frequency = <19200000>;
> + };
> +
> + dp0_pwr_3v3: fixedregulator-dp0-pwr {
use std node name: regulator-...
> + compatible = "regulator-fixed";
> + regulator-name = "dp0-pwr";
> + regulator-min-microvolt = <3300000>;
> + regulator-max-microvolt = <3300000>;
> + gpio = <&exp2 2 GPIO_ACTIVE_HIGH>; /*P0 - DP0_3V3 _EN */
> + enable-active-high;
> + regulator-always-on;
> + };
> +
> + dp0: dp0-connector {
> + compatible = "dp-connector";
> + label = "DP0";
> + type = "full-size";
> + dp-pwr-supply = <&dp0_pwr_3v3>;
> +
> + port {
> + dp0_connector_in: endpoint {
> + remote-endpoint = <&dp0_out>;
> + };
> + };
> + };
> +
> connector-hdmi {
> compatible = "hdmi-connector";
> label = "hdmi";
> @@ -605,6 +634,39 @@ exp2: gpio@20 {
> gpio-line-names = "HDMI_PDn","HDMI_LS_OE",
> "DP0_3V3_EN","eDP_ENABLE";
> };
> +
> + dsi_edp_bridge: dsi-edp-bridge@2c {
> + compatible = "ti,sn65dsi86";
> + reg = <0x2c>;
> + clock-names = "refclk";
> + clocks = <&edp0_refclk>;
> + enable-gpios = <&exp2 3 GPIO_ACTIVE_HIGH>;
> + vpll-supply = <&vsys_io_1v8>;
> + vccio-supply = <&vsys_io_1v8>;
> + vcca-supply = <&vsys_io_1v2>;
> + vcc-supply = <&vsys_io_1v2>;
> +
> + dsi_edp_bridge_ports: ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + port@0 {
> + reg = <0>;
> +
> + dp0_in: endpoint {
> + remote-endpoint = <&dsi0_out>;
> + };
> + };
> +
> + port@1 {
> + reg = <1>;
> +
> + dp0_out: endpoint {
> + remote-endpoint = <&dp0_connector_in>;
> + };
> + };
> + };
> + };
> };
>
> &main_sdhci1 {
> @@ -699,6 +761,15 @@ dpi_out0: endpoint {
> remote-endpoint = <&tfp410_in>;
> };
> };
> +
> + /* DSI */
> + port@2 {
> + reg = <2>;
> +
> + dpi0_out: endpoint {
> + remote-endpoint = <&dsi0_in>;
> + };
> + };
> };
>
> &serdes_ln_ctrl {
> @@ -756,3 +827,28 @@ &usb0 {
> phys = <&serdes0_usb_link>;
> phy-names = "cdns3,usb3-phy";
> };
> +
> +&dphy_tx0 {
> + status = "okay";
> +};
> +
> +&dsi0 {
> + status = "okay";
> +};
> +
> +&dsi0_ports {
> +
> + port@0 {
> + reg = <0>;
> + dsi0_out: endpoint {
> + remote-endpoint = <&dp0_in>;
> + };
> + };
> +
> + port@1 {
> + reg = <1>;
> + dsi0_in: endpoint {
> + remote-endpoint = <&dpi0_out>;
> + };
> + };
> +};
--
Regards
Vignesh
https://ti.com/opensource
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2 7/7] arm64: dts: ti: k3-am68-sk: Enable DSI on DisplayPort-0
2025-06-26 6:34 ` Vignesh Raghavendra
@ 2025-07-15 9:48 ` Jayesh Choudhary
0 siblings, 0 replies; 18+ messages in thread
From: Jayesh Choudhary @ 2025-07-15 9:48 UTC (permalink / raw)
To: Vignesh Raghavendra, nm, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1
Hi,
On 26/06/25 12:04, Vignesh Raghavendra wrote:
>
>
> On 24/06/25 13:56, Jayesh Choudhary wrote:
>> Enable DSI support for AM68-SK platform.
>>
>> Add DT node for DSI2eDP bridge. The DSI to eDP bridge is sn65dsi86
>> on the board.
>>
>> Add the endpoint nodes to describe connection from:
>> DSS => DSI => SN65DSI86 bridge => DisplayPort-0
>>
>> Set status for all required nodes for DisplayPort-0 as 'okay'.
>>
>> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
>> ---
>> .../boot/dts/ti/k3-am68-sk-base-board.dts | 96 +++++++++++++++++++
>> 1 file changed, 96 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
>> index 5fa70a874d7b..aef63ae2994c 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
>> @@ -135,6 +135,35 @@ transceiver4: can-phy3 {
>> max-bitrate = <5000000>;
>> };
>>
>> + edp0_refclk: clock-edp0-refclk {
>> + #clock-cells = <0>;
>> + compatible = "fixed-clock";
>> + clock-frequency = <19200000>;
>> + };
>> +
>> + dp0_pwr_3v3: fixedregulator-dp0-pwr {
>
> use std node name: regulator-...
Okay! In v3, I will change this.
>
>> + compatible = "regulator-fixed";
>> + regulator-name = "dp0-pwr";
>> + regulator-min-microvolt = <3300000>;
>> + regulator-max-microvolt = <3300000>;
>> + gpio = <&exp2 2 GPIO_ACTIVE_HIGH>; /*P0 - DP0_3V3 _EN */
>> + enable-active-high;
>> + regulator-always-on;
>> + };
>> +
>> + dp0: dp0-connector {
>> + compatible = "dp-connector";
>> + label = "DP0";
>> + type = "full-size";
>> + dp-pwr-supply = <&dp0_pwr_3v3>;
>> +
>> + port {
>> + dp0_connector_in: endpoint {
>> + remote-endpoint = <&dp0_out>;
>> + };
>> + };
>> + };
>> +
>> connector-hdmi {
>> compatible = "hdmi-connector";
>> label = "hdmi";
>> @@ -605,6 +634,39 @@ exp2: gpio@20 {
>> gpio-line-names = "HDMI_PDn","HDMI_LS_OE",
>> "DP0_3V3_EN","eDP_ENABLE";
>> };
>> +
>> + dsi_edp_bridge: dsi-edp-bridge@2c {
>> + compatible = "ti,sn65dsi86";
>> + reg = <0x2c>;
>> + clock-names = "refclk";
>> + clocks = <&edp0_refclk>;
>> + enable-gpios = <&exp2 3 GPIO_ACTIVE_HIGH>;
>> + vpll-supply = <&vsys_io_1v8>;
>> + vccio-supply = <&vsys_io_1v8>;
>> + vcca-supply = <&vsys_io_1v2>;
>> + vcc-supply = <&vsys_io_1v2>;
>> +
>> + dsi_edp_bridge_ports: ports {
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> +
>> + port@0 {
>> + reg = <0>;
>> +
>> + dp0_in: endpoint {
>> + remote-endpoint = <&dsi0_out>;
>> + };
>> + };
>> +
>> + port@1 {
>> + reg = <1>;
>> +
>> + dp0_out: endpoint {
>> + remote-endpoint = <&dp0_connector_in>;
>> + };
>> + };
>> + };
>> + };
>> };
>>
>> &main_sdhci1 {
>> @@ -699,6 +761,15 @@ dpi_out0: endpoint {
>> remote-endpoint = <&tfp410_in>;
>> };
>> };
>> +
>> + /* DSI */
>> + port@2 {
>> + reg = <2>;
>> +
>> + dpi0_out: endpoint {
>> + remote-endpoint = <&dsi0_in>;
>> + };
>> + };
>> };
>>
>> &serdes_ln_ctrl {
>> @@ -756,3 +827,28 @@ &usb0 {
>> phys = <&serdes0_usb_link>;
>> phy-names = "cdns3,usb3-phy";
>> };
>> +
>> +&dphy_tx0 {
>> + status = "okay";
>> +};
>> +
>> +&dsi0 {
>> + status = "okay";
>> +};
>> +
>> +&dsi0_ports {
>> +
>> + port@0 {
>> + reg = <0>;
>> + dsi0_out: endpoint {
>> + remote-endpoint = <&dp0_in>;
>> + };
>> + };
>> +
>> + port@1 {
>> + reg = <1>;
>> + dsi0_in: endpoint {
>> + remote-endpoint = <&dpi0_out>;
>> + };
>> + };
>> +};
>
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2 0/7] Add DSI display support for TI's Jacinto platforms
2025-06-24 8:26 [PATCH v2 0/7] Add DSI display support for TI's Jacinto platforms Jayesh Choudhary
` (6 preceding siblings ...)
2025-06-24 8:26 ` [PATCH v2 7/7] arm64: dts: ti: k3-am68-sk: Enable DSI on DisplayPort-0 Jayesh Choudhary
@ 2025-06-26 6:15 ` Vignesh Raghavendra
2025-07-15 9:43 ` Jayesh Choudhary
7 siblings, 1 reply; 18+ messages in thread
From: Vignesh Raghavendra @ 2025-06-26 6:15 UTC (permalink / raw)
To: Jayesh Choudhary, nm, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1
On 24/06/25 13:56, Jayesh Choudhary wrote:
> Hello All,
>
> This series adds the dts support to enable DSI on 3 platforms for TI SoCs:
> - J784S4-EVM
> - J721S2-EVM
> - AM68-SK
>
> Relevant driver fixes, CDNS-DSI fixes[0] and SN65DSI86 detect fix[1]
> are Reviewed and Tested.
>
Are these merged?
[...]
--
Regards
Vignesh
https://ti.com/opensource
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2 0/7] Add DSI display support for TI's Jacinto platforms
2025-06-26 6:15 ` [PATCH v2 0/7] Add DSI display support for TI's Jacinto platforms Vignesh Raghavendra
@ 2025-07-15 9:43 ` Jayesh Choudhary
0 siblings, 0 replies; 18+ messages in thread
From: Jayesh Choudhary @ 2025-07-15 9:43 UTC (permalink / raw)
To: Vignesh Raghavendra, nm, devicetree
Cc: kristo, robh, krzk+dt, conor+dt, linux-arm-kernel, linux-kernel,
devarsht, u-kumar1
Hello Vignesh,
On 26/06/25 11:45, Vignesh Raghavendra wrote:
>
>
> On 24/06/25 13:56, Jayesh Choudhary wrote:
>> Hello All,
>>
>> This series adds the dts support to enable DSI on 3 platforms for TI SoCs:
>> - J784S4-EVM
>> - J721S2-EVM
>> - AM68-SK
>>
>> Relevant driver fixes, CDNS-DSI fixes[0] and SN65DSI86 detect fix[1]
>> are Reviewed and Tested.
>>
>
> Are these merged?
SN65DSI86 fix which was an essential patch is merged.
CDNS-DSI fixes will soon get merged (and that does not break these
integration patches for DSI).
Warm Regards,
Jayesh
>
> [...]
>
^ permalink raw reply [flat|nested] 18+ messages in thread