[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