[PATCH] OMAP4 do not set TPS62361_REG_ADDR_SET0

[PATCH] OMAP4 do not set TPS62361_REG_ADDR_SET0

[Jan Weitzel]

Don't set SET0 voltage, because it acts as boot voltage for OPP boot.
Take care that the kernel doesn't drive vset gpio to low. This may
happen while reseting the gpio module at initialization, look for
HWMOD_INIT_NO_RESET.

Signed-off-by: Jan Weitzel <j.weitzel@phytec.de>
---
 arch/arm/mach-omap/omap4_generic.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/arch/arm/mach-omap/omap4_generic.c b/arch/arm/mach-omap/omap4_generic.c
index 7200393..bf1f1a7 100644
--- a/arch/arm/mach-omap/omap4_generic.c
+++ b/arch/arm/mach-omap/omap4_generic.c
@@ -586,11 +586,10 @@ noinline int omap4460_scale_vcores(unsigned vsel0_pin, unsigned volt_mv)
 	writel(0x6026, OMAP44XX_PRM_VC_CFG_I2C_CLK);
 
 	/* TPS - supplies vdd_mpu on 4460
-	 * Setup SET1 and SET0 with right values so that kernel
-	 * can use either of them based on its needs.
+	 * Setup SET1 and don't touch SET0 it acts as boot voltage
+	 * source after reset.
 	 */
 
-	omap4_do_scale_tps62361(TPS62361_REG_ADDR_SET0, volt_mv);
 	omap4_do_scale_tps62361(TPS62361_REG_ADDR_SET1, volt_mv);
 
 	/*
-- 
1.7.0.4


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH] OMAP4 do not set TPS62361_REG_ADDR_SET0

[Nishanth Menon]

On Wed, Jun 12, 2013 at 2:17 AM, Jan Weitzel <j.weitzel@phytec.de> wrote:
> Don't set SET0 voltage, because it acts as boot voltage for OPP boot.
> Take care that the kernel doesn't drive vset gpio to low. This may
> happen while reseting the gpio module at initialization, look for
> HWMOD_INIT_NO_RESET.
>
> Signed-off-by: Jan Weitzel <j.weitzel@phytec.de>
> ---
>  arch/arm/mach-omap/omap4_generic.c |    5 ++---
>  1 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/mach-omap/omap4_generic.c b/arch/arm/mach-omap/omap4_generic.c
> index 7200393..bf1f1a7 100644
> --- a/arch/arm/mach-omap/omap4_generic.c
> +++ b/arch/arm/mach-omap/omap4_generic.c
> @@ -586,11 +586,10 @@ noinline int omap4460_scale_vcores(unsigned vsel0_pin, unsigned volt_mv)
>         writel(0x6026, OMAP44XX_PRM_VC_CFG_I2C_CLK);
>
>         /* TPS - supplies vdd_mpu on 4460
> -        * Setup SET1 and SET0 with right values so that kernel
> -        * can use either of them based on its needs.
> +        * Setup SET1 and don't touch SET0 it acts as boot voltage
> +        * source after reset.
>          */
>
> -       omap4_do_scale_tps62361(TPS62361_REG_ADDR_SET0, volt_mv);
>         omap4_do_scale_tps62361(TPS62361_REG_ADDR_SET1, volt_mv);
>
>         /*
> --
> 1.7.0.4
Reviewed-by: Nishanth Menon <nm@ti.com>

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH v2] OMAP4 do not set TPS62361_REG_ADDR_SET0

[Jan Weitzel]

Don't set SET0 voltage, because it acts as boot voltage for OPP boot.
Take care that the kernel doesn't drive vset gpio to low. This may
happen while reseting the gpio module at initialization, look for
HWMOD_INIT_NO_RESET.

Signed-off-by: Jan Weitzel <j.weitzel@phytec.de>
Reviewed-by: Nishanth Menon <nm@ti.com>
---
on top of 88796b1
v2: add Reviewed-by

 arch/arm/mach-omap/omap4_generic.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/arch/arm/mach-omap/omap4_generic.c b/arch/arm/mach-omap/omap4_generic.c
index 7200393..bf1f1a7 100644
--- a/arch/arm/mach-omap/omap4_generic.c
+++ b/arch/arm/mach-omap/omap4_generic.c
@@ -586,11 +586,10 @@ noinline int omap4460_scale_vcores(unsigned vsel0_pin, unsigned volt_mv)
 	writel(0x6026, OMAP44XX_PRM_VC_CFG_I2C_CLK);
 
 	/* TPS - supplies vdd_mpu on 4460
-	 * Setup SET1 and SET0 with right values so that kernel
-	 * can use either of them based on its needs.
+	 * Setup SET1 and don't touch SET0 it acts as boot voltage
+	 * source after reset.
 	 */
 
-	omap4_do_scale_tps62361(TPS62361_REG_ADDR_SET0, volt_mv);
 	omap4_do_scale_tps62361(TPS62361_REG_ADDR_SET1, volt_mv);
 
 	/*
-- 
1.7.0.4


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH v2] OMAP4 do not set TPS62361_REG_ADDR_SET0

[Sascha Hauer]

On Fri, Jun 21, 2013 at 10:24:02AM +0200, Jan Weitzel wrote:
> Don't set SET0 voltage, because it acts as boot voltage for OPP boot.
> Take care that the kernel doesn't drive vset gpio to low. This may
> happen while reseting the gpio module at initialization, look for
> HWMOD_INIT_NO_RESET.
> 
> Signed-off-by: Jan Weitzel <j.weitzel@phytec.de>
> Reviewed-by: Nishanth Menon <nm@ti.com>

Applied, thanks

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox