On Tue, Mar 15, 2011 at 8:14 PM, Andy Green <a...@warmcat.com> wrote:
> On 03/15/2011 07:10 PM, Somebody in the thread at some point said:
>>
>> the pandaboard does not use the VUSIM or VAUX1 power regulators on the
>> TWL6030
>> and are left floating. if the VUSIM and VAUX1 power regulators are
>> initilized,
>> noise on the unloaded regulators generates an overcurrent interrupt
>> causing the
>> system to power down. this patch removes the initialization of the unused
>> power
>> regulators of VUSIM and VAUX1.
>>
>> Signed-off-by: David Anders<x0132...@ti.com>
>
> Acked-by: Andy Green <andy.gr...@linaro.org>
>
> Great job finding this one David.
Great!

Acked-by: Jean Pihet <j-pi...@ti.com>

Thanks,
Jean

>
>> ---
>>  arch/arm/mach-omap2/board-omap4panda.c |   28
>> ----------------------------
>>  1 files changed, 0 insertions(+), 28 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/board-omap4panda.c
>> b/arch/arm/mach-omap2/board-omap4panda.c
>> index e944025..abc913b 100644
>> --- a/arch/arm/mach-omap2/board-omap4panda.c
>> +++ b/arch/arm/mach-omap2/board-omap4panda.c
>> @@ -220,19 +220,6 @@ static int __init omap4_twl6030_hsmmc_init(struct
>> omap2_hsmmc_info *controllers)
>>        return 0;
>>  }
>>
>> -static struct regulator_init_data omap4_panda_vaux1 = {
>> -       .constraints = {
>> -               .min_uV                 = 1000000,
>> -               .max_uV                 = 3000000,
>> -               .apply_uV               = true,
>> -               .valid_modes_mask       = REGULATOR_MODE_NORMAL
>> -                                       | REGULATOR_MODE_STANDBY,
>> -               .valid_ops_mask  = REGULATOR_CHANGE_VOLTAGE
>> -                                       | REGULATOR_CHANGE_MODE
>> -                                       | REGULATOR_CHANGE_STATUS,
>> -       },
>> -};
>> -
>>  static struct regulator_init_data omap4_panda_vaux2 = {
>>        .constraints = {
>>                .min_uV                 = 1200000,
>> @@ -288,19 +275,6 @@ static struct regulator_init_data omap4_panda_vpp = {
>>        },
>>  };
>>
>> -static struct regulator_init_data omap4_panda_vusim = {
>> -       .constraints = {
>> -               .min_uV                 = 1200000,
>> -               .max_uV                 = 2900000,
>> -               .apply_uV               = true,
>> -               .valid_modes_mask       = REGULATOR_MODE_NORMAL
>> -                                       | REGULATOR_MODE_STANDBY,
>> -               .valid_ops_mask  = REGULATOR_CHANGE_VOLTAGE
>> -                                       | REGULATOR_CHANGE_MODE
>> -                                       | REGULATOR_CHANGE_STATUS,
>> -       },
>> -};
>> -
>>  static struct regulator_init_data omap4_panda_vana = {
>>        .constraints = {
>>                .min_uV                 = 2100000,
>> @@ -356,12 +330,10 @@ static struct twl4030_platform_data
>> omap4_panda_twldata = {
>>        /* Regulators */
>>        .vmmc           =&omap4_panda_vmmc,
>>        .vpp            =&omap4_panda_vpp,
>> -       .vusim          =&omap4_panda_vusim,
>>        .vana           =&omap4_panda_vana,
>>        .vcxio          =&omap4_panda_vcxio,
>>        .vdac           =&omap4_panda_vdac,
>>        .vusb           =&omap4_panda_vusb,
>> -       .vaux1          =&omap4_panda_vaux1,
>>        .vaux2          =&omap4_panda_vaux2,
>>        .vaux3          =&omap4_panda_vaux3,
>>        .usb            =&omap4_usbphy_data,
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to