On Thu, Apr 29, 2010 at 1:38 PM, Andy Fleming <aflem...@gmail.com> wrote:
> On Thu, Apr 29, 2010 at 3:26 PM, Steve Sakoman <sako...@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 <st...@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
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
