[U-Boot] [PATCH RFC v2] OMAP: mmc: add support for second and third mmc channels

[Steve Sakoman]

On Thu, Apr 29, 2010 at 1:38 PM, Andy Fleming <afleming at gmail.com> wrote:
> On Thu, Apr 29, 2010 at 3:26 PM, Steve Sakoman <sakoman at gmail.com> wrote:
>> This patch adds support for the second and third mmc channels on OMAP3
>> processors
>>
>> Boards wishing to use this feature should define CONFIG_SYS_MMC_SET_DEV
>>
>> Tested on Overo
>>
>> Signed-off-by: Steve Sakoman <steve at sakoman.com>
>> ---
>>
>> diff --git a/arch/arm/include/asm/arch-omap3/mmc_host_def.h
>> b/arch/arm/include/asm/arch-omap3/mmc_host_def.h
>> index aa751c9..f081b43 100644
>> --- a/arch/arm/include/asm/arch-omap3/mmc_host_def.h
>> +++ b/arch/arm/include/asm/arch-omap3/mmc_host_def.h
>> @@ -31,11 +31,18 @@
>>  typedef struct t2 {
>>        unsigned char res1[0x274];
>>        unsigned int devconf0;          /* 0x274 */
>> -       unsigned char res2[0x2A8];
>> +       unsigned char res2[0x060];
>> +       unsigned int devconf1;          /* 0x2D8 */
>> +       unsigned char res3[0x244];
>>        unsigned int pbias_lite;        /* 0x520 */
>>  } t2_t;
>
>
> Errr...just to make sure, you *meant* to move devconf1, too, right?

Yes!  It is used one time to set a "magic" bit that makes the mmc2
channel work.  That is why my testing didn't pick up this error --
mmc2 is not available on my hardware, only mmc1 and mmc3.  The error
with pbias_lite seemed to have no negative effect in my testing!

At any rate, both registers now seem to be correct :-)

Steve