[U-Boot] [PATCH] TI: Davinci: NAND Driver Cleanup

Chemparathy, Cyril cyril at ti.com
Tue Mar 16 19:05:33 CET 2010


Nick,

> Since this is really just a constant, why setup a variable locally where
> ever EMIF registers are accessed? What's wrong with the define you removed
> below? ...
> > -#define davinci_emif_regs \
> > -	((struct davinci_emif_regs *)DAVINCI_ASYNC_EMIF_CNTRL_BASE)
> ...This one.

Thanks.  I'll incorporate this change and send out an update.

Also, it turns out that getting rid of this macro definition would also break DA830 EVM.
 
I have put in fixes for these issues, run a MAKEALL and send out an updated patch shortly.

Regards
Cyril.




More information about the U-Boot mailing list