| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <5464198D.9030608@samsung.com>
Date: Thu, 13 Nov 2014 11:38:05 +0900
From: Kukjin Kim <kgene.kim@...sung.com>
To: Javier Martinez Canillas <javier.martinez@...labora.co.uk>
CC: Mark Brown <broonie@...nel.org>,
Kukjin Kim <kgene.kim@...sung.com>,
Chanwoo Choi <cw00.choi@...sung.com>,
Olof Johansson <olof@...om.net>,
Chris Zhong <zyw@...k-chips.com>,
Krzysztof Kozlowski <k.kozlowski@...sung.com>,
Abhilash Kesavan <kesavan.abhilash@...il.com>,
linux-samsung-soc@...r.kernel.org, linux-kernel@...r.kernel.org,
devicetree@...r.kernel.org
Subject: Re: [PATCH 3/3] ARM: dts: Configure regulators for suspend on exynos
Peach boards
On 11/11/14 21:04, Javier Martinez Canillas wrote:
> The regulator core now has support to choose if a regulator
> has to be enabled or disabled during system suspend and also
> supports changing the regulator operating mode during runtime
> and when the system enters into sleep mode.
>
> To lower power during suspend, configure the regulators state
> using the same configuration found in the ChromeOS 3.8 kernel.
>
Basically I'm fine on this dt changes but this should be queued after
confirming 1st and 2nd on this series from regulator maintainer...
Once they are landing into regulator tree, I will apply this.
Thanks,
Kukjin
> Signed-off-by: Javier Martinez Canillas<javier.martinez@...labora.co.uk>
> ---
> arch/arm/boot/dts/exynos5420-peach-pit.dts | 81 ++++++++++++++++++++++++++++++
> arch/arm/boot/dts/exynos5800-peach-pi.dts | 81 ++++++++++++++++++++++++++++++
> 2 files changed, 162 insertions(+)
>
> diff --git a/arch/arm/boot/dts/exynos5420-peach-pit.dts b/arch/arm/boot/dts/exynos5420-peach-pit.dts
> index 9a050e1..8b744c7 100644
> --- a/arch/arm/boot/dts/exynos5420-peach-pit.dts
> +++ b/arch/arm/boot/dts/exynos5420-peach-pit.dts
> @@ -13,6 +13,7 @@
> #include<dt-bindings/gpio/gpio.h>
> #include<dt-bindings/interrupt-controller/irq.h>
> #include<dt-bindings/clock/maxim,max77802.h>
> +#include<dt-bindings/regulator/maxim,max77802.h>
> #include "exynos5420.dtsi"
>
> / {
> @@ -192,6 +193,9 @@
> regulator-always-on;
> regulator-boot-on;
> regulator-ramp-delay =<12500>;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck2_reg: BUCK2 {
> @@ -201,6 +205,9 @@
> regulator-always-on;
> regulator-boot-on;
> regulator-ramp-delay =<12500>;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck3_reg: BUCK3 {
> @@ -210,6 +217,9 @@
> regulator-always-on;
> regulator-boot-on;
> regulator-ramp-delay =<12500>;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck4_reg: BUCK4 {
> @@ -219,6 +229,9 @@
> regulator-always-on;
> regulator-boot-on;
> regulator-ramp-delay =<12500>;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck5_reg: BUCK5 {
> @@ -227,6 +240,9 @@
> regulator-max-microvolt =<1200000>;
> regulator-always-on;
> regulator-boot-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck6_reg: BUCK6 {
> @@ -236,6 +252,9 @@
> regulator-always-on;
> regulator-boot-on;
> regulator-ramp-delay =<12500>;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck7_reg: BUCK7 {
> @@ -244,6 +263,9 @@
> regulator-max-microvolt =<1350000>;
> regulator-always-on;
> regulator-boot-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + };
> };
>
> buck8_reg: BUCK8 {
> @@ -252,6 +274,9 @@
> regulator-max-microvolt =<2850000>;
> regulator-always-on;
> regulator-boot-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck9_reg: BUCK9 {
> @@ -260,6 +285,9 @@
> regulator-max-microvolt =<2000000>;
> regulator-always-on;
> regulator-boot-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + };
> };
>
> buck10_reg: BUCK10 {
> @@ -268,6 +296,9 @@
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> regulator-boot-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + };
> };
>
> ldo1_reg: LDO1 {
> @@ -275,6 +306,10 @@
> regulator-min-microvolt =<1000000>;
> regulator-max-microvolt =<1000000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + regulator-mode =<MAX77802_OPMODE_LP>;
> + };
> };
>
> ldo2_reg: LDO2 {
> @@ -288,6 +323,10 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + regulator-mode =<MAX77802_OPMODE_LP>;
> + };
> };
>
> vqmmc_sdcard: ldo4_reg: LDO4 {
> @@ -295,6 +334,9 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<2800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo5_reg: LDO5 {
> @@ -302,6 +344,9 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo6_reg: LDO6 {
> @@ -309,6 +354,9 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo7_reg: LDO7 {
> @@ -322,6 +370,9 @@
> regulator-min-microvolt =<1000000>;
> regulator-max-microvolt =<1000000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo9_reg: LDO9 {
> @@ -329,6 +380,10 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + regulator-mode =<MAX77802_OPMODE_LP>;
> + };
> };
>
> ldo10_reg: LDO10 {
> @@ -336,6 +391,9 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo11_reg: LDO11 {
> @@ -343,6 +401,10 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + regulator-mode =<MAX77802_OPMODE_LP>;
> + };
> };
>
> ldo12_reg: LDO12 {
> @@ -350,6 +412,9 @@
> regulator-min-microvolt =<3000000>;
> regulator-max-microvolt =<3000000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo13_reg: LDO13 {
> @@ -357,6 +422,10 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + regulator-mode =<MAX77802_OPMODE_LP>;
> + };
> };
>
> ldo14_reg: LDO14 {
> @@ -364,6 +433,9 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo15_reg: LDO15 {
> @@ -371,6 +443,9 @@
> regulator-min-microvolt =<1000000>;
> regulator-max-microvolt =<1000000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo17_reg: LDO17 {
> @@ -378,6 +453,9 @@
> regulator-min-microvolt =<900000>;
> regulator-max-microvolt =<1400000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo18_reg: LDO18 {
> @@ -451,6 +529,9 @@
> regulator-min-microvolt =<1000000>;
> regulator-max-microvolt =<1000000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo32_reg: LDO32 {
> diff --git a/arch/arm/boot/dts/exynos5800-peach-pi.dts b/arch/arm/boot/dts/exynos5800-peach-pi.dts
> index e8fdda8..df7fbde 100644
> --- a/arch/arm/boot/dts/exynos5800-peach-pi.dts
> +++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts
> @@ -13,6 +13,7 @@
> #include<dt-bindings/gpio/gpio.h>
> #include<dt-bindings/interrupt-controller/irq.h>
> #include<dt-bindings/clock/maxim,max77802.h>
> +#include<dt-bindings/regulator/maxim,max77802.h>
> #include "exynos5800.dtsi"
>
> / {
> @@ -191,6 +192,9 @@
> regulator-always-on;
> regulator-boot-on;
> regulator-ramp-delay =<12500>;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck2_reg: BUCK2 {
> @@ -200,6 +204,9 @@
> regulator-always-on;
> regulator-boot-on;
> regulator-ramp-delay =<12500>;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck3_reg: BUCK3 {
> @@ -209,6 +216,9 @@
> regulator-always-on;
> regulator-boot-on;
> regulator-ramp-delay =<12500>;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck4_reg: BUCK4 {
> @@ -218,6 +228,9 @@
> regulator-always-on;
> regulator-boot-on;
> regulator-ramp-delay =<12500>;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck5_reg: BUCK5 {
> @@ -226,6 +239,9 @@
> regulator-max-microvolt =<1200000>;
> regulator-always-on;
> regulator-boot-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck6_reg: BUCK6 {
> @@ -235,6 +251,9 @@
> regulator-always-on;
> regulator-boot-on;
> regulator-ramp-delay =<12500>;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck7_reg: BUCK7 {
> @@ -243,6 +262,9 @@
> regulator-max-microvolt =<1350000>;
> regulator-always-on;
> regulator-boot-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + };
> };
>
> buck8_reg: BUCK8 {
> @@ -251,6 +273,9 @@
> regulator-max-microvolt =<2850000>;
> regulator-always-on;
> regulator-boot-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> buck9_reg: BUCK9 {
> @@ -259,6 +284,9 @@
> regulator-max-microvolt =<2000000>;
> regulator-always-on;
> regulator-boot-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + };
> };
>
> buck10_reg: BUCK10 {
> @@ -267,6 +295,9 @@
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> regulator-boot-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + };
> };
>
> ldo1_reg: LDO1 {
> @@ -274,6 +305,10 @@
> regulator-min-microvolt =<1000000>;
> regulator-max-microvolt =<1000000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + regulator-mode =<MAX77802_OPMODE_LP>;
> + };
> };
>
> ldo2_reg: LDO2 {
> @@ -287,6 +322,10 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + regulator-mode =<MAX77802_OPMODE_LP>;
> + };
> };
>
> vqmmc_sdcard: ldo4_reg: LDO4 {
> @@ -294,6 +333,9 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<2800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo5_reg: LDO5 {
> @@ -301,6 +343,9 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo6_reg: LDO6 {
> @@ -308,6 +353,9 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo7_reg: LDO7 {
> @@ -321,6 +369,9 @@
> regulator-min-microvolt =<1000000>;
> regulator-max-microvolt =<1000000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo9_reg: LDO9 {
> @@ -328,6 +379,10 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + regulator-mode =<MAX77802_OPMODE_LP>;
> + };
> };
>
> ldo10_reg: LDO10 {
> @@ -335,6 +390,9 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo11_reg: LDO11 {
> @@ -342,6 +400,10 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + regulator-mode =<MAX77802_OPMODE_LP>;
> + };
> };
>
> ldo12_reg: LDO12 {
> @@ -349,6 +411,9 @@
> regulator-min-microvolt =<3000000>;
> regulator-max-microvolt =<3000000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo13_reg: LDO13 {
> @@ -356,6 +421,10 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-on-in-suspend;
> + regulator-mode =<MAX77802_OPMODE_LP>;
> + };
> };
>
> ldo14_reg: LDO14 {
> @@ -363,6 +432,9 @@
> regulator-min-microvolt =<1800000>;
> regulator-max-microvolt =<1800000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo15_reg: LDO15 {
> @@ -370,6 +442,9 @@
> regulator-min-microvolt =<1000000>;
> regulator-max-microvolt =<1000000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo17_reg: LDO17 {
> @@ -377,6 +452,9 @@
> regulator-min-microvolt =<900000>;
> regulator-max-microvolt =<1400000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo18_reg: LDO18 {
> @@ -450,6 +528,9 @@
> regulator-min-microvolt =<1000000>;
> regulator-max-microvolt =<1000000>;
> regulator-always-on;
> + regulator-state-mem {
> + regulator-off-in-suspend;
> + };
> };
>
> ldo32_reg: LDO32 {
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists