devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [RESEND PATCH 0/3] ARM: dts: Enable suspend configuration
@ 2016-08-11  5:27 Keerthy
  2016-08-11  5:27 ` [RESEND PATCH 1/3] ARM: dts: AM437X-GP-EVM: AM437X-SK-EVM: Make dcdc3 dcdc5 and dcdc6 enable during suspend Keerthy
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Keerthy @ 2016-08-11  5:27 UTC (permalink / raw)
  To: tony-4v6yS6AI5VpBDgjK7y7TUQ
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
	broonie-DgEjT+Ai2ygdnm+yROfE0A, lee.jones-QSEj5FYQhm4dnm+yROfE0A,
	t-kristo-l0cyMroinI0, linux-omap-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA, j-keerthy-l0cyMroinI0

Now that https://lkml.org/lkml/2016/8/10/243 series is in. 
Here is the set of Device Tree patches which were dependent on
the driver patches.

These are boot tested on am437x-gp-evm.

Keerthy (1):
  ARM: dts: AM437X-GP-EVM: AM437X-SK-EVM: Make dcdc3 dcdc5 and dcdc6
    enable during suspend

Tero Kristo (2):
  ARM: dts: am437x-gp-evm: disable DDR regulator in rtc-only/poweroff
    mode
  ARM: dts: am437x-sk-evm: disable DDR regulator in rtc-only/poweroff
    mode

 arch/arm/boot/dts/am437x-gp-evm.dts | 13 +++++++++++++
 arch/arm/boot/dts/am437x-sk-evm.dts | 30 ++++++++++++++++++++++++++++++
 2 files changed, 43 insertions(+)

-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [RESEND PATCH 1/3] ARM: dts: AM437X-GP-EVM: AM437X-SK-EVM: Make dcdc3 dcdc5 and dcdc6 enable during suspend
  2016-08-11  5:27 [RESEND PATCH 0/3] ARM: dts: Enable suspend configuration Keerthy
@ 2016-08-11  5:27 ` Keerthy
  2016-08-11  5:27 ` [RESEND PATCH 2/3] ARM: dts: am437x-gp-evm: disable DDR regulator in rtc-only/poweroff mode Keerthy
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 6+ messages in thread
From: Keerthy @ 2016-08-11  5:27 UTC (permalink / raw)
  To: tony
  Cc: devicetree, robh+dt, broonie, lee.jones, t-kristo, linux-omap,
	linux-kernel, j-keerthy

dcdc3, dcdc5, dcdc6 supply ddr and rtc respectively. These
are required to be on during suspend. Hence set the state accordingly.

Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 arch/arm/boot/dts/am437x-gp-evm.dts | 10 ++++++++++
 arch/arm/boot/dts/am437x-sk-evm.dts | 27 +++++++++++++++++++++++++++
 2 files changed, 37 insertions(+)

diff --git a/arch/arm/boot/dts/am437x-gp-evm.dts b/arch/arm/boot/dts/am437x-gp-evm.dts
index 14677d5..fba26c9 100644
--- a/arch/arm/boot/dts/am437x-gp-evm.dts
+++ b/arch/arm/boot/dts/am437x-gp-evm.dts
@@ -537,7 +537,11 @@
 			regulator-max-microvolt = <1500000>;
 			regulator-boot-on;
 			regulator-always-on;
+			regulator-state-mem {
+				regulator-on-in-suspend;
+			};
 		};
+
 		dcdc5: regulator-dcdc5 {
 			compatible = "ti,tps65218-dcdc5";
 			regulator-name = "v1_0bat";
@@ -545,6 +549,9 @@
 			regulator-max-microvolt = <1000000>;
 			regulator-boot-on;
 			regulator-always-on;
+			regulator-state-mem {
+				regulator-on-in-suspend;
+			};
 		};
 
 		dcdc6: regulator-dcdc6 {
@@ -554,6 +561,9 @@
 			regulator-max-microvolt = <1800000>;
 			regulator-boot-on;
 			regulator-always-on;
+			regulator-state-mem {
+				regulator-on-in-suspend;
+			};
 		};
 
 		ldo1: regulator-ldo1 {
diff --git a/arch/arm/boot/dts/am437x-sk-evm.dts b/arch/arm/boot/dts/am437x-sk-evm.dts
index 5687d6b..b5e84e6 100644
--- a/arch/arm/boot/dts/am437x-sk-evm.dts
+++ b/arch/arm/boot/dts/am437x-sk-evm.dts
@@ -454,6 +454,9 @@
 			regulator-max-microvolt = <1500000>;
 			regulator-boot-on;
 			regulator-always-on;
+			regulator-state-mem {
+				regulator-on-in-suspend;
+			};
 		};
 
 		dcdc4: regulator-dcdc4 {
@@ -465,6 +468,30 @@
 			regulator-always-on;
 		};
 
+		dcdc5: regulator-dcdc5 {
+			compatible = "ti,tps65218-dcdc5";
+			regulator-name = "v1_0bat";
+			regulator-min-microvolt = <1000000>;
+			regulator-max-microvolt = <1000000>;
+			regulator-boot-on;
+			regulator-always-on;
+			regulator-state-mem {
+				regulator-on-in-suspend;
+			};
+		};
+
+		dcdc6: regulator-dcdc6 {
+			compatible = "ti,tps65218-dcdc6";
+			regulator-name = "v1_8bat";
+			regulator-min-microvolt = <1800000>;
+			regulator-max-microvolt = <1800000>;
+			regulator-boot-on;
+			regulator-always-on;
+			regulator-state-mem {
+				regulator-on-in-suspend;
+			};
+		};
+
 		ldo1: regulator-ldo1 {
 			compatible = "ti,tps65218-ldo1";
 			regulator-name = "v1_8d";
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [RESEND PATCH 2/3] ARM: dts: am437x-gp-evm: disable DDR regulator in rtc-only/poweroff mode
  2016-08-11  5:27 [RESEND PATCH 0/3] ARM: dts: Enable suspend configuration Keerthy
  2016-08-11  5:27 ` [RESEND PATCH 1/3] ARM: dts: AM437X-GP-EVM: AM437X-SK-EVM: Make dcdc3 dcdc5 and dcdc6 enable during suspend Keerthy
@ 2016-08-11  5:27 ` Keerthy
  2016-08-11  5:27 ` [RESEND PATCH 3/3] ARM: dts: am437x-sk-evm: " Keerthy
       [not found] ` <1470893270-5042-1-git-send-email-j-keerthy-l0cyMroinI0@public.gmane.org>
  3 siblings, 0 replies; 6+ messages in thread
From: Keerthy @ 2016-08-11  5:27 UTC (permalink / raw)
  To: tony
  Cc: devicetree, robh+dt, broonie, lee.jones, t-kristo, linux-omap,
	linux-kernel, j-keerthy, Dave Gerlach

From: Tero Kristo <t-kristo@ti.com>

Without this, the memory will remain active during poweroff consuming
extra power.

Signed-off-by: Tero Kristo <t-kristo@ti.com>
Signed-off-by: Dave Gerlach <d-gerlach@ti.com>
Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 arch/arm/boot/dts/am437x-gp-evm.dts | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/boot/dts/am437x-gp-evm.dts b/arch/arm/boot/dts/am437x-gp-evm.dts
index fba26c9..1cecc09 100644
--- a/arch/arm/boot/dts/am437x-gp-evm.dts
+++ b/arch/arm/boot/dts/am437x-gp-evm.dts
@@ -540,6 +540,9 @@
 			regulator-state-mem {
 				regulator-on-in-suspend;
 			};
+			regulator-state-disk {
+				regulator-off-in-suspend;
+			};
 		};
 
 		dcdc5: regulator-dcdc5 {
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [RESEND PATCH 3/3] ARM: dts: am437x-sk-evm: disable DDR regulator in rtc-only/poweroff mode
  2016-08-11  5:27 [RESEND PATCH 0/3] ARM: dts: Enable suspend configuration Keerthy
  2016-08-11  5:27 ` [RESEND PATCH 1/3] ARM: dts: AM437X-GP-EVM: AM437X-SK-EVM: Make dcdc3 dcdc5 and dcdc6 enable during suspend Keerthy
  2016-08-11  5:27 ` [RESEND PATCH 2/3] ARM: dts: am437x-gp-evm: disable DDR regulator in rtc-only/poweroff mode Keerthy
@ 2016-08-11  5:27 ` Keerthy
       [not found] ` <1470893270-5042-1-git-send-email-j-keerthy-l0cyMroinI0@public.gmane.org>
  3 siblings, 0 replies; 6+ messages in thread
From: Keerthy @ 2016-08-11  5:27 UTC (permalink / raw)
  To: tony
  Cc: devicetree, robh+dt, broonie, lee.jones, t-kristo, linux-omap,
	linux-kernel, j-keerthy, Dave Gerlach

From: Tero Kristo <t-kristo@ti.com>

Without this, the memory will remain active during poweroff consuming
extra power. Please note revision 2.1 PMIC seems to fail when DCDC3
disable is attempted, so this is not done on that PMIC revision. The
PMIC revision checks in the regulator patches make sure of this.

Signed-off-by: Tero Kristo <t-kristo@ti.com>
Signed-off-by: Dave Gerlach <d-gerlach@ti.com>
Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 arch/arm/boot/dts/am437x-sk-evm.dts | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/boot/dts/am437x-sk-evm.dts b/arch/arm/boot/dts/am437x-sk-evm.dts
index b5e84e6..3af92b6 100644
--- a/arch/arm/boot/dts/am437x-sk-evm.dts
+++ b/arch/arm/boot/dts/am437x-sk-evm.dts
@@ -457,6 +457,9 @@
 			regulator-state-mem {
 				regulator-on-in-suspend;
 			};
+			regulator-state-disk {
+				regulator-off-in-suspend;
+			};
 		};
 
 		dcdc4: regulator-dcdc4 {
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [RESEND PATCH 0/3] ARM: dts: Enable suspend configuration
       [not found] ` <1470893270-5042-1-git-send-email-j-keerthy-l0cyMroinI0@public.gmane.org>
@ 2016-08-23  4:38   ` Keerthy
       [not found]     ` <a22b1bd6-b8a2-6512-6c0e-a4cdd0c1d787-l0cyMroinI0@public.gmane.org>
  0 siblings, 1 reply; 6+ messages in thread
From: Keerthy @ 2016-08-23  4:38 UTC (permalink / raw)
  To: Keerthy, tony-4v6yS6AI5VpBDgjK7y7TUQ
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
	broonie-DgEjT+Ai2ygdnm+yROfE0A, lee.jones-QSEj5FYQhm4dnm+yROfE0A,
	t-kristo-l0cyMroinI0, linux-omap-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA

Tony,

On Thursday 11 August 2016 10:57 AM, Keerthy wrote:
> Now that https://lkml.org/lkml/2016/8/10/243 series is in.
> Here is the set of Device Tree patches which were dependent on
> the driver patches.
>

Could you please pull this set if there are no further comments. All the 
driver dependencies are picked by Mark already.

> These are boot tested on am437x-gp-evm.
>
> Keerthy (1):
>   ARM: dts: AM437X-GP-EVM: AM437X-SK-EVM: Make dcdc3 dcdc5 and dcdc6
>     enable during suspend
>
> Tero Kristo (2):
>   ARM: dts: am437x-gp-evm: disable DDR regulator in rtc-only/poweroff
>     mode
>   ARM: dts: am437x-sk-evm: disable DDR regulator in rtc-only/poweroff
>     mode
>
>  arch/arm/boot/dts/am437x-gp-evm.dts | 13 +++++++++++++
>  arch/arm/boot/dts/am437x-sk-evm.dts | 30 ++++++++++++++++++++++++++++++
>  2 files changed, 43 insertions(+)
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [RESEND PATCH 0/3] ARM: dts: Enable suspend configuration
       [not found]     ` <a22b1bd6-b8a2-6512-6c0e-a4cdd0c1d787-l0cyMroinI0@public.gmane.org>
@ 2016-08-26 14:48       ` Tony Lindgren
  0 siblings, 0 replies; 6+ messages in thread
From: Tony Lindgren @ 2016-08-26 14:48 UTC (permalink / raw)
  To: Keerthy
  Cc: Keerthy, devicetree-u79uwXL29TY76Z2rM5mHXA,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, broonie-DgEjT+Ai2ygdnm+yROfE0A,
	lee.jones-QSEj5FYQhm4dnm+yROfE0A, t-kristo-l0cyMroinI0,
	linux-omap-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA

* Keerthy <a0393675-l0cyMroinI0@public.gmane.org> [160822 21:39]:
> Tony,
> 
> On Thursday 11 August 2016 10:57 AM, Keerthy wrote:
> > Now that https://lkml.org/lkml/2016/8/10/243 series is in.
> > Here is the set of Device Tree patches which were dependent on
> > the driver patches.
> > 
> 
> Could you please pull this set if there are no further comments. All the
> driver dependencies are picked by Mark already.

Applying into omap-for-v4.9/dt finally thanks.

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-08-26 14:48 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-11  5:27 [RESEND PATCH 0/3] ARM: dts: Enable suspend configuration Keerthy
2016-08-11  5:27 ` [RESEND PATCH 1/3] ARM: dts: AM437X-GP-EVM: AM437X-SK-EVM: Make dcdc3 dcdc5 and dcdc6 enable during suspend Keerthy
2016-08-11  5:27 ` [RESEND PATCH 2/3] ARM: dts: am437x-gp-evm: disable DDR regulator in rtc-only/poweroff mode Keerthy
2016-08-11  5:27 ` [RESEND PATCH 3/3] ARM: dts: am437x-sk-evm: " Keerthy
     [not found] ` <1470893270-5042-1-git-send-email-j-keerthy-l0cyMroinI0@public.gmane.org>
2016-08-23  4:38   ` [RESEND PATCH 0/3] ARM: dts: Enable suspend configuration Keerthy
     [not found]     ` <a22b1bd6-b8a2-6512-6c0e-a4cdd0c1d787-l0cyMroinI0@public.gmane.org>
2016-08-26 14:48       ` Tony Lindgren

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).