All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] OMAP4: Fix bug in omap4460_volts struct
@ 2013-04-09  8:05 Lubomir Popov
  2013-04-10 20:08 ` Tom Rini
  0 siblings, 1 reply; 2+ messages in thread
From: Lubomir Popov @ 2013-04-09  8:05 UTC (permalink / raw)
  To: u-boot

The omap4460_volts struct was incorrectly referencing tps62361
instead of twl6030 as PMIC for the core and mm voltages (the
tps is used for mpu supply only). This shall lead to bad OPP
settings while booting kernel. Fixing it.

Fix some comments as well.

Signed-off-by: Lubomir Popov <lpopov@mm-sol.com>
---
This patch is separated from the OMAP4470/TWL6032 support series
because it fixes an existing bug.

 arch/arm/cpu/armv7/omap4/hw_data.c |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm/cpu/armv7/omap4/hw_data.c b/arch/arm/cpu/armv7/omap4/hw_data.c
index 7551b98..04977b4 100644
--- a/arch/arm/cpu/armv7/omap4/hw_data.c
+++ b/arch/arm/cpu/armv7/omap4/hw_data.c
@@ -216,14 +216,14 @@ struct dplls omap4460_dplls = {
 
 struct pmic_data twl6030_4430es1 = {
 	.base_offset = PHOENIX_SMPS_BASE_VOLT_STD_MODE_UV,
-	.step = 12660, /* 10 mV represented in uV */
+	.step = 12660, /* 12.66 mV represented in uV */
 	/* The code starts at 1 not 0 */
 	.start_code = 1,
 };
 
 struct pmic_data twl6030 = {
 	.base_offset = PHOENIX_SMPS_BASE_VOLT_STD_MODE_WITH_OFFSET_UV,
-	.step = 12660, /* 10 mV represented in uV */
+	.step = 12660, /* 12.66 mV represented in uV */
 	/* The code starts at 1 not 0 */
 	.start_code = 1,
 };
@@ -271,11 +271,11 @@ struct vcores_data omap4460_volts = {
 
 	.core.value = 1200,
 	.core.addr = SMPS_REG_ADDR_VCORE1,
-	.core.pmic = &tps62361,
+	.core.pmic = &twl6030,
 
 	.mm.value = 1200,
 	.mm.addr = SMPS_REG_ADDR_VCORE2,
-	.mm.pmic = &tps62361,
+	.mm.pmic = &twl6030,
 };
 
 /*
-- 
1.7.9.6 (Apple Git-31.1)

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

* [U-Boot] [PATCH] OMAP4: Fix bug in omap4460_volts struct
  2013-04-09  8:05 [U-Boot] [PATCH] OMAP4: Fix bug in omap4460_volts struct Lubomir Popov
@ 2013-04-10 20:08 ` Tom Rini
  0 siblings, 0 replies; 2+ messages in thread
From: Tom Rini @ 2013-04-10 20:08 UTC (permalink / raw)
  To: u-boot

On Tue, Apr 09, 2013 at 11:05:33AM +0300, Lubomir Popov wrote:

> The omap4460_volts struct was incorrectly referencing tps62361
> instead of twl6030 as PMIC for the core and mm voltages (the
> tps is used for mpu supply only). This shall lead to bad OPP
> settings while booting kernel. Fixing it.
> 
> Fix some comments as well.
> 
> Signed-off-by: Lubomir Popov <lpopov@mm-sol.com>

Applied to u-boot-ti/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130410/1ac74205/attachment.pgp>

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

end of thread, other threads:[~2013-04-10 20:08 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-04-09  8:05 [U-Boot] [PATCH] OMAP4: Fix bug in omap4460_volts struct Lubomir Popov
2013-04-10 20:08 ` Tom Rini

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.