diff mbox

[v3,2/2] ARM: dts: imx51: Fix inconsistent display port names

Message ID 1507213901-1349-2-git-send-email-marco.franchi@nxp.com (mailing list archive)
State New, archived
Headers show

Commit Message

Marco Franchi Oct. 5, 2017, 2:31 p.m. UTC 
Contrary to later i.MX SoCs, the parallel display interface pad groups on 
i.MX51 are called DISP1 and DISP2 in the Reference Manual, not DISP0 and 
DISP1.

Fix this inconsistence by changing the DISP names in the i.mx51 dts.

Signed-off-by: Marco Franchi <marco.franchi@nxp.com>
Acked-by: Philipp Zabel <p.zabel@pengutronix.de>
---
Changes since v2:
-change message log text
 arch/arm/boot/dts/imx51-apf51dev.dts |  4 ++--
 arch/arm/boot/dts/imx51-babbage.dts  | 12 ++++++------
 arch/arm/boot/dts/imx51-ts4800.dts   |  6 +++---
 arch/arm/boot/dts/imx51.dtsi         |  4 ++--
 4 files changed, 13 insertions(+), 13 deletions(-)

Comments

Fabio Estevam Oct. 5, 2017, 3:56 p.m. UTC | #1 
On Thu, Oct 5, 2017 at 11:31 AM, Marco Franchi <marco.franchi@nxp.com> wrote:
> Contrary to later i.MX SoCs, the parallel display interface pad groups on
> i.MX51 are called DISP1 and DISP2 in the Reference Manual, not DISP0 and
> DISP1.
>
> Fix this inconsistence by changing the DISP names in the i.mx51 dts.
>
> Signed-off-by: Marco Franchi <marco.franchi@nxp.com>
> Acked-by: Philipp Zabel <p.zabel@pengutronix.de>

Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com>
diff mbox

Patch

diff --git a/arch/arm/boot/dts/imx51-apf51dev.dts b/arch/arm/boot/dts/imx51-apf51dev.dts
index f04d0df..2f1a9d2 100644
--- a/arch/arm/boot/dts/imx51-apf51dev.dts
+++ b/arch/arm/boot/dts/imx51-apf51dev.dts
@@ -51,7 +51,7 @@ 
 
 		port {
 			display_in: endpoint {
-				remote-endpoint = <&ipu_di0_disp0>;
+				remote-endpoint = <&ipu_di0_disp1>;
 			};
 		};
 	};
@@ -218,6 +218,6 @@ 
 	};
 };
 
-&ipu_di0_disp0 {
+&ipu_di0_disp1 {
 	remote-endpoint = <&display_in>;
 };
diff --git a/arch/arm/boot/dts/imx51-babbage.dts b/arch/arm/boot/dts/imx51-babbage.dts
index 297953c..668c37b 100644
--- a/arch/arm/boot/dts/imx51-babbage.dts
+++ b/arch/arm/boot/dts/imx51-babbage.dts
@@ -39,7 +39,7 @@ 
 		};
 	};
 
-	display0: disp0 {
+	display1: disp1 {
 		compatible = "fsl,imx-parallel-display";
 		interface-pix-fmt = "rgb24";
 		pinctrl-names = "default";
@@ -61,12 +61,12 @@ 
 
 		port {
 			display0_in: endpoint {
-				remote-endpoint = <&ipu_di0_disp0>;
+				remote-endpoint = <&ipu_di0_disp1>;
 			};
 		};
 	};
 
-	display1: disp1 {
+	display2: disp2 {
 		compatible = "fsl,imx-parallel-display";
 		interface-pix-fmt = "rgb565";
 		pinctrl-names = "default";
@@ -93,7 +93,7 @@ 
 
 		port {
 			display1_in: endpoint {
-				remote-endpoint = <&ipu_di1_disp1>;
+				remote-endpoint = <&ipu_di1_disp2>;
 			};
 		};
 	};
@@ -348,11 +348,11 @@ 
 	};
 };
 
-&ipu_di0_disp0 {
+&ipu_di0_disp1 {
 	remote-endpoint = <&display0_in>;
 };
 
-&ipu_di1_disp1 {
+&ipu_di1_disp2 {
 	remote-endpoint = <&display1_in>;
 };
 
diff --git a/arch/arm/boot/dts/imx51-ts4800.dts b/arch/arm/boot/dts/imx51-ts4800.dts
index e6be869..35a1112 100644
--- a/arch/arm/boot/dts/imx51-ts4800.dts
+++ b/arch/arm/boot/dts/imx51-ts4800.dts
@@ -50,7 +50,7 @@ 
 		power-supply = <&backlight_reg>;
 	};
 
-	display0: disp0 {
+	display1: disp1 {
 		compatible = "fsl,imx-parallel-display";
 		interface-pix-fmt = "rgb24";
 		pinctrl-names = "default";
@@ -73,7 +73,7 @@ 
 
 		port@0 {
 			display0_in: endpoint {
-				remote-endpoint = <&ipu_di0_disp0>;
+				remote-endpoint = <&ipu_di0_disp1>;
 			};
 		};
 	};
@@ -107,7 +107,7 @@ 
 	};
 };
 
-&ipu_di0_disp0 {
+&ipu_di0_disp1 {
 	remote-endpoint = <&display0_in>;
 };
 
diff --git a/arch/arm/boot/dts/imx51.dtsi b/arch/arm/boot/dts/imx51.dtsi
index 1ee1d54..378be72 100644
--- a/arch/arm/boot/dts/imx51.dtsi
+++ b/arch/arm/boot/dts/imx51.dtsi
@@ -148,14 +148,14 @@ 
 			ipu_di0: port@2 {
 				reg = <2>;
 
-				ipu_di0_disp0: endpoint {
+				ipu_di0_disp1: endpoint {
 				};
 			};
 
 			ipu_di1: port@3 {
 				reg = <3>;
 
-				ipu_di1_disp1: endpoint {
+				ipu_di1_disp2: endpoint {
 				};
 			};
 		};