[PATCH] ARM: dts: dra7-evm: Keep all VDD rails always-on

All of lore.kernel.org
 help / color / mirror / Atom feed

* [PATCH] ARM: dts: dra7-evm: Keep all VDD rails always-on
@ 2014-10-21 14:38 ` Nishanth Menon
  0 siblings, 0 replies; 9+ messages in thread
From: Nishanth Menon @ 2014-10-21 14:38 UTC (permalink / raw)
  To: Tony Lindgren
  Cc: linux-kernel, linux-arm-kernel, devicetree, linux-omap,
	Nishanth Menon

DRA7 Data Manual (SPRS857L - August 2014) section 4.1.1 states: "All
unused power supply balls must be supplied with the voltages specified
in the Section 5.2, Recommended Operating Conditions".

This implies that all unused voltage rails for Vayu can never be
switched off even if the hardware blocks inside that voltage domain is
unused. Switching off these unused rails may result in stability issues
on other domains and increased leakage and power-on-hour impacts.

J6eco-evm dts file already considers this, however j6evm-dts file needs
to be fixed to consider this constraint of the SoC.

Signed-off-by: Nishanth Menon <nm@ti.com>
---

Patch is based on v3.18-rc1 tag.

 arch/arm/boot/dts/dra7-evm.dts |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
index c6ce625..4f4c469 100644
--- a/arch/arm/boot/dts/dra7-evm.dts
+++ b/arch/arm/boot/dts/dra7-evm.dts
@@ -201,6 +201,7 @@
 					regulator-name = "smps45";
 					regulator-min-microvolt = < 850000>;
 					regulator-max-microvolt = <1150000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
@@ -209,6 +210,7 @@
 					regulator-name = "smps6";
 					regulator-min-microvolt = <850000>;
 					regulator-max-microvolt = <12500000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
@@ -226,6 +228,7 @@
 					regulator-name = "smps8";
 					regulator-min-microvolt = < 850000>;
 					regulator-max-microvolt = <1250000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
@@ -252,6 +255,7 @@
 					regulator-name = "ldo2";
 					regulator-min-microvolt = <3300000>;
 					regulator-max-microvolt = <3300000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
@@ -269,6 +273,7 @@
 					regulator-name = "ldo9";
 					regulator-min-microvolt = <1050000>;
 					regulator-max-microvolt = <1050000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
-- 
1.7.9.5

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

* [PATCH] ARM: dts: dra7-evm: Keep all VDD rails always-on
@ 2014-10-21 14:38 ` Nishanth Menon
  0 siblings, 0 replies; 9+ messages in thread
From: Nishanth Menon @ 2014-10-21 14:38 UTC (permalink / raw)
  To: Tony Lindgren
  Cc: linux-kernel, linux-arm-kernel, devicetree, linux-omap,
	Nishanth Menon

DRA7 Data Manual (SPRS857L - August 2014) section 4.1.1 states: "All
unused power supply balls must be supplied with the voltages specified
in the Section 5.2, Recommended Operating Conditions".

This implies that all unused voltage rails for Vayu can never be
switched off even if the hardware blocks inside that voltage domain is
unused. Switching off these unused rails may result in stability issues
on other domains and increased leakage and power-on-hour impacts.

J6eco-evm dts file already considers this, however j6evm-dts file needs
to be fixed to consider this constraint of the SoC.

Signed-off-by: Nishanth Menon <nm@ti.com>
---

Patch is based on v3.18-rc1 tag.

 arch/arm/boot/dts/dra7-evm.dts |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
index c6ce625..4f4c469 100644
--- a/arch/arm/boot/dts/dra7-evm.dts
+++ b/arch/arm/boot/dts/dra7-evm.dts
@@ -201,6 +201,7 @@
 					regulator-name = "smps45";
 					regulator-min-microvolt = < 850000>;
 					regulator-max-microvolt = <1150000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
@@ -209,6 +210,7 @@
 					regulator-name = "smps6";
 					regulator-min-microvolt = <850000>;
 					regulator-max-microvolt = <12500000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
@@ -226,6 +228,7 @@
 					regulator-name = "smps8";
 					regulator-min-microvolt = < 850000>;
 					regulator-max-microvolt = <1250000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
@@ -252,6 +255,7 @@
 					regulator-name = "ldo2";
 					regulator-min-microvolt = <3300000>;
 					regulator-max-microvolt = <3300000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
@@ -269,6 +273,7 @@
 					regulator-name = "ldo9";
 					regulator-min-microvolt = <1050000>;
 					regulator-max-microvolt = <1050000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
-- 
1.7.9.5


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

* [PATCH] ARM: dts: dra7-evm: Keep all VDD rails always-on
@ 2014-10-21 14:38 ` Nishanth Menon
  0 siblings, 0 replies; 9+ messages in thread
From: Nishanth Menon @ 2014-10-21 14:38 UTC (permalink / raw)
  To: linux-arm-kernel

DRA7 Data Manual (SPRS857L - August 2014) section 4.1.1 states: "All
unused power supply balls must be supplied with the voltages specified
in the Section 5.2, Recommended Operating Conditions".

This implies that all unused voltage rails for Vayu can never be
switched off even if the hardware blocks inside that voltage domain is
unused. Switching off these unused rails may result in stability issues
on other domains and increased leakage and power-on-hour impacts.

J6eco-evm dts file already considers this, however j6evm-dts file needs
to be fixed to consider this constraint of the SoC.

Signed-off-by: Nishanth Menon <nm@ti.com>
---

Patch is based on v3.18-rc1 tag.

 arch/arm/boot/dts/dra7-evm.dts |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
index c6ce625..4f4c469 100644
--- a/arch/arm/boot/dts/dra7-evm.dts
+++ b/arch/arm/boot/dts/dra7-evm.dts
@@ -201,6 +201,7 @@
 					regulator-name = "smps45";
 					regulator-min-microvolt = < 850000>;
 					regulator-max-microvolt = <1150000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
@@ -209,6 +210,7 @@
 					regulator-name = "smps6";
 					regulator-min-microvolt = <850000>;
 					regulator-max-microvolt = <12500000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
@@ -226,6 +228,7 @@
 					regulator-name = "smps8";
 					regulator-min-microvolt = < 850000>;
 					regulator-max-microvolt = <1250000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
@@ -252,6 +255,7 @@
 					regulator-name = "ldo2";
 					regulator-min-microvolt = <3300000>;
 					regulator-max-microvolt = <3300000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
@@ -269,6 +273,7 @@
 					regulator-name = "ldo9";
 					regulator-min-microvolt = <1050000>;
 					regulator-max-microvolt = <1050000>;
+					regulator-always-on;
 					regulator-boot-on;
 				};
 
-- 
1.7.9.5

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

* Re: [PATCH] ARM: dts: dra7-evm: Keep all VDD rails always-on
  2014-10-21 14:38 ` Nishanth Menon
  (?)
@ 2014-11-03 14:43   ` Roger Quadros
  -1 siblings, 0 replies; 9+ messages in thread
From: Roger Quadros @ 2014-11-03 14:43 UTC (permalink / raw)
  To: Nishanth Menon, Tony Lindgren
  Cc: linux-kernel, linux-arm-kernel, devicetree, linux-omap

Hi Nishant,

On 10/21/2014 05:38 PM, Nishanth Menon wrote:
> DRA7 Data Manual (SPRS857L - August 2014) section 4.1.1 states: "All
> unused power supply balls must be supplied with the voltages specified
> in the Section 5.2, Recommended Operating Conditions".
> 
> This implies that all unused voltage rails for Vayu can never be
> switched off even if the hardware blocks inside that voltage domain is
> unused. Switching off these unused rails may result in stability issues
> on other domains and increased leakage and power-on-hour impacts.
> 
> J6eco-evm dts file already considers this, however j6evm-dts file needs
> to be fixed to consider this constraint of the SoC.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>

Acked-by: Roger Quadros <rogerq@ti.com>

I'm assuming that vddshv8 (MMC1 power group) can be switched off.

cheers,
-roger

> ---
> 
> Patch is based on v3.18-rc1 tag.
> 
>  arch/arm/boot/dts/dra7-evm.dts |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
> index c6ce625..4f4c469 100644
> --- a/arch/arm/boot/dts/dra7-evm.dts
> +++ b/arch/arm/boot/dts/dra7-evm.dts
> @@ -201,6 +201,7 @@
>  					regulator-name = "smps45";
>  					regulator-min-microvolt = < 850000>;
>  					regulator-max-microvolt = <1150000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> @@ -209,6 +210,7 @@
>  					regulator-name = "smps6";
>  					regulator-min-microvolt = <850000>;
>  					regulator-max-microvolt = <12500000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> @@ -226,6 +228,7 @@
>  					regulator-name = "smps8";
>  					regulator-min-microvolt = < 850000>;
>  					regulator-max-microvolt = <1250000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> @@ -252,6 +255,7 @@
>  					regulator-name = "ldo2";
>  					regulator-min-microvolt = <3300000>;
>  					regulator-max-microvolt = <3300000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> @@ -269,6 +273,7 @@
>  					regulator-name = "ldo9";
>  					regulator-min-microvolt = <1050000>;
>  					regulator-max-microvolt = <1050000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> 

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

* Re: [PATCH] ARM: dts: dra7-evm: Keep all VDD rails always-on
@ 2014-11-03 14:43   ` Roger Quadros
  0 siblings, 0 replies; 9+ messages in thread
From: Roger Quadros @ 2014-11-03 14:43 UTC (permalink / raw)
  To: Nishanth Menon, Tony Lindgren
  Cc: linux-kernel, linux-arm-kernel, devicetree, linux-omap

Hi Nishant,

On 10/21/2014 05:38 PM, Nishanth Menon wrote:
> DRA7 Data Manual (SPRS857L - August 2014) section 4.1.1 states: "All
> unused power supply balls must be supplied with the voltages specified
> in the Section 5.2, Recommended Operating Conditions".
> 
> This implies that all unused voltage rails for Vayu can never be
> switched off even if the hardware blocks inside that voltage domain is
> unused. Switching off these unused rails may result in stability issues
> on other domains and increased leakage and power-on-hour impacts.
> 
> J6eco-evm dts file already considers this, however j6evm-dts file needs
> to be fixed to consider this constraint of the SoC.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>

Acked-by: Roger Quadros <rogerq@ti.com>

I'm assuming that vddshv8 (MMC1 power group) can be switched off.

cheers,
-roger

> ---
> 
> Patch is based on v3.18-rc1 tag.
> 
>  arch/arm/boot/dts/dra7-evm.dts |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
> index c6ce625..4f4c469 100644
> --- a/arch/arm/boot/dts/dra7-evm.dts
> +++ b/arch/arm/boot/dts/dra7-evm.dts
> @@ -201,6 +201,7 @@
>  					regulator-name = "smps45";
>  					regulator-min-microvolt = < 850000>;
>  					regulator-max-microvolt = <1150000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> @@ -209,6 +210,7 @@
>  					regulator-name = "smps6";
>  					regulator-min-microvolt = <850000>;
>  					regulator-max-microvolt = <12500000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> @@ -226,6 +228,7 @@
>  					regulator-name = "smps8";
>  					regulator-min-microvolt = < 850000>;
>  					regulator-max-microvolt = <1250000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> @@ -252,6 +255,7 @@
>  					regulator-name = "ldo2";
>  					regulator-min-microvolt = <3300000>;
>  					regulator-max-microvolt = <3300000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> @@ -269,6 +273,7 @@
>  					regulator-name = "ldo9";
>  					regulator-min-microvolt = <1050000>;
>  					regulator-max-microvolt = <1050000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> 


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

* [PATCH] ARM: dts: dra7-evm: Keep all VDD rails always-on
@ 2014-11-03 14:43   ` Roger Quadros
  0 siblings, 0 replies; 9+ messages in thread
From: Roger Quadros @ 2014-11-03 14:43 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Nishant,

On 10/21/2014 05:38 PM, Nishanth Menon wrote:
> DRA7 Data Manual (SPRS857L - August 2014) section 4.1.1 states: "All
> unused power supply balls must be supplied with the voltages specified
> in the Section 5.2, Recommended Operating Conditions".
> 
> This implies that all unused voltage rails for Vayu can never be
> switched off even if the hardware blocks inside that voltage domain is
> unused. Switching off these unused rails may result in stability issues
> on other domains and increased leakage and power-on-hour impacts.
> 
> J6eco-evm dts file already considers this, however j6evm-dts file needs
> to be fixed to consider this constraint of the SoC.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>

Acked-by: Roger Quadros <rogerq@ti.com>

I'm assuming that vddshv8 (MMC1 power group) can be switched off.

cheers,
-roger

> ---
> 
> Patch is based on v3.18-rc1 tag.
> 
>  arch/arm/boot/dts/dra7-evm.dts |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
> index c6ce625..4f4c469 100644
> --- a/arch/arm/boot/dts/dra7-evm.dts
> +++ b/arch/arm/boot/dts/dra7-evm.dts
> @@ -201,6 +201,7 @@
>  					regulator-name = "smps45";
>  					regulator-min-microvolt = < 850000>;
>  					regulator-max-microvolt = <1150000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> @@ -209,6 +210,7 @@
>  					regulator-name = "smps6";
>  					regulator-min-microvolt = <850000>;
>  					regulator-max-microvolt = <12500000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> @@ -226,6 +228,7 @@
>  					regulator-name = "smps8";
>  					regulator-min-microvolt = < 850000>;
>  					regulator-max-microvolt = <1250000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> @@ -252,6 +255,7 @@
>  					regulator-name = "ldo2";
>  					regulator-min-microvolt = <3300000>;
>  					regulator-max-microvolt = <3300000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> @@ -269,6 +273,7 @@
>  					regulator-name = "ldo9";
>  					regulator-min-microvolt = <1050000>;
>  					regulator-max-microvolt = <1050000>;
> +					regulator-always-on;
>  					regulator-boot-on;
>  				};
>  
> 

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

[parent not found: <54579476.10702-l0cyMroinI0@public.gmane.org>]

* Re: [PATCH] ARM: dts: dra7-evm: Keep all VDD rails always-on
  2014-11-03 14:43   ` Roger Quadros
  (?)
@ 2014-11-10 20:39       ` Tony Lindgren
  -1 siblings, 0 replies; 9+ messages in thread
From: Tony Lindgren @ 2014-11-10 20:39 UTC (permalink / raw)
  To: Roger Quadros
  Cc: Nishanth Menon, linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-omap-u79uwXL29TY76Z2rM5mHXA

* Roger Quadros <rogerq-l0cyMroinI0@public.gmane.org> [141103 06:44]:
> Hi Nishant,
> 
> On 10/21/2014 05:38 PM, Nishanth Menon wrote:
> > DRA7 Data Manual (SPRS857L - August 2014) section 4.1.1 states: "All
> > unused power supply balls must be supplied with the voltages specified
> > in the Section 5.2, Recommended Operating Conditions".
> > 
> > This implies that all unused voltage rails for Vayu can never be
> > switched off even if the hardware blocks inside that voltage domain is
> > unused. Switching off these unused rails may result in stability issues
> > on other domains and increased leakage and power-on-hour impacts.
> > 
> > J6eco-evm dts file already considers this, however j6evm-dts file needs
> > to be fixed to consider this constraint of the SoC.
> > 
> > Signed-off-by: Nishanth Menon <nm-l0cyMroinI0@public.gmane.org>
> 
> Acked-by: Roger Quadros <rogerq-l0cyMroinI0@public.gmane.org>
> 
> I'm assuming that vddshv8 (MMC1 power group) can be switched off.

Applying into omap-for-v3.19/dt thanks.

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] 9+ messages in thread

* Re: [PATCH] ARM: dts: dra7-evm: Keep all VDD rails always-on
@ 2014-11-10 20:39       ` Tony Lindgren
  0 siblings, 0 replies; 9+ messages in thread
From: Tony Lindgren @ 2014-11-10 20:39 UTC (permalink / raw)
  To: Roger Quadros
  Cc: Nishanth Menon, linux-kernel, linux-arm-kernel, devicetree,
	linux-omap

* Roger Quadros <rogerq@ti.com> [141103 06:44]:
> Hi Nishant,
> 
> On 10/21/2014 05:38 PM, Nishanth Menon wrote:
> > DRA7 Data Manual (SPRS857L - August 2014) section 4.1.1 states: "All
> > unused power supply balls must be supplied with the voltages specified
> > in the Section 5.2, Recommended Operating Conditions".
> > 
> > This implies that all unused voltage rails for Vayu can never be
> > switched off even if the hardware blocks inside that voltage domain is
> > unused. Switching off these unused rails may result in stability issues
> > on other domains and increased leakage and power-on-hour impacts.
> > 
> > J6eco-evm dts file already considers this, however j6evm-dts file needs
> > to be fixed to consider this constraint of the SoC.
> > 
> > Signed-off-by: Nishanth Menon <nm@ti.com>
> 
> Acked-by: Roger Quadros <rogerq@ti.com>
> 
> I'm assuming that vddshv8 (MMC1 power group) can be switched off.

Applying into omap-for-v3.19/dt thanks.

Tony

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

* [PATCH] ARM: dts: dra7-evm: Keep all VDD rails always-on
@ 2014-11-10 20:39       ` Tony Lindgren
  0 siblings, 0 replies; 9+ messages in thread
From: Tony Lindgren @ 2014-11-10 20:39 UTC (permalink / raw)
  To: linux-arm-kernel

* Roger Quadros <rogerq@ti.com> [141103 06:44]:
> Hi Nishant,
> 
> On 10/21/2014 05:38 PM, Nishanth Menon wrote:
> > DRA7 Data Manual (SPRS857L - August 2014) section 4.1.1 states: "All
> > unused power supply balls must be supplied with the voltages specified
> > in the Section 5.2, Recommended Operating Conditions".
> > 
> > This implies that all unused voltage rails for Vayu can never be
> > switched off even if the hardware blocks inside that voltage domain is
> > unused. Switching off these unused rails may result in stability issues
> > on other domains and increased leakage and power-on-hour impacts.
> > 
> > J6eco-evm dts file already considers this, however j6evm-dts file needs
> > to be fixed to consider this constraint of the SoC.
> > 
> > Signed-off-by: Nishanth Menon <nm@ti.com>
> 
> Acked-by: Roger Quadros <rogerq@ti.com>
> 
> I'm assuming that vddshv8 (MMC1 power group) can be switched off.

Applying into omap-for-v3.19/dt thanks.

Tony

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

end of thread, other threads:[~2014-11-10 20:40 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-21 14:38 [PATCH] ARM: dts: dra7-evm: Keep all VDD rails always-on Nishanth Menon
2014-10-21 14:38 ` Nishanth Menon
2014-10-21 14:38 ` Nishanth Menon
2014-11-03 14:43 ` Roger Quadros
2014-11-03 14:43   ` Roger Quadros
2014-11-03 14:43   ` Roger Quadros
     [not found]   ` <54579476.10702-l0cyMroinI0@public.gmane.org>
2014-11-10 20:39     ` Tony Lindgren
2014-11-10 20:39       ` Tony Lindgren
2014-11-10 20:39       ` Tony Lindgren

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.