From mboxrd@z Thu Jan 1 00:00:00 1970 From: Detlev Zundel Date: Thu, 10 Sep 2009 14:30:15 +0200 Subject: [U-Boot] [PATCHv2 1/2] mpc52xx, mucmc52, uc101: config cleanup In-Reply-To: <4AA89502.1070809@invitel.hu> (Heiko Schocher's message of "Thu, 10 Sep 2009 07:56:18 +0200") References: <4AA755CD.8050807@denx.de> <20090909120529.0F9E3832E8DE@gemini.denx.de> <4AA89502.1070809@invitel.hu> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 10404265599082718160noreply at 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: dzu at denx.de