[U-Boot] [PATCH V2] ARM: mxs: Consolidate configuration options

Marek Vasut marex at denx.de
Wed Jun 26 21:56:43 CEST 2013


Hi Stefano,

> 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 <marex at denx.de>
> > Cc: Fabio Estevam <fabio.estevam at freescale.com>
> > Cc: Lauri Hintsala <lauri.hintsala at bluegiga.com>
> > Cc: Otavio Salvador <otavio at ossystems.com.br>
> > Cc: Stefano Babic <sbabic at 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.

Come on guys, test this! I know you want to have it in current release ;-)

Best regards,
Marek Vasut


More information about the U-Boot mailing list