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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
