[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