[U-Boot] [PATCH v3 1/2] APM821xx: Add CPU support

Wolfgang Denk wd at denx.de
Thu Sep 9 16:53:20 CEST 2010


Dear Stefan Roese,

In message <201009091645.16076.sr at denx.de> you wrote:
>
> >  include/apm821xx.h                      |  493 +++++++++++++++++++++++++++++++
> 
> I know that Wolfgang specifically asked you to add this file for this
> "new" SoC. But I'm not so sure if this is the right way. Most of the
> defines in this new file are a plain copy from include/ppc440.h.
> 
> Wolfgang, do you really think this is better than adding support for
> this SoC into ppc440.h? After all, the CPU core of this SoC is still
> a PPC4xx PowerPC.

It makes no sense to duplicate common parts. Only different code
should be moved to separate files.

Viele Grüße,

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
When in doubt, mumble;   when in trouble, delegate;  when in  charge,
ponder.                                             -- James H. Boren


More information about the U-Boot mailing list