[U-Boot] [PATCH v2 2/3] arm: omap: innovator: Prepare for mach-types.h changes

Wolfgang Denk wd at denx.de
Tue May 3 14:29:54 CEST 2011


Dear Igor Grinberg,

In message <1304417333-30745-1-git-send-email-grinberg at compulab.co.il> you wrote:
> This board used machine_is_* macros for identifying the arch number.
> Fix this by introducing a board specific configuration variable.
> 
> Signed-off-by: Igor Grinberg <grinberg at compulab.co.il>
> ---
> v2:	remove the ifdeferry by introducing config variable,
> 	Alessandro, what about this one?
...
> +	gd->bd->bi_arch_number = CONFIG_BOARD_MACH_TYPE;

In principle this is OK, but why do you invent yet another new CONFIG_
variable (and without documenting it) ?

We have a number of boards that already use a similar construct with
CONFIG_MACH_TYPE, so I suggest you do the same.

And while being there, could you please also add a description for
CONFIG_MACH_TYPE to the README?   Thanks!

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
If the car industry behaved like the computer industry over the  last
30  years, a Rolls-Royce would cost $5, get 300 miles per gallon, and
blow up once a year killing all passengers inside.


More information about the U-Boot mailing list