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

Stefano Babic
2013-06-26T14:40:39Z

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