[PATCH v2 3/3] ARM: dts: rockchip: correct regulator power states for suspend

Javier Martinez Canillas javier at dowhile0.org
Wed Aug 19 16:36:46 PDT 2015


Hello Brian,

On Thu, Aug 20, 2015 at 12:18 AM, Brian Norris <briannorris at chromium.org> wrote:
> 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 at chromium.org>
> Cc: Alexandru M Stan <amstan at chromium.org>
> Cc: Douglas Anderson <dianders at chromium.org>
> ---
> New in v2. Tested jaq.
>
>  arch/arm/boot/dts/rk3288-veyron-jerry.dts  | 2 +-
>  arch/arm/boot/dts/rk3288-veyron-pinky.dts  | 2 +-
>  arch/arm/boot/dts/rk3288-veyron-sdmmc.dtsi | 4 ++--
>  3 files changed, 4 insertions(+), 4 deletions(-)
>

Reviewed-by: Javier Martinez Canillas <javier at osg.samsung.com>

Best regards,
Javier



More information about the Linux-rockchip mailing list