[U-Boot] [PATCH 6/8] arm, davinci, am1808: add lowlevel functions for booting from NOR

[Heiko Schocher]

Hello Christian,

Christian Riesch wrote:
> Hello Heiko,
> Thank you for your reply. I have just noticed that the patches have
> already been pulled into arm/next so I guess I am quite late with my
> comments. I'd like to apologize for this; apparently I need another

No problem. Thanks for your reply!

> email filter that also filters for messages containing "am1808" and
> not only "da850" ;-)

;-)

> Which brings me to my next point: AFAIK Texas Instruments's AM18xx
> SoCs, their OMAP-L138, and their DA850 based devices basically differ
> only in the DSP/PRU part of the device. If you are (like in u-boot)
> only interested in the ARM part of chip, they are basically the same
> device. For some reason the Linux kernel and u-boot support for all
> these devices is labelled da850... Although I have no hardware to test
> it, I guess that your DDR memory controller initialization code could
> be used for the OMAP-L138 and the corresponding Davinci device as
> well. So since all code in u-boot for the entire device family is
> labelled da850... I guess your DDR initialization functions should get
> this prefix as well for consistency. Any comments from the
> maintainers/TI employees who know these devices better than I do?

Yes of course, such a rename should be done, if somebody has tested it!

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany