[U-Boot] [PATCH 1/2] mpc52xx, mucmc52, uc101: config cleanup

Wolfgang Denk wd at denx.de
Wed Sep 9 14:05:29 CEST 2009


Dear Detlev Zundel,

In message <m27hw8cozk.fsf at ohwell.denx.de> you wrote:
> 
> > +#define CONFIG_MPC5xxx		1	/* This is an MPC5xxx CPU		*/
> > +#define CONFIG_MPC5200		1	/* (more precisely an MPC5200 CPU)	*/
> > +
> > +#define CONFIG_SYS_MPC5XXX_CLKIN	33000000 /* ... running at 33.000000MHz	*/
> 
> Are you sure that this is a good choice to put in a manufacturer common
> header file?  Isn't the probability of this becoming incorrect nearly
> 100% for the next board from this manufacturer?
> 
> I'd advise to keep CPU stuff in the individual board config files -
> maybe, if you really want, create a "manroland-mpc52xx-common.h". The
> rest of the common configs should be re-evaluated in the same spirit.

And right you are.  Thanks for pointing out.

Heiko, please rework. I haven't pushed the repo out yet, so I just
remove that commit again.

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
Every solution breeds new problems.


More information about the U-Boot mailing list