On Wed, Nov 18, 2009 at 12:09:46AM +0100, Wolfgang Denk wrote:
> In message <4af9823c.8090...@gefanuc.com> you wrote:
> > Provides initial support for TI OMAP-L1x/DA8xx SoC devices.
> > See http://www.ti.com
> > 
> > The DA8xx devices are similar to DaVinci devices but have a differing
> > memory map and updated peripheral versions.
> > 
> > Signed-off-by: Nick Thompson <nick.thomp...@gefanuc.com>
> > Signed-off-by: Sekhar Nori <nsek...@ti.com>
> ...
> > +enum davinci_lpsc_ids {
> > +   DAVINCI_LPSC_TPCC = 0,
> > +   DAVINCI_LPSC_TPTC0,
> > +   DAVINCI_LPSC_TPTC1,
> > +   DAVINCI_LPSC_AEMIF,
> > +   DAVINCI_LPSC_SPI0,
> > +   DAVINCI_LPSC_MMC_SD,
[snip]
> > +   DAVINCI_LPSC_eQEP,
> > +   DAVINCI_LPSC_SCR_P0,
> > +   DAVINCI_LPSC_SCR_P1,
> > +   DAVINCI_LPSC_CR_P3,
> > +   DAVINCI_LPSC_L3_CBA_RAM
> > +};
> 
> Variable names must be lower case. Please fix.

I suspect the compiler would take issue if you tried to vary those
constants...

-Scott
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to