2024-03-01 09:14:35

by Alexander Stein

Subject: [PATCH 0/5] i.MX8M add DSI output endpoint

Hi,

this series was initially triggered by dtbs_check warnings. Despite [1]
rendering them invalid, this series still simplifies the DSI OF graphs on all
i.MX8M variants by adding an empty output endpoint.
Each board enabling DSI connection can simply reference the label.
Patch 5 uses applies this to MBa8Mx board.

Best regards,
Alexander

[1] https://lore.kernel.org/linux-samsung-soc/[email protected]/

Alexander Stein (5):
arm64: dts: imx8mm: Add empty DSI output endpoint
arm64: dts: imx8mn: Add empty DSI output endpoint
arm64: dts: imx8mq: Add empty DSI output endpoint
arm64: dts: imx8mp: Add empty DSI output endpoint
arm64: dts: mba8mx: Simplify DSI connection

arch/arm64/boot/dts/freescale/imx8mm.dtsi | 7 +++++++
arch/arm64/boot/dts/freescale/imx8mn.dtsi | 7 +++++++
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 7 +++++++
arch/arm64/boot/dts/freescale/imx8mq.dtsi | 7 +++++++
arch/arm64/boot/dts/freescale/mba8mx.dtsi | 14 ++++----------
5 files changed, 32 insertions(+), 10 deletions(-)

--
2.34.1

2024-03-01 09:14:46

by Alexander Stein

Subject: [PATCH 1/5] arm64: dts: imx8mm: Add empty DSI output endpoint

Also add a corresponding label so overlays can reference that node.

Signed-off-by: Alexander Stein <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mm.dtsi | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
index 22b813f6d8405..dc8643d899612 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
@@ -1168,6 +1168,13 @@ dsim_from_lcdif: endpoint {
remote-endpoint = <&lcdif_to_dsim>;
};
};
+
+ port@1 {
+ reg = <1>;
+
+ mipi_dsi_out: endpoint {
+ };
+ };
};
};

--
2.34.1

2024-03-01 09:15:00

by Alexander Stein

Subject: [PATCH 2/5] arm64: dts: imx8mn: Add empty DSI output endpoint

Also add a corresponding label so overlays can reference that node.

Signed-off-by: Alexander Stein <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mn.dtsi | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mn.dtsi b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
index 619cedd8407c4..81ba70d7880b5 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
@@ -1104,6 +1104,13 @@ dsim_from_lcdif: endpoint {
remote-endpoint = <&lcdif_to_dsim>;
};
};
+
+ port@1 {
+ reg = <1>;
+
+ mipi_dsi_out: endpoint {
+ };
+ };
};
};

--
2.34.1

2024-03-01 09:15:11

by Alexander Stein

Subject: [PATCH 3/5] arm64: dts: imx8mq: Add empty DSI output endpoint

Also add a corresponding label so overlays can reference that node.

Signed-off-by: Alexander Stein <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mq.dtsi | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
index 74523fa69a5e7..5e906428d5e70 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
@@ -1290,6 +1290,13 @@ mipi_dsi_lcdif_in: endpoint@0 {
remote-endpoint = <&lcdif_mipi_dsi>;
};
};
+
+ port@1 {
+ reg = <1>;
+
+ mipi_dsi_out: endpoint {
+ };
+ };
};
};

--
2.34.1

2024-03-01 09:15:24

by Alexander Stein

Subject: [PATCH 4/5] arm64: dts: imx8mp: Add empty DSI output endpoint

Also add a corresponding label so overlays can reference that node.

Signed-off-by: Alexander Stein <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
index 6afefbb851842..922b7dc2edf66 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
@@ -1751,6 +1751,13 @@ dsim_from_lcdif1: endpoint {
remote-endpoint = <&lcdif1_to_dsim>;
};
};
+
+ port@1 {
+ reg = <1>;
+
+ mipi_dsi_out: endpoint {
+ };
+ };
};
};

--
2.34.1

2024-03-01 09:15:35

by Alexander Stein

Subject: [PATCH 5/5] arm64: dts: mba8mx: Simplify DSI connection

With the DSI output label in place, use the phandle directly.

Signed-off-by: Alexander Stein <[email protected]>
---
arch/arm64/boot/dts/freescale/mba8mx.dtsi | 14 ++++----------
1 file changed, 4 insertions(+), 10 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/mba8mx.dtsi b/arch/arm64/boot/dts/freescale/mba8mx.dtsi
index 9843c42a3eb49..daba4ba146277 100644
--- a/arch/arm64/boot/dts/freescale/mba8mx.dtsi
+++ b/arch/arm64/boot/dts/freescale/mba8mx.dtsi
@@ -359,17 +359,11 @@ lvds_bridge_out: endpoint {
&mipi_dsi {
samsung,burst-clock-frequency = <891000000>;
samsung,esc-clock-frequency = <20000000>;
+};

- ports {
- port@1 {
- reg = <1>;
-
- mipi_dsi_out: endpoint {
- data-lanes = <1 2 3 4>;
- remote-endpoint = <&lvds_bridge_in>;
- };
- };
- };
+&mipi_dsi_out {
+ data-lanes = <1 2 3 4>;
+ remote-endpoint = <&lvds_bridge_in>;
};

&pwm3 {
--
2.34.1

2024-03-29 12:17:54

by Shawn Guo

Subject: Re: [PATCH 0/5] i.MX8M add DSI output endpoint

On Fri, Mar 01, 2024 at 10:14:08AM +0100, Alexander Stein wrote:
> Hi,
>
> this series was initially triggered by dtbs_check warnings. Despite [1]
> rendering them invalid, this series still simplifies the DSI OF graphs on all
> i.MX8M variants by adding an empty output endpoint.
> Each board enabling DSI connection can simply reference the label.
> Patch 5 uses applies this to MBa8Mx board.
>
> Best regards,
> Alexander
>
> [1] https://lore.kernel.org/linux-samsung-soc/[email protected]/
>
> Alexander Stein (5):
> arm64: dts: imx8mm: Add empty DSI output endpoint
> arm64: dts: imx8mn: Add empty DSI output endpoint
> arm64: dts: imx8mq: Add empty DSI output endpoint
> arm64: dts: imx8mp: Add empty DSI output endpoint
> arm64: dts: mba8mx: Simplify DSI connection

Applied all, thanks!