Re: [PATCH V2] ARM: dts: imx6q-logicpd: Shutdown LCD regulator during suspend

2019-04-10 Thread Shawn Guo
On Tue, Apr 02, 2019 at 02:25:46PM -0500, Adam Ford wrote:
> The LCD power sequencer is very finicky.  The backlight cannot
> be driven until after the sequencer is done.  Until now, the
> regulators were marked with 'regulator-always-on' to make sure
> it came up before the backlight.  This patch allows the LCD
> regulators to power down and prevent the backlight from being
> used again until the sequencer is ready.  This reduces
> standby power consumption by ~100mW.
> 
> Signed-off-by: Adam Ford 

Applied, thanks.


[PATCH V2] ARM: dts: imx6q-logicpd: Shutdown LCD regulator during suspend

2019-04-02 Thread Adam Ford
The LCD power sequencer is very finicky.  The backlight cannot
be driven until after the sequencer is done.  Until now, the
regulators were marked with 'regulator-always-on' to make sure
it came up before the backlight.  This patch allows the LCD
regulators to power down and prevent the backlight from being
used again until the sequencer is ready.  This reduces
standby power consumption by ~100mW.

Signed-off-by: Adam Ford 
---
V2: Remove 'fixes' reference per Shawn's request

diff --git a/arch/arm/boot/dts/imx6q-logicpd.dts 
b/arch/arm/boot/dts/imx6q-logicpd.dts
index 45eb0b7f75f8..d96ae54be338 100644
--- a/arch/arm/boot/dts/imx6q-logicpd.dts
+++ b/arch/arm/boot/dts/imx6q-logicpd.dts
@@ -21,6 +21,8 @@
 
panel-lvds0 {
compatible = "okaya,rs800480t-7x0gp";
+   power-supply = <_lcd_reset>;
+   backlight = <>;
 
port {
panel_in_lvds0: endpoint {
@@ -38,7 +40,6 @@
regulator-max-microvolt = <330>;
gpio = < 17 GPIO_ACTIVE_HIGH>;
enable-active-high;
-   regulator-always-on;
vin-supply = <_3v3>;
startup-delay-us = <50>;
};
@@ -52,7 +53,6 @@
regulator-max-microvolt = <330>;
gpio = < 2 GPIO_ACTIVE_HIGH>;
enable-active-high;
-   regulator-always-on;
vin-supply = <_lcd>;
};
 };
-- 
2.17.1