Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

ARM: dts: rockchip: correct regulator power states for suspend

When getting translated from a downstream device tree that used slightly
different DT bindings, these regulators got labeled with the
"on-in-suspend" state, when they were actually supposed to be turned off
for S3 suspend. This was harmless, but not intentional, AFAICT.

Let's turn them off to get the optimal power state.

Signed-off-by: Brian Norris <briannorris@chromium.org>
Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
Signed-off-by: Heiko Stuebner <heiko@sntech.de>

authored by

Brian Norris and committed by
Heiko Stuebner
467fb18a 6a414e46

+4 -4
+1 -1
arch/arm/boot/dts/rk3288-veyron-jerry.dts
··· 99 99 regulator-min-microvolt = <1800000>; 100 100 regulator-max-microvolt = <1800000>; 101 101 regulator-state-mem { 102 - regulator-on-in-suspend; 102 + regulator-off-in-suspend; 103 103 }; 104 104 }; 105 105 };
+1 -1
arch/arm/boot/dts/rk3288-veyron-pinky.dts
··· 109 109 regulator-boot-on; 110 110 regulator-name = "vcc18_lcd"; 111 111 regulator-state-mem { 112 - regulator-on-in-suspend; 112 + regulator-off-in-suspend; 113 113 }; 114 114 }; 115 115 };
+2 -2
arch/arm/boot/dts/rk3288-veyron-sdmmc.dtsi
··· 93 93 regulator-min-microvolt = <3300000>; 94 94 regulator-max-microvolt = <3300000>; 95 95 regulator-state-mem { 96 - regulator-on-in-suspend; 96 + regulator-off-in-suspend; 97 97 }; 98 98 }; 99 99 ··· 102 102 regulator-min-microvolt = <3300000>; 103 103 regulator-max-microvolt = <3300000>; 104 104 regulator-state-mem { 105 - regulator-on-in-suspend; 105 + regulator-off-in-suspend; 106 106 }; 107 107 }; 108 108 };