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.


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

Reply via email to