Hi Marek, On 16/06/2013 15:39, Marek Vasut wrote: > Pull all the duplicate configuration options into configs/mxs.h > from the board configuration files. This reduces the files greatly > and makes them somewhat more readable. Besides, we do no longer > have such a horrible duplication of code. > > Note that the mx23evk grew in size slightly. This is due to the > CONFIG_SYS_CBSIZE now being set to 1024 as it is on the rest of > MXS systems. > > This patch also fixes the OCRAM size for i.MX23. The i.MX23 has > only 32kB of OCRAM, while i.MX28 has 128kB of OCRAM. > > I verified the configuration didn't change for each of the boards, > but I didn't boot-test it on the boards I do not have. I configured > U-Boot for each board using the "make ... <board>_config" command > and then ran "cpp -I include -dM include/config.h" , which dumped > all the configuration options. I did this both before and after this > patch and finally compared the results for each MXS board. Actually, > the results do differ slightly, since the configs/mxs.h file now > properly includes the correct iomux-mx23.h or iomux-mx28.h , so > while comparing, I had to ignore these new defines. These have no > impact on U-Boot configuration though. > > Signed-off-by: Marek Vasut <ma...@denx.de> > Cc: Fabio Estevam <fabio.este...@freescale.com> > Cc: Lauri Hintsala <lauri.hints...@bluegiga.com> > Cc: Otavio Salvador <ota...@ossystems.com.br> > Cc: Stefano Babic <sba...@denx.de> > ---
This is a nice cleanup, but as we probably cannot test all boards before release, I apply it to u-boot-imx, next branch. Regards, Stefano -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de ===================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot