devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/5] i.MX8M add DSI output endpoint
@ 2024-03-01  9:14 Alexander Stein
  2024-03-01  9:14 ` [PATCH 1/5] arm64: dts: imx8mm: Add empty " Alexander Stein
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: Alexander Stein @ 2024-03-01  9:14 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
  Cc: Alexander Stein, devicetree, imx, linux-arm-kernel, linux-kernel,
	linux

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/20231122-dtc-warnings-v2-1-bd4087325392@kernel.org/

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


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

* [PATCH 1/5] arm64: dts: imx8mm: Add empty DSI output endpoint
  2024-03-01  9:14 [PATCH 0/5] i.MX8M add DSI output endpoint Alexander Stein
@ 2024-03-01  9:14 ` Alexander Stein
  2024-03-01  9:14 ` [PATCH 2/5] arm64: dts: imx8mn: " Alexander Stein
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Alexander Stein @ 2024-03-01  9:14 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
  Cc: Alexander Stein, devicetree, imx, linux-arm-kernel, linux-kernel,
	linux

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

Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
 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


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

* [PATCH 2/5] arm64: dts: imx8mn: Add empty DSI output endpoint
  2024-03-01  9:14 [PATCH 0/5] i.MX8M add DSI output endpoint Alexander Stein
  2024-03-01  9:14 ` [PATCH 1/5] arm64: dts: imx8mm: Add empty " Alexander Stein
@ 2024-03-01  9:14 ` Alexander Stein
  2024-03-01  9:14 ` [PATCH 3/5] arm64: dts: imx8mq: " Alexander Stein
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Alexander Stein @ 2024-03-01  9:14 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
  Cc: Alexander Stein, devicetree, imx, linux-arm-kernel, linux-kernel,
	linux

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

Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
 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


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

* [PATCH 3/5] arm64: dts: imx8mq: Add empty DSI output endpoint
  2024-03-01  9:14 [PATCH 0/5] i.MX8M add DSI output endpoint Alexander Stein
  2024-03-01  9:14 ` [PATCH 1/5] arm64: dts: imx8mm: Add empty " Alexander Stein
  2024-03-01  9:14 ` [PATCH 2/5] arm64: dts: imx8mn: " Alexander Stein
@ 2024-03-01  9:14 ` Alexander Stein
  2024-03-01  9:14 ` [PATCH 4/5] arm64: dts: imx8mp: " Alexander Stein
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Alexander Stein @ 2024-03-01  9:14 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
  Cc: Alexander Stein, devicetree, imx, linux-arm-kernel, linux-kernel,
	linux

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

Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
 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


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

* [PATCH 4/5] arm64: dts: imx8mp: Add empty DSI output endpoint
  2024-03-01  9:14 [PATCH 0/5] i.MX8M add DSI output endpoint Alexander Stein
                   ` (2 preceding siblings ...)
  2024-03-01  9:14 ` [PATCH 3/5] arm64: dts: imx8mq: " Alexander Stein
@ 2024-03-01  9:14 ` Alexander Stein
  2024-03-01  9:14 ` [PATCH 5/5] arm64: dts: mba8mx: Simplify DSI connection Alexander Stein
  2024-03-29 12:16 ` [PATCH 0/5] i.MX8M add DSI output endpoint Shawn Guo
  5 siblings, 0 replies; 7+ messages in thread
From: Alexander Stein @ 2024-03-01  9:14 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
  Cc: Alexander Stein, devicetree, imx, linux-arm-kernel, linux-kernel,
	linux

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

Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
 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


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

* [PATCH 5/5] arm64: dts: mba8mx: Simplify DSI connection
  2024-03-01  9:14 [PATCH 0/5] i.MX8M add DSI output endpoint Alexander Stein
                   ` (3 preceding siblings ...)
  2024-03-01  9:14 ` [PATCH 4/5] arm64: dts: imx8mp: " Alexander Stein
@ 2024-03-01  9:14 ` Alexander Stein
  2024-03-29 12:16 ` [PATCH 0/5] i.MX8M add DSI output endpoint Shawn Guo
  5 siblings, 0 replies; 7+ messages in thread
From: Alexander Stein @ 2024-03-01  9:14 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
  Cc: Alexander Stein, devicetree, imx, linux-arm-kernel, linux-kernel,
	linux

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

Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
 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


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

* Re: [PATCH 0/5] i.MX8M add DSI output endpoint
  2024-03-01  9:14 [PATCH 0/5] i.MX8M add DSI output endpoint Alexander Stein
                   ` (4 preceding siblings ...)
  2024-03-01  9:14 ` [PATCH 5/5] arm64: dts: mba8mx: Simplify DSI connection Alexander Stein
@ 2024-03-29 12:16 ` Shawn Guo
  5 siblings, 0 replies; 7+ messages in thread
From: Shawn Guo @ 2024-03-29 12:16 UTC (permalink / raw)
  To: Alexander Stein
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, devicetree,
	imx, linux-arm-kernel, linux-kernel, linux

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/20231122-dtc-warnings-v2-1-bd4087325392@kernel.org/
> 
> 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!


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

end of thread, other threads:[~2024-03-29 12:17 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-01  9:14 [PATCH 0/5] i.MX8M add DSI output endpoint Alexander Stein
2024-03-01  9:14 ` [PATCH 1/5] arm64: dts: imx8mm: Add empty " Alexander Stein
2024-03-01  9:14 ` [PATCH 2/5] arm64: dts: imx8mn: " Alexander Stein
2024-03-01  9:14 ` [PATCH 3/5] arm64: dts: imx8mq: " Alexander Stein
2024-03-01  9:14 ` [PATCH 4/5] arm64: dts: imx8mp: " Alexander Stein
2024-03-01  9:14 ` [PATCH 5/5] arm64: dts: mba8mx: Simplify DSI connection Alexander Stein
2024-03-29 12:16 ` [PATCH 0/5] i.MX8M add DSI output endpoint Shawn Guo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).