ARM: dts: sabresd: use simple-panel instead of display-timings for LVDS1

PATCH New Fabio Estevam 2 participants 2 messages Jul 14, 2015 – Jul 27, 2015

Subsystems (2)

Device tree bindings Arm i.MX and Layerscape

lore

Share & tools

lore

Fabio Estevam

author

2015-07-14 14:15 UTC #0 lore raw

PATCH 1/1 ARM: dts: sabresd: use simple-panel instead of display-timings for LVDS1

We can use the compatible string "hannstar,hsd100pxn1" for representing the LCD panel that is connected to the LVDS1 port.

Signed-off-by Fabio Estevam <[email protected]>

---
 arch/arm/boot/dts/imx6qdl-sabresd.dtsi | 30 +++++++++++++++++-------------
 1 file changed, 17 insertions(+), 13 deletions(-)

arch/arm/boot/dts/imx6qdl-sabresd.dtsi +17 ~~-13~~

index cca847e..5fecb3a 100644
@@ -115,7 +115,7 @@
 		mux-ext-port = <3>;
 	};
 
-	backlight {
+	backlight_lvds: backlight_lvds {
 		compatible = "pwm-backlight";
 		pwms = <&pwm1 0 5000000>;
 		brightness-levels = <0 4 8 16 32 64 128 255>;

@@ -123,6 +123,17 @@
 		status = "okay";
 	};
 
+	panel {
+		compatible = "hannstar,hsd100pxn1";
+		backlight = <&backlight_lvds>;
+
+		port {
+			panel_in: endpoint {
+				remote-endpoint = <&lvds1_out>;
+			};
+		};
+	};
+
 	leds {
 		compatible = "gpio-leds";
 		pinctrl-names = "default";

@@ -508,18 +519,11 @@
 		fsl,data-width = <18>;
 		status = "okay";
 
-		display-timings {
-			native-mode = <&timing0>;
-			timing0: hsd100pxn1 {
-				clock-frequency = <65000000>;
-				hactive = <1024>;
-				vactive = <768>;
-				hback-porch = <220>;
-				hfront-porch = <40>;
-				vback-porch = <21>;
-				vfront-porch = <7>;
-				hsync-len = <60>;
-				vsync-len = <10>;
+		port at 4 {
+			reg = <4>;
+
+			lvds1_out: endpoint {
+				remote-endpoint = <&panel_in>;
 			};
 		};
 	};

Signature · 1 line

1.9.1

Shawn Guo

2015-07-27 14:44 UTC #1 lore raw

PATCH 1/1 ARM: dts: sabresd: use simple-panel instead of display-timings for LVDS1

Quoted 13 lines · Fabio Estevam

We can use the compatible string "hannstar,hsd100pxn1" for representing the LCD panel that is connected to the LVDS1 port.
Signed-off-by Fabio Estevam <[email protected]>
---
 arch/arm/boot/dts/imx6qdl-sabresd.dtsi | 30 +++++++++++++++++-------------
 1 file changed, 17 insertions(+), 13 deletions(-)
arch/arm/boot/dts/imx6qdl-sabresd.dtsi +0 -0
index cca847e..5fecb3a 100644

arch/arm/boot/dts/imx6qdl-sabresd.dtsi line 118

Quoted 5 lines

arch/arm/boot/dts/imx6qdl-sabresd.dtsi +1 -1

@@ -115,7 +115,7 @@
 		mux-ext-port = <3>;
 	};
 
-	backlight {
+	backlight_lvds: backlight_lvds {

Minus rather than underscore should be used in node name. I'm actually wondering whether the change is needed at all.

Shawn

Quoted 54 lines

arch/arm/boot/dts/imx6qdl-sabresd.dtsi +16 ~~-12~~

 		compatible = "pwm-backlight";
 		pwms = <&pwm1 0 5000000>;
 		brightness-levels = <0 4 8 16 32 64 128 255>;

@@ -123,6 +123,17 @@
 		status = "okay";
 	};
 
+	panel {
+		compatible = "hannstar,hsd100pxn1";
+		backlight = <&backlight_lvds>;
+
+		port {
+			panel_in: endpoint {
+				remote-endpoint = <&lvds1_out>;
+			};
+		};
+	};
+
 	leds {
 		compatible = "gpio-leds";
 		pinctrl-names = "default";

@@ -508,18 +519,11 @@
 		fsl,data-width = <18>;
 		status = "okay";
 
-		display-timings {
-			native-mode = <&timing0>;
-			timing0: hsd100pxn1 {
-				clock-frequency = <65000000>;
-				hactive = <1024>;
-				vactive = <768>;
-				hback-porch = <220>;
-				hfront-porch = <40>;
-				vback-porch = <21>;
-				vfront-porch = <7>;
-				hsync-len = <60>;
-				vsync-len = <10>;
+		port at 4 {
+			reg = <4>;
+
+			lvds1_out: endpoint {
+				remote-endpoint = <&panel_in>;
 			};
 		};
 	};

Signature · 7 lines

1.9.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel at lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel