RE: [PATCH V9 13/14] ARM: dts: snow: represent the connection between bridge and panel using videoport and endpoints

3 messages, 3 authors, 2015-02-04 · open the first message on its own page

RE: [PATCH V9 13/14] ARM: dts: snow: represent the connection between bridge and panel using videoport and endpoints

From: Kukjin Kim <hidden>
Date: 2015-01-23 07:01:41

Ajay Kumar wrote:
quoted hunk
Define videoports and use endpoints to describe the connection between
the encoder, bridge and the panel, instead of using phandles.

Signed-off-by: Ajay Kumar <redacted>
Acked-by: Inki Dae <inki.dae@samsung.com>
Tested-by: Rahul Sharma <redacted>
Tested-by: Javier Martinez Canillas <redacted>
Tested-by: Gustavo Padovan <redacted>
Tested-by: Sjoerd Simons <redacted>
---
 arch/arm/boot/dts/exynos5250-snow.dts |   30 ++++++++++++++++++++++++++++--
 1 file changed, 28 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
index b9aeec4..1bd5d3a 100644
--- a/arch/arm/boot/dts/exynos5250-snow.dts
+++ b/arch/arm/boot/dts/exynos5250-snow.dts
@@ -183,7 +183,20 @@
 			powerdown-gpios = <&gpy2 5 GPIO_ACTIVE_HIGH>;
 			reset-gpios = <&gpx1 5 GPIO_ACTIVE_HIGH>;
 			edid-emulation = <5>;
-			panel = <&panel>;
+
+			ports {
+				port at 0 {
+					bridge_out: endpoint {
+						remote-endpoint = <&panel_in>;
+					};
+				};
+
+				port at 1 {
+					bridge_in: endpoint {
+						remote-endpoint = <&dp_out>;
+					};
+				};
+			};
 		};
 	};
@@ -228,6 +241,12 @@
 		compatible = "auo,b116xw03";
 		power-supply = <&fet6>;
 		backlight = <&backlight>;
+
+		port {
+			panel_in: endpoint {
+				remote-endpoint = <&bridge_out>;
+			};
+		};
 	};
 };
@@ -242,7 +261,14 @@
 	samsung,link-rate = <0x0a>;
 	samsung,lane-count = <2>;
 	samsung,hpd-gpio = <&gpx0 7 GPIO_ACTIVE_HIGH>;
-	bridge = <&ptn3460>;
+
+	ports {
+		port at 0 {
+			dp_out: endpoint {
+				remote-endpoint = <&bridge_in>;
+			};
+		};
+	};
 };

 &ehci {
--
1.7.9.5
I'm fine on the DT changes in this series, shall I take 13/14 and 14/14 in
Samsung tree?

- Kukjin

Re: [PATCH V9 13/14] ARM: dts: snow: represent the connection between bridge and panel using videoport and endpoints

From: Ajay kumar <hidden>
Date: 2015-01-27 04:27:39

Hi Kukjin,

On Fri, Jan 23, 2015 at 12:31 PM, Kukjin Kim [off-list ref] wrote:
Ajay Kumar wrote:
quoted
Define videoports and use endpoints to describe the connection between
the encoder, bridge and the panel, instead of using phandles.

Signed-off-by: Ajay Kumar <redacted>
Acked-by: Inki Dae <inki.dae@samsung.com>
Tested-by: Rahul Sharma <redacted>
Tested-by: Javier Martinez Canillas <redacted>
Tested-by: Gustavo Padovan <redacted>
Tested-by: Sjoerd Simons <redacted>
---
 arch/arm/boot/dts/exynos5250-snow.dts |   30 ++++++++++++++++++++++++++++--
 1 file changed, 28 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
index b9aeec4..1bd5d3a 100644
--- a/arch/arm/boot/dts/exynos5250-snow.dts
+++ b/arch/arm/boot/dts/exynos5250-snow.dts
@@ -183,7 +183,20 @@
                      powerdown-gpios = <&gpy2 5 GPIO_ACTIVE_HIGH>;
                      reset-gpios = <&gpx1 5 GPIO_ACTIVE_HIGH>;
                      edid-emulation = <5>;
-                     panel = <&panel>;
+
+                     ports {
+                             port at 0 {
+                                     bridge_out: endpoint {
+                                             remote-endpoint = <&panel_in>;
+                                     };
+                             };
+
+                             port at 1 {
+                                     bridge_in: endpoint {
+                                             remote-endpoint = <&dp_out>;
+                                     };
+                             };
+                     };
              };
      };
@@ -228,6 +241,12 @@
              compatible = "auo,b116xw03";
              power-supply = <&fet6>;
              backlight = <&backlight>;
+
+             port {
+                     panel_in: endpoint {
+                             remote-endpoint = <&bridge_out>;
+                     };
+             };
      };
 };
@@ -242,7 +261,14 @@
      samsung,link-rate = <0x0a>;
      samsung,lane-count = <2>;
      samsung,hpd-gpio = <&gpx0 7 GPIO_ACTIVE_HIGH>;
-     bridge = <&ptn3460>;
+
+     ports {
+             port at 0 {
+                     dp_out: endpoint {
+                             remote-endpoint = <&bridge_in>;
+                     };
+             };
+     };
 };

 &ehci {
--
1.7.9.5
I'm fine on the DT changes in this series, shall I take 13/14 and 14/14 in
Samsung tree?
Yes, please take these patches.

Thanks and regards,
Ajay Kumar

Re: [PATCH V9 13/14] ARM: dts: snow: represent the connection between bridge and panel using videoport and endpoints

From: Kukjin Kim <kgene@kernel.org>
Date: 2015-02-04 15:30:47

On 01/27/15 13:27, Ajay kumar wrote:
Hi Kukjin,
Hi,

[...]
quoted
I'm fine on the DT changes in this series, shall I take 13/14 and 14/14 in
Samsung tree?
Yes, please take these patches.
Done, thanks.

- Kukjin
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help