Hello Heiko,

> - as this boards are similiar, collect common config option
>   in manroland-common.h and manroland-mpc52xx-common.h
>   for mpc52xx specific common options for this manufacturer.
> - add OF support
> - update default environment

Argh, sorry for interrupting again, but I think I suggested something
problematic on second thought:

>  include/configs/manroland-mpc52xx-common.h |  230 ++++++++++++++++++++++++

[...]

> diff --git a/include/configs/manroland-mpc52xx-common.h 
> b/include/configs/manroland-mpc52xx-common.h
> new file mode 100644
> index 0000000..b29ef9b
> --- /dev/null
> +++ b/include/configs/manroland-mpc52xx-common.h

[...]

> +#define CONFIG_MPC5xxx               1       /* This is an MPC5xxx CPU       
>         */
> +#define CONFIG_MPC5200               1       /* (more precisely an MPC5200 
> CPU)      */

So all boards using manroland-mpc52xx-common.h are in fact 5200s.  In
this case, I should have suggested the filename
"manroland-mpc5200-common.h" as the "xx" part surely cannot be anything
else then "00".

Sorry for this wrong lead.

Cheers
  Detlev

-- 
Some people seem to think that C is a real programming language, but they are
sadly mistaken.  It really is about writing almost-portable assembly language
[...]               -- Linus Torvalds 10404265599082718160nore...@blogger.com
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: d...@denx.de
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to