diff mbox

[2/2,v3] ARM: dts: imx7d-sdb: Pass 'enable-gpios' and

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

Commit Message

Marco Franchi July 14, 2017, 7:05 p.m. UTC 
Currently the LCD is turned on thanks to the bootloader initialization.

In order to make the kernel to turn on the LCD on is own, pass the
'enable-gpios' and 'power-supply' properties.

Also, the GPIO1_IO01 is not used as PWM functionality on this board. It is
connected to the PWREN pin of connector J14 and has a GPIO function, so 
remove the PWM1 node and change the GPIO1_IO01 IOMUX to GPIO function.

Signed-off-by: Marco Franchi <marco.franchi@nxp.com>
---
Changes since v2:
- none
 arch/arm/boot/dts/imx7d-sdb.dts | 21 +++++++++++++--------
 1 file changed, 13 insertions(+), 8 deletions(-)

Comments

Fabio Estevam July 14, 2017, 8:40 p.m. UTC | #1 
Hi Marco,

On Fri, Jul 14, 2017 at 4:05 PM, Marco Franchi <marco.franchi@nxp.com> wrote:
> Currently the LCD is turned on thanks to the bootloader initialization.
>
> In order to make the kernel to turn on the LCD on is own, pass the
> 'enable-gpios' and 'power-supply' properties.
>
> Also, the GPIO1_IO01 is not used as PWM functionality on this board. It is
> connected to the PWREN pin of connector J14 and has a GPIO function, so
> remove the PWM1 node and change the GPIO1_IO01 IOMUX to GPIO function.
>
> Signed-off-by: Marco Franchi <marco.franchi@nxp.com>

The word 'power-supply' is missing in the Subject.
diff mbox

Patch

diff --git a/arch/arm/boot/dts/imx7d-sdb.dts b/arch/arm/boot/dts/imx7d-sdb.dts
index 1600c7d..a01e6b3 100644
--- a/arch/arm/boot/dts/imx7d-sdb.dts
+++ b/arch/arm/boot/dts/imx7d-sdb.dts
@@ -118,8 +118,19 @@ 
 		startup-delay-us = <200000>;
 	};
 
+	reg_lcd_3v3: regulator-lcd-3v3 {
+		compatible = "regulator-fixed";
+		regulator-name = "lcd-3v3";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+		gpio = <&extended_io 7 GPIO_ACTIVE_LOW>;
+	};
+
 	panel {
 		compatible = "innolux,at043tn24";
+		pinctrl-0 = <&pinctrl_backlight>;
+		enable-gpios = <&gpio1 1 GPIO_ACTIVE_HIGH>;
+		power-supply = <&reg_lcd_3v3>;
 
 		port {
 			panel_in: endpoint {
@@ -347,12 +358,6 @@ 
 	status = "okay";
 };
 
-&pwm1 {
-	pinctrl-names = "default";
-	pinctrl-0 = <&pinctrl_pwm1>;
-	status = "okay";
-};
-
 &uart1 {
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_uart1>;
@@ -684,9 +689,9 @@ 
 		>;
 	};
 
-	pinctrl_pwm1: pwm1grp {
+	pinctrl_backlight: backlightgrp {
 		fsl,pins = <
-			MX7D_PAD_LPSR_GPIO1_IO01__PWM1_OUT		0x110b0
+			MX7D_PAD_LPSR_GPIO1_IO01__GPIO1_IO1		0x110b0
 		>;
 
 		pinctrl_spi4: spi4grp {