[U-Boot] [PATCH 1/2] Removed CONFIG_NUM_CPUS for 85xx and 86xx Freescale processors.

[Wolfgang Denk]

Dear Kumar Gala,

In message <D46E33D8-E298-4C7F-BF99-1B0269148AB8 at kernel.crashing.org> you wrote:
> 
> > Sorry, but this does not scale. Please find a better solution.
> 
> is there a better solution?  We effectively do things like this in the  
> board CONFIG.h and its just duplicated for every board.  Here its just  
> isolated.  Just wondering if you had any suggestions (I agree its ugly  
> and if/when we get Kconfig its something that could be put there).

Well, no, sorry, I don't have any clever suggestion.

It would at least be a bit shorter if you used

#if defined(..) || defined(..) || defined(..) ...
#define CONFIG_MAX_CPUS	1
#elif defined(..) || defined(..) || defined(..) ...
#define CONFIG_MAX_CPUS 2
#else
#error
#endif

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
"Today's robots are very primitive, capable of understanding  only  a
few  simple  instructions  such  as 'go left', 'go right', and 'build
car'."                                                  - John Sladek