[U-Boot] [PATCH] Fix define for MPC5121 ADS board.
Wolfgang Denk
wd at denx.de
Tue Oct 5 20:21:07 CEST 2010
Dear mark.vels at team-embedded.nl,
In message <1286293579-26587-1-git-send-email-mark.vels at team-embedded.nl> you wrote:
> Originally the Makefile defines CONFIG_ADS5121_REV2 when configuring
> for board mpc5121ads_rev2. The config in include/configs/mpc5121ads.h
> expects a define CONFIG_MPC5121ADS_REV2 for this.
> Therefore there are two defines used but only 1 is defined.
> This patch renames references to CONFIG_ADS5121_REV2 to CONFIG_MPC5121ADS_REV2
> to have one common define.
>
> This patch fixes faulty mem size detection caused by wrong config in
> include/configs/mpc5121ads.h
Thanks. But... should we not rather drop that code completely? I
guess the number of remaining Rev. 2 boards should be epsilon. Do you
think these veterans are still in use anywhere?
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Brain fried - Core dumped
More information about the U-Boot
mailing list