Skip to contentSkip to conversation

ARM: dts: dra7-evm: Regulator max voltage fixes

PATCH In review Ravikumar Kattekola 3 participants 5 messages Dec 3, 2014 – Dec 10, 2014 Device tree bindings

lore
Share & tools
lore
#0 lore raw

cover ARM: dts: dra7-evm: Regulator max voltage fixes

Hi,

This set does a minor fix and an update:

- Fix a typo in the max voltage property of smps6 (vdd_gpu) regulator node added by

     the patch : c56a831ca47e ("ARM: dts: DRA7: Add TPS659038 PMIC nodes")

- Update max voltage property of smps7 (vdd_core) regulator node to align with the latest Data Manual

Regards,
RK

Ravikumar Kattekola (2):
ARM: dts: dra7-evm: Fix typo in SMPS6 (VDD_GPU) max voltage
ARM: dts: dra7-evm: Update SMPS7 (VDD_CORE) max voltage to match DM

 arch/arm/boot/dts/dra7-evm.dts  |    4 ++--
 arch/arm/boot/dts/dra72-evm.dts |    2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)
Signature · 1 line
1.7.9.5
#1 lore raw

PATCH 1/2 ARM: dts: dra7-evm: Fix typo in SMPS6 (VDD_GPU) max voltage

The max expected voltage for VDD_GPU, connected to SMPS6, is 1.25V. Correct regulator max voltage constraint to meet this requirement.

Document reference: DRA74 Data Manual, SPRS857M - Dec 2012,

		    Revised Oct 2014.
Fixes c56a831ca47e ("ARM: dts: DRA7: Add TPS659038 PMIC nodes")
---
 arch/arm/boot/dts/dra7-evm.dts |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm/boot/dts/dra7-evm.dts +1 -1
index 736092b..b6b9286 100644
@@ -304,7 +304,7 @@
 					/* VDD_GPU - over VDD_SMPS6 */
 					regulator-name = "smps6";
 					regulator-min-microvolt = <850000>;
-					regulator-max-microvolt = <12500000>;
+					regulator-max-microvolt = <1250000>;
 					regulator-always-on;
 					regulator-boot-on;
 				};
Signature · 1 line
1.7.9.5
#2 lore raw

PATCH 2/2 ARM: dts: dra7-evm: Update SMPS7 (VDD_CORE) max voltage to match DM

In reply to Ravikumar Kattekola

As per the latest Data Manual, for newer samples, the nominal voltage required for VDD_CORE at OPP_NOM can be upto 1.06V which was 1.03V earlier.

Update the regulator max voltage constraint for SMPS7, connected to VDD_CORE, to meet this requirement.

Document reference:
DRA74 Data Manual, SPRS857M - Dec 2012, Revised Oct 2014. DRA72 Data Manual, SPRS906G - Dec 2012, revised Oct 2014.

---
 arch/arm/boot/dts/dra7-evm.dts  |    2 +-
 arch/arm/boot/dts/dra72-evm.dts |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
arch/arm/boot/dts/dra7-evm.dts +1 -1
index b6b9286..10b725c 100644
@@ -313,7 +313,7 @@
 					/* CORE_VDD */
 					regulator-name = "smps7";
 					regulator-min-microvolt = <850000>;
-					regulator-max-microvolt = <1030000>;
+					regulator-max-microvolt = <1060000>;
 					regulator-always-on;
 					regulator-boot-on;
 				};
arch/arm/boot/dts/dra72-evm.dts +1 -1
index afc74fd..89085d0 100644
@@ -160,7 +160,7 @@
 					/* VDD_CORE */
 					regulator-name = "smps2";
 					regulator-min-microvolt = <850000>;
-					regulator-max-microvolt = <1030000>;
+					regulator-max-microvolt = <1060000>;
 					regulator-boot-on;
 					regulator-always-on;
 				};
Signature · 1 line
1.7.9.5
Acked-by on the v1 series
#3 lore raw
In reply to Ravikumar Kattekola
Quoted 13 lines · Ravikumar Kattekola

Hi,

This set does a minor fix and an update:

- Fix a typo in the max voltage property of smps6 (vdd_gpu) regulator node added by

     the patch : c56a831ca47e ("ARM: dts: DRA7: Add TPS659038 PMIC nodes")

- Update max voltage property of smps7 (vdd_core) regulator node to align with the latest Data Manual

Regards,
RK

Quoted 8 lines

Ravikumar Kattekola (2):
ARM: dts: dra7-evm: Fix typo in SMPS6 (VDD_GPU) max voltage
ARM: dts: dra7-evm: Update SMPS7 (VDD_CORE) max voltage to match DM

 arch/arm/boot/dts/dra7-evm.dts  |    4 ++--
 arch/arm/boot/dts/dra72-evm.dts |    2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

These apply to

https://git.kernel.org/cgit/linux/kernel/git/tmlind/linux-omap.git/log/?h=omap-for-v3.19/dt-v2

Signature · 2 lines
Regards,
Nishanth Menon
#4 lore raw
Quoted 22 lines · Nishanth Menon
Quoted 13 lines · Ravikumar Kattekola

Hi,

This set does a minor fix and an update:

- Fix a typo in the max voltage property of smps6 (vdd_gpu) regulator node added by

     the patch : c56a831ca47e ("ARM: dts: DRA7: Add TPS659038 PMIC nodes")

- Update max voltage property of smps7 (vdd_core) regulator node to align with the latest Data Manual

Regards,
RK

Quoted 8 lines

Ravikumar Kattekola (2):
ARM: dts: dra7-evm: Fix typo in SMPS6 (VDD_GPU) max voltage
ARM: dts: dra7-evm: Update SMPS7 (VDD_CORE) max voltage to match DM

 arch/arm/boot/dts/dra7-evm.dts  |    4 ++--
 arch/arm/boot/dts/dra72-evm.dts |    2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)
Quoted 5 lines

These apply to

https://git.kernel.org/cgit/linux/kernel/git/tmlind/linux-omap.git/log/?h=omap-for-v3.19/dt-v2

Applying both into omap-for-v3.19/fixes thanks.

Tony