All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3] dts: omap5: board-common: fix wrong SMPS6 (VDD-DDR3) voltage
@ 2016-11-14 11:55 H. Nikolaus Schaller
  2016-11-14 21:17 ` Tony Lindgren
  0 siblings, 1 reply; 2+ messages in thread
From: H. Nikolaus Schaller @ 2016-11-14 11:55 UTC (permalink / raw)
  To: Benoît Cousson, Tony Lindgren, Rob Herring, Mark Rutland,
	Russell King
  Cc: linux-omap, devicetree, linux-kernel, kernel, H. Nikolaus Schaller

DDR3L is usually specified as

	JEDEC standard 1.35V(1.28V~1.45V) & 1.5V(1.425V~1.575V)

Therefore setting smps6 regulator to 1.2V is definitively below
minimum. It appears that real world chips are more forgiving than
data sheets indicate, but let's set the regulator right.

Note: a board that uses other voltages (DDR with 1.5V) can
overwrite by referencing &smps6_reg.

Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com>
---
 arch/arm/boot/dts/omap5-board-common.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/omap5-board-common.dtsi b/arch/arm/boot/dts/omap5-board-common.dtsi
index ab60a8e..e4a14d5 100644
--- a/arch/arm/boot/dts/omap5-board-common.dtsi
+++ b/arch/arm/boot/dts/omap5-board-common.dtsi
@@ -476,8 +476,8 @@
 				smps6_reg: smps6 {
 					/* VDD_DDR3 - over VDD_SMPS6 */
 					regulator-name = "smps6";
-					regulator-min-microvolt = <1200000>;
-					regulator-max-microvolt = <1200000>;
+					regulator-min-microvolt = <1350000>;
+					regulator-max-microvolt = <1350000>;
 					regulator-always-on;
 					regulator-boot-on;
 				};
-- 
2.7.3

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

* Re: [PATCH v3] dts: omap5: board-common: fix wrong SMPS6 (VDD-DDR3) voltage
  2016-11-14 11:55 [PATCH v3] dts: omap5: board-common: fix wrong SMPS6 (VDD-DDR3) voltage H. Nikolaus Schaller
@ 2016-11-14 21:17 ` Tony Lindgren
  0 siblings, 0 replies; 2+ messages in thread
From: Tony Lindgren @ 2016-11-14 21:17 UTC (permalink / raw)
  To: H. Nikolaus Schaller
  Cc: Benoît Cousson, Rob Herring, Mark Rutland, Russell King,
	linux-omap, devicetree, linux-kernel, kernel

* H. Nikolaus Schaller <hns@goldelico.com> [161114 03:55]:
> DDR3L is usually specified as
> 
> 	JEDEC standard 1.35V(1.28V~1.45V) & 1.5V(1.425V~1.575V)
> 
> Therefore setting smps6 regulator to 1.2V is definitively below
> minimum. It appears that real world chips are more forgiving than
> data sheets indicate, but let's set the regulator right.
> 
> Note: a board that uses other voltages (DDR with 1.5V) can
> overwrite by referencing &smps6_reg.

Applying into omap-for-v4.9/fixes thanks.

Tony

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

end of thread, other threads:[~2016-11-14 21:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-14 11:55 [PATCH v3] dts: omap5: board-common: fix wrong SMPS6 (VDD-DDR3) voltage H. Nikolaus Schaller
2016-11-14 21:17 ` 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.