>-----Original Message-----
>From: Tony Lindgren [mailto:[email protected]]
>Sent: Monday, June 22, 2009 7:52 AM
>To: Pandita, Vikram
>Cc: [email protected]
>Subject: Re: [PATCH v2 1/2] OMAP3:Zoom2: Add TWL4030 support
>
>* Vikram Pandita <[email protected]> [090619 01:18]:
>> Add TWL4030 CORE and TWL4030 drivers to Zoom2 board file
>> TWL drivers enabled are:
>>      bci
>>      madc
>>      usb
>>      keypad
>>      mmc
>>
>> Signed-off-by: Vikram Pandita <[email protected]>
>> ---
>>  arch/arm/mach-omap2/board-zoom2.c |  209 
>> +++++++++++++++++++++++++++++++++++--
>>  1 files changed, 198 insertions(+), 11 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/board-zoom2.c 
>> b/arch/arm/mach-omap2/board-zoom2.c
>> index bcc0f76..443859c 100644
>> --- a/arch/arm/mach-omap2/board-zoom2.c
>> +++ b/arch/arm/mach-omap2/board-zoom2.c
>>
>>  #include "mmc-twl4030.h"
>>
>> +#ifndef CONFIG_TWL4030_CORE
>> +#error "no power companion board defined!"
>> +#endif
>
>This does not look right. You should still be able to boot the board even 
>without
>twl compiled in. Or compile twl as a module. Currently we do have a bug where
>mmc-twl4030.c forces mmc core to be compiled in because of the regulator fwk, 
>but
>that should be fixed by moving the regulator fwk calls to 
>drivers/mmc/host/omap_hsmmc.c.

Correcting this is v3 of patch.
Posting it out today.

>
>Regards,
>
>Tony

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to