[U-Boot] mpc83xx: SICRL #defines
Kim Phillips
kim.phillips at freescale.com
Sat May 29 03:46:00 CEST 2010
On Fri, 28 May 2010 16:44:21 -0700
Ron Madrid <ron_madrid at sbcglobal.net> wrote:
> Kim,
>
> This is defined this way:
>
> #define SICRL_USBDR 0x00000C00
>
> However in order to actually set these pins to the USB_DR
> functionality the setting should be 0x00000800. So it appears
> that these (#define SICRL_*) are all masks. Each field seems
> to be two bits and therefore 4 possible settings.
>
> So, should they all be used as masks, should this be changed,
> or should we create new '#define's for the different
> functionalities? I need to make a change to my config file
> and I thought I'd get your opinion before I submitted a patch;
> I can make any suggested changes with my change at the same time.
It looks like the pin functions wouldn't be able to be efficiently
named in a uniform fashion - I'd be ok with just introducing the four
new defines SICRL_USBDR_00, SICRL_USBDR_01, and so on.
Kim
More information about the U-Boot
mailing list