Chris, On Mon, Nov 24, 2014 at 11:32 PM, Chris Zhong <zyw at rock-chips.com> wrote: > vcc_ddr: DCDC_REG3 { > regulator-always-on; > regulator-boot-on; > regulator-name = "vcc_ddr"; > + regulator-suspend-mem-enabled; The "regulator-suspend-mem-enabled" is not an upstream property and isn't doing anything. It should be removed. Possibly this is something Heiko could do when applying (depends on what he wants) if no other spins are required. > vcc_18: LDO_REG7 { > @@ -128,6 +163,11 @@ > regulator-min-microvolt = <1800000>; > regulator-max-microvolt = <1800000>; > regulator-name = "vcc_18"; > + regulator-suspend-mem-microvolt = <1800000>; Another non-upstream property: regulator-suspend-mem-microvolt