[U-Boot] [PATCH] ARMV7: OMAP3: BeagleBoard: add xM rev B to ID table

Steve Sakoman sakoman at gmail.com
Sat Nov 6 04:05:27 CET 2010


On Fri, Nov 5, 2010 at 10:54 AM, Nishanth Menon
<menon.nishanth at gmail.com> wrote:
> Jason Kridner wrote, on 11/05/2010 01:46 AM:
>> From: Koen Kooi<koen at dominion.thruhere.net>
>>
>> Patch was updated by Jason Kridner<jkridner at beagleboard.org>:
>> * Use tabs to match style of other board revisions
>> * Only include board revisions that exist
>> * Default to the same configuration as the latest revision, but
>>    without setting 'beaglerev'
>>
>> Signed-off-by: Jason Kridner<jkridner at beagleboard.org>
> not signed-off-by: Koen?
>
>> ---
>>   board/ti/beagle/beagle.c |   20 +++++++++++++++++++-
>>   board/ti/beagle/beagle.h |    3 ++-
>>   2 files changed, 21 insertions(+), 2 deletions(-)
>>
>> diff --git a/board/ti/beagle/beagle.c b/board/ti/beagle/beagle.c
>> index 520e57d..93c452e 100644
>> --- a/board/ti/beagle/beagle.c
>> +++ b/board/ti/beagle/beagle.c
>> @@ -176,7 +176,7 @@ int misc_init_r(void)
>>                                       TWL4030_PM_RECEIVER_VAUX2_DEV_GRP,
>>                                       TWL4030_PM_RECEIVER_DEV_GRP_P1);
>>               break;
>> -     case REVISION_XM:
>> +     case REVISION_XM_A:
>>               printf("Beagle xM Rev A\n");
>>               setenv("beaglerev", "xMA");
>>               setenv("mpurate", "1000");
>> @@ -187,8 +187,26 @@ int misc_init_r(void)
>>                                       TWL4030_PM_RECEIVER_VAUX2_DEV_GRP,
>>                                       TWL4030_PM_RECEIVER_DEV_GRP_P1);
>>               break;
>> +     case REVISION_XM_B:
>> +             printf("Beagle xM Rev B\n");
>> +             setenv("beaglerev", "xMB");
>> +             setenv("mpurate", "1000");
>> +             MUX_BEAGLE_XM();
>> +             /* Set VAUX2 to 1.8V for EHCI PHY */
>> +             twl4030_pmrecv_vsel_cfg(TWL4030_PM_RECEIVER_VAUX2_DEDICATED,
>> +                                     TWL4030_PM_RECEIVER_VAUX2_VSEL_18,
>> +                                     TWL4030_PM_RECEIVER_VAUX2_DEV_GRP,
>> +                                     TWL4030_PM_RECEIVER_DEV_GRP_P1);
>> +             break;
>>       default:
>>               printf("Beagle unknown 0x%02x\n", get_board_revision());
>> +             setenv("mpurate", "1000");
>
> It looks to me looking at the file that mpurate usage is CPU based and
> NOT board based.. maybe you should use the cpu idendity to decide on
> mpurate instead?

I noticed this too.  I just submitted a patch for Overo that sets the
mpurate to the cpu maximum (based on cpu type and version) if the
mpurate environment variable is set to "auto"

This solves an additional issue: with things as they are now, it is
not possible for a user to set the mpurate to a specific value -- it
will always be overwritten.  The scheme above allows the user to set a
specific value or to allow u-boot to set the maximum automatically.

Note that for 36xx my patch sets the max to 720 -- this is because
mainline/linux-omap currently does not support 1000.  We can adjust
that when kernel support for 1000 appears.

My plan was to submit a similar patch for Beagle.

Steve


>> +             MUX_BEAGLE_XM();
>> +             /* Set VAUX2 to 1.8V for EHCI PHY */
>> +             twl4030_pmrecv_vsel_cfg(TWL4030_PM_RECEIVER_VAUX2_DEDICATED,
>> +                                     TWL4030_PM_RECEIVER_VAUX2_VSEL_18,
>> +                                     TWL4030_PM_RECEIVER_VAUX2_DEV_GRP,
>> +                                     TWL4030_PM_RECEIVER_DEV_GRP_P1);
>>       }
>>
>>       switch (get_expansion_id()) {
>> diff --git a/board/ti/beagle/beagle.h b/board/ti/beagle/beagle.h
>> index 7d8dee0..fa893c4 100644
>> --- a/board/ti/beagle/beagle.h
>> +++ b/board/ti/beagle/beagle.h
>> @@ -37,7 +37,8 @@ const omap3_sysinfo sysinfo = {
>>   #define REVISION_AXBX       0x7
>>   #define REVISION_CX 0x6
>>   #define REVISION_C4 0x5
>> -#define REVISION_XM  0x0
>> +#define REVISION_XM_A        0x0
>> +#define REVISION_XM_B        0x1
>>
>>   /*
>>    * IEN  - Input Enable
>
>
> --
> Regards,
> Nishanth Menon
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>


More information about the U-Boot mailing list