Am 30.10.2011 11:32, schrieb Albert ARIBAUD: > Hi Jens, Marek, > > Le 30/10/2011 00:47, Marek Vasut a écrit : >>> * re-introduce the MACH_TYPE_XXXXXX for EB_CPUX9K2 board >>> >>> Signed-off-by: Jens Scharsig<js_at...@scharsoft.de> >>> --- >>> include/configs/eb_cpux9k2.h | 4 ++++ >>> 1 files changed, 4 insertions(+), 0 deletions(-) >>> >>> diff --git a/include/configs/eb_cpux9k2.h b/include/configs/eb_cpux9k2.h >>> index 4324172..e437f25 100644 >>> --- a/include/configs/eb_cpux9k2.h >>> +++ b/include/configs/eb_cpux9k2.h >>> @@ -42,6 +42,10 @@ >>> #define CONFIG_BOARD_EARLY_INIT_F >>> >>> >>> /*------------------------------------------------------------------------ >>> --*/ >>> +#ifndef MACH_TYPE_EB_CPUX9K2 >>> +#define MACH_TYPE_EB_CPUX9K2 1977 >>> +#endif >>> +/*------------------------------------------------------------------------ >>> --*/ >>> #define CONFIG_SYS_TEXT_BASE 0x00000000 >>> #define CONFIG_SYS_LOAD_ADDR 0x21000000 /* default load >>> address */ >> >> You don't need the ifndef. Cheers > > I'd even say you need to not have them: this way, when the MACH_ID shows > up eventually in machine-type, the preprocessor will emit a warning for > the redefinition in the config file, thus letting us know it is time to > remove the temporary MACH_TYPE there. > > Amicalement,
Ok, I will send a straightforward variant as V2 patch Regards Jens _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot