public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
* [PATCH 1/2 v2] ARM: dts: imx6ul-evk: Add DRM panel support
@ 2017-07-13 15:35 Marco Franchi
  2017-07-13 15:36 ` [PATCH 2/2 v2] ARM: dts: imx6ul-evk: Pass the 'backlight' property Marco Franchi
  2017-07-14  2:19 ` [PATCH 1/2 v2] ARM: dts: imx6ul-evk: Add DRM panel support Shawn Guo
  0 siblings, 2 replies; 3+ messages in thread
From: Marco Franchi @ 2017-07-13 15:35 UTC (permalink / raw)
  To: linux-arm-kernel

It is preferred to use the panel compatible string rather than passing the LCD
timming in the device tree.

So pass the "innolux,at043tn24" compatible string to describe the parallel LCD
on this board.

Signed-off-by: Marco Franchi <marco.franchi@nxp.com>
---
Changes since v1:
-Add missed space
 arch/arm/boot/dts/imx6ul-14x14-evk.dts | 36 ++++++++++++----------------------
 1 file changed, 13 insertions(+), 23 deletions(-)

diff --git a/arch/arm/boot/dts/imx6ul-14x14-evk.dts b/arch/arm/boot/dts/imx6ul-14x14-evk.dts
index d2be8aa..ed11cd1 100644
--- a/arch/arm/boot/dts/imx6ul-14x14-evk.dts
+++ b/arch/arm/boot/dts/imx6ul-14x14-evk.dts
@@ -78,6 +78,16 @@
 			clocks = <&clks IMX6UL_CLK_SAI2>;
 		};
 	};
+
+	panel {
+		compatible = "innolux,at043tn24";
+
+		port {
+			panel_in: endpoint {
+				remote-endpoint = <&display_out>;
+			};
+		};
+	};
 };
 
 &clks {
@@ -139,31 +149,11 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_lcdif_dat
 		     &pinctrl_lcdif_ctrl>;
-	display = <&display0>;
 	status = "okay";
 
-	display0: display {
-		bits-per-pixel = <16>;
-		bus-width = <24>;
-
-		display-timings {
-			native-mode = <&timing0>;
-
-			timing0: timing0 {
-				clock-frequency = <9200000>;
-				hactive = <480>;
-				vactive = <272>;
-				hfront-porch = <8>;
-				hback-porch = <4>;
-				hsync-len = <41>;
-				vback-porch = <2>;
-				vfront-porch = <4>;
-				vsync-len = <10>;
-				hsync-active = <0>;
-				vsync-active = <0>;
-				de-active = <1>;
-				pixelclk-active = <0>;
-			};
+	port {
+		display_out: endpoint {
+			remote-endpoint = <&panel_in>;
 		};
 	};
 };
-- 
2.7.4

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

* [PATCH 2/2 v2] ARM: dts: imx6ul-evk: Pass the 'backlight' property
  2017-07-13 15:35 [PATCH 1/2 v2] ARM: dts: imx6ul-evk: Add DRM panel support Marco Franchi
@ 2017-07-13 15:36 ` Marco Franchi
  2017-07-14  2:19 ` [PATCH 1/2 v2] ARM: dts: imx6ul-evk: Add DRM panel support Shawn Guo
  1 sibling, 0 replies; 3+ messages in thread
From: Marco Franchi @ 2017-07-13 15:36 UTC (permalink / raw)
  To: linux-arm-kernel

Pass the 'backlight' property as described in
Documentation/devicetree/bindings/display/panel/simple-panel.txt

Signed-off-by: Marco Franchi <marco.franchi@nxp.com>
---
Changes since v1:
-none
 arch/arm/boot/dts/imx6ul-14x14-evk.dts | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/imx6ul-14x14-evk.dts b/arch/arm/boot/dts/imx6ul-14x14-evk.dts
index ed11cd1..b3f72a3 100644
--- a/arch/arm/boot/dts/imx6ul-14x14-evk.dts
+++ b/arch/arm/boot/dts/imx6ul-14x14-evk.dts
@@ -22,7 +22,7 @@
 		reg = <0x80000000 0x20000000>;
 	};
 
-	backlight {
+	backlight_display: backlight-display {
 		compatible = "pwm-backlight";
 		pwms = <&pwm1 0 5000000>;
 		brightness-levels = <0 4 8 16 32 64 128 255>;
@@ -81,6 +81,7 @@
 
 	panel {
 		compatible = "innolux,at043tn24";
+		backlight = <&backlight_display>;
 
 		port {
 			panel_in: endpoint {
-- 
2.7.4

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

* [PATCH 1/2 v2] ARM: dts: imx6ul-evk: Add DRM panel support
  2017-07-13 15:35 [PATCH 1/2 v2] ARM: dts: imx6ul-evk: Add DRM panel support Marco Franchi
  2017-07-13 15:36 ` [PATCH 2/2 v2] ARM: dts: imx6ul-evk: Pass the 'backlight' property Marco Franchi
@ 2017-07-14  2:19 ` Shawn Guo
  1 sibling, 0 replies; 3+ messages in thread
From: Shawn Guo @ 2017-07-14  2:19 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Jul 13, 2017 at 12:35:59PM -0300, Marco Franchi wrote:
> It is preferred to use the panel compatible string rather than passing the LCD
> timming in the device tree.
> 
> So pass the "innolux,at043tn24" compatible string to describe the parallel LCD
> on this board.
> 
> Signed-off-by: Marco Franchi <marco.franchi@nxp.com>

Applied both, thanks.

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

end of thread, other threads:[~2017-07-14  2:19 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-07-13 15:35 [PATCH 1/2 v2] ARM: dts: imx6ul-evk: Add DRM panel support Marco Franchi
2017-07-13 15:36 ` [PATCH 2/2 v2] ARM: dts: imx6ul-evk: Pass the 'backlight' property Marco Franchi
2017-07-14  2:19 ` [PATCH 1/2 v2] ARM: dts: imx6ul-evk: Add DRM panel support Shawn Guo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox