[U-Boot] da8xx

Nick Thompson nick.thompson at gefanuc.com
Thu Nov 5 10:32:36 CET 2009


Tom wrote:
> The v4 is looking good.
> No regressions on MAKEALL arm
> 
> Are there any outstanding issues besides needing lowercase variable names and 
> the pin multiplex dependency ?
> 
> Tom

Sekhar Nori over at TI was kind enough to review this for me prior to submission.
The only outstanding question was whether clk_get() should be used by the rest of
DaVinci as well as DA8xx.

The answer is probably yes, but this implementation relies on the C structure
register overlays, which DaVinci devices do not yet use. Changing DaVinci is
outside the remit of adding DA8xx support I believe.

One question relating to lower case variable names. You said:

> > +#define PINMUX	&DAVINCI_SYSCFG_REGS->pinmux
> > +
>
> Lowercase variables
> Apply globally.

These are not true variables, but #defines. Is it considered okay to use lowercase
in this kind of situation? I would be happier with lowercase, I admit.

I would have preferred to use static "const" variables, but of coursse C doesn't
allow those to be used in static assignments as they are not really constant. Bah.

Thanks,
Nick.


More information about the U-Boot mailing list