Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1403980 > unrolled thread

[PATCH] ARM: dts: exynos: Remove unneded always-on for regulators on Peach boards

Started byJavier Martinez Canillas <javier@osg.samsung.com>
First post2016-05-20 00:00 +0200
Last post2016-05-30 10:00 +0200
Articles 4 — 2 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH] ARM: dts: exynos: Remove unneded always-on for regulators on Peach boards Javier Martinez Canillas <javier@osg.samsung.com> - 2016-05-20 00:00 +0200
    Re: [PATCH] ARM: dts: exynos: Remove unneded always-on for regulators  on Peach boards Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2016-05-20 12:00 +0200
      Re: [PATCH] ARM: dts: exynos: Remove unneded always-on for regulators  on Peach boards Javier Martinez Canillas <javier@osg.samsung.com> - 2016-05-20 17:20 +0200
    Re: [PATCH] ARM: dts: exynos: Remove unneded always-on for regulators  on Peach boards Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2016-05-30 10:00 +0200

#1403980 — [PATCH] ARM: dts: exynos: Remove unneded always-on for regulators on Peach boards

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2016-05-20 00:00 +0200
Subject[PATCH] ARM: dts: exynos: Remove unneded always-on for regulators on Peach boards
Message-ID<rAE93-5ko-11@gated-at.bofh.it>
The regulator always-on property should only be used for regulators
that either can't be disabled or the drivers for the client devices
are not enabling the regulator and so being disabled due to unused.

There are some max77802 regulators in the Peach Pit and Pi boards
that are always-on but shouldn't since they are enabled by drivers:

BUCK5 is max77802 inl1 supply
BUCK7 is max77802 inl{2,10} supply
BUCK9 is max77802 inl{3,4,5,7} supply
BUCK10 is mmc1 vqmmc supply
LDO9 is adc vdd supply

This patch removes the always-on property for these regulators.

Some regulators should remain always-on even when they are used as
input supplies, because these regulators are also used by other
components that are not described in the DT, for example:

LDO10 is hdmi vdd_osc supply but it's also used as input supply for
the Exynos5420 temperature sensor.

LDO8 is hdmi vdd and vdd_pll supply but it's also used as supply for
the Exynos5420 Display port.

BUCK2 and BUCK6 are the vdd supplies for the EAGLE and KFC clusters.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 arch/arm/boot/dts/exynos5420-peach-pit.dts | 5 -----
 arch/arm/boot/dts/exynos5800-peach-pi.dts  | 5 -----
 2 files changed, 10 deletions(-)

diff --git a/arch/arm/boot/dts/exynos5420-peach-pit.dts b/arch/arm/boot/dts/exynos5420-peach-pit.dts
index f9d2e4f1a0e0..c1247402cb17 100644
--- a/arch/arm/boot/dts/exynos5420-peach-pit.dts
+++ b/arch/arm/boot/dts/exynos5420-peach-pit.dts
@@ -278,7 +278,6 @@
 				regulator-name = "vdd_1v2";
 				regulator-min-microvolt = <1200000>;
 				regulator-max-microvolt = <1200000>;
-				regulator-always-on;
 				regulator-boot-on;
 				regulator-state-mem {
 					regulator-off-in-suspend;
@@ -301,7 +300,6 @@
 				regulator-name = "vdd_1v35";
 				regulator-min-microvolt = <1350000>;
 				regulator-max-microvolt = <1350000>;
-				regulator-always-on;
 				regulator-boot-on;
 				regulator-state-mem {
 					regulator-on-in-suspend;
@@ -323,7 +321,6 @@
 				regulator-name = "vdd_2v";
 				regulator-min-microvolt = <2000000>;
 				regulator-max-microvolt = <2000000>;
-				regulator-always-on;
 				regulator-boot-on;
 				regulator-state-mem {
 					regulator-on-in-suspend;
@@ -334,7 +331,6 @@
 				regulator-name = "vdd_1v8";
 				regulator-min-microvolt = <1800000>;
 				regulator-max-microvolt = <1800000>;
-				regulator-always-on;
 				regulator-boot-on;
 				regulator-state-mem {
 					regulator-on-in-suspend;
@@ -419,7 +415,6 @@
 				regulator-name = "vdd_ldo9";
 				regulator-min-microvolt = <1800000>;
 				regulator-max-microvolt = <1800000>;
-				regulator-always-on;
 				regulator-state-mem {
 					regulator-on-in-suspend;
 					regulator-mode = <MAX77802_OPMODE_LP>;
diff --git a/arch/arm/boot/dts/exynos5800-peach-pi.dts b/arch/arm/boot/dts/exynos5800-peach-pi.dts
index 62ceb89e073f..af1c9b9a63f9 100644
--- a/arch/arm/boot/dts/exynos5800-peach-pi.dts
+++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts
@@ -278,7 +278,6 @@
 				regulator-name = "vdd_1v2";
 				regulator-min-microvolt = <1200000>;
 				regulator-max-microvolt = <1200000>;
-				regulator-always-on;
 				regulator-boot-on;
 				regulator-state-mem {
 					regulator-off-in-suspend;
@@ -301,7 +300,6 @@
 				regulator-name = "vdd_1v35";
 				regulator-min-microvolt = <1350000>;
 				regulator-max-microvolt = <1350000>;
-				regulator-always-on;
 				regulator-boot-on;
 				regulator-state-mem {
 					regulator-on-in-suspend;
@@ -323,7 +321,6 @@
 				regulator-name = "vdd_2v";
 				regulator-min-microvolt = <2000000>;
 				regulator-max-microvolt = <2000000>;
-				regulator-always-on;
 				regulator-boot-on;
 				regulator-state-mem {
 					regulator-on-in-suspend;
@@ -334,7 +331,6 @@
 				regulator-name = "vdd_1v8";
 				regulator-min-microvolt = <1800000>;
 				regulator-max-microvolt = <1800000>;
-				regulator-always-on;
 				regulator-boot-on;
 				regulator-state-mem {
 					regulator-on-in-suspend;
@@ -419,7 +415,6 @@
 				regulator-name = "vdd_ldo9";
 				regulator-min-microvolt = <1800000>;
 				regulator-max-microvolt = <1800000>;
-				regulator-always-on;
 				regulator-state-mem {
 					regulator-on-in-suspend;
 					regulator-mode = <MAX77802_OPMODE_LP>;
-- 
2.5.5

[toc] | [next] | [standalone]

#1404289 — Re: [PATCH] ARM: dts: exynos: Remove unneded always-on for regulators on Peach boards

FromKrzysztof Kozlowski <k.kozlowski@samsung.com>
Date2016-05-20 12:00 +0200
SubjectRe: [PATCH] ARM: dts: exynos: Remove unneded always-on for regulators on Peach boards
Message-ID<rAPnP-3SW-1@gated-at.bofh.it>
In reply to#1403980
On 05/19/2016 11:52 PM, Javier Martinez Canillas wrote:
> The regulator always-on property should only be used for regulators
> that either can't be disabled or the drivers for the client devices
> are not enabling the regulator and so being disabled due to unused.
> 
> There are some max77802 regulators in the Peach Pit and Pi boards
> that are always-on but shouldn't since they are enabled by drivers:
> 
> BUCK5 is max77802 inl1 supply
> BUCK7 is max77802 inl{2,10} supply
> BUCK9 is max77802 inl{3,4,5,7} supply
> BUCK10 is mmc1 vqmmc supply
> LDO9 is adc vdd supply
> 
> This patch removes the always-on property for these regulators.
> 
> Some regulators should remain always-on even when they are used as
> input supplies, because these regulators are also used by other
> components that are not described in the DT, for example:
> 
> LDO10 is hdmi vdd_osc supply but it's also used as input supply for
> the Exynos5420 temperature sensor.

I don't have the schematics, so just guessing: by temperature sensor do
you mean the tmu_cpu* node (supplying power to VDD_TS01/23/4)?

If yes then you can also change it, like on Odroid XU boards.

> 
> LDO8 is hdmi vdd and vdd_pll supply but it's also used as supply for
> the Exynos5420 Display port.
> 
> BUCK2 and BUCK6 are the vdd supplies for the EAGLE and KFC clusters.
> 
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
> ---
> 
>  arch/arm/boot/dts/exynos5420-peach-pit.dts | 5 -----
>  arch/arm/boot/dts/exynos5800-peach-pi.dts  | 5 -----
>  2 files changed, 10 deletions(-)

Anyway,

Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>

Best regards,
Krzysztof

[toc] | [prev] | [next] | [standalone]

#1404539 — Re: [PATCH] ARM: dts: exynos: Remove unneded always-on for regulators on Peach boards

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2016-05-20 17:20 +0200
SubjectRe: [PATCH] ARM: dts: exynos: Remove unneded always-on for regulators on Peach boards
Message-ID<rAUnw-77u-3@gated-at.bofh.it>
In reply to#1404289
Hello Krzysztof,

Thanks a lot for the feedback.

On 05/20/2016 05:54 AM, Krzysztof Kozlowski wrote:
> On 05/19/2016 11:52 PM, Javier Martinez Canillas wrote:
>> The regulator always-on property should only be used for regulators
>> that either can't be disabled or the drivers for the client devices
>> are not enabling the regulator and so being disabled due to unused.
>>
>> There are some max77802 regulators in the Peach Pit and Pi boards
>> that are always-on but shouldn't since they are enabled by drivers:
>>
>> BUCK5 is max77802 inl1 supply
>> BUCK7 is max77802 inl{2,10} supply
>> BUCK9 is max77802 inl{3,4,5,7} supply
>> BUCK10 is mmc1 vqmmc supply
>> LDO9 is adc vdd supply
>>
>> This patch removes the always-on property for these regulators.
>>
>> Some regulators should remain always-on even when they are used as
>> input supplies, because these regulators are also used by other
>> components that are not described in the DT, for example:
>>
>> LDO10 is hdmi vdd_osc supply but it's also used as input supply for
>> the Exynos5420 temperature sensor.
> 
> I don't have the schematics, so just guessing: by temperature sensor do
> you mean the tmu_cpu* node (supplying power to VDD_TS01/23/4)?
>

Yes, I meant that the LDO10 regulator supplies 1v8 to VDD18_TS{01,23,4}.
 
> If yes then you can also change it, like on Odroid XU boards.
>

Indeed, I think that should be a separate patch though. I'll do it
next week since I don't have access to the machine today to test.
 
>>
>> LDO8 is hdmi vdd and vdd_pll supply but it's also used as supply for
>> the Exynos5420 Display port.
>>
>> BUCK2 and BUCK6 are the vdd supplies for the EAGLE and KFC clusters.
>>
>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
>> ---
>>
>>  arch/arm/boot/dts/exynos5420-peach-pit.dts | 5 -----
>>  arch/arm/boot/dts/exynos5800-peach-pi.dts  | 5 -----
>>  2 files changed, 10 deletions(-)
> 
> Anyway,
> 
> Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>

Thanks!
 
> Best regards,
> Krzysztof
> --

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America

[toc] | [prev] | [next] | [standalone]

#1408837 — Re: [PATCH] ARM: dts: exynos: Remove unneded always-on for regulators on Peach boards

FromKrzysztof Kozlowski <k.kozlowski@samsung.com>
Date2016-05-30 10:00 +0200
SubjectRe: [PATCH] ARM: dts: exynos: Remove unneded always-on for regulators on Peach boards
Message-ID<rEqhb-1fU-15@gated-at.bofh.it>
In reply to#1403980
On 05/19/2016 11:52 PM, Javier Martinez Canillas wrote:
> The regulator always-on property should only be used for regulators
> that either can't be disabled or the drivers for the client devices
> are not enabling the regulator and so being disabled due to unused.
> 
> There are some max77802 regulators in the Peach Pit and Pi boards
> that are always-on but shouldn't since they are enabled by drivers:
> 
> BUCK5 is max77802 inl1 supply
> BUCK7 is max77802 inl{2,10} supply
> BUCK9 is max77802 inl{3,4,5,7} supply
> BUCK10 is mmc1 vqmmc supply
> LDO9 is adc vdd supply
> 
> This patch removes the always-on property for these regulators.
> 
> Some regulators should remain always-on even when they are used as
> input supplies, because these regulators are also used by other
> components that are not described in the DT, for example:
> 
> LDO10 is hdmi vdd_osc supply but it's also used as input supply for
> the Exynos5420 temperature sensor.
> 
> LDO8 is hdmi vdd and vdd_pll supply but it's also used as supply for
> the Exynos5420 Display port.
> 
> BUCK2 and BUCK6 are the vdd supplies for the EAGLE and KFC clusters.
> 
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
> ---
> 
>  arch/arm/boot/dts/exynos5420-peach-pit.dts | 5 -----
>  arch/arm/boot/dts/exynos5800-peach-pi.dts  | 5 -----
>  2 files changed, 10 deletions(-)

Thanks, applied.

Best regards,
Krzysztof

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web