[U-Boot] [PATCH v2 1/2] mkconfig: split the board make target to multiple config targets

[Wolfgang Denk]

Dear Mingkai Hu,

In message <1252466603-25103-1-git-send-email-Mingkai.hu at freescale.com> you wrote:
> To simplify the top level makefile it useful to be able to parse
> the top level makefile target to multiple individual target, then
> put them to the config.h, leave the board config file to handle
> the different targets.
> 
> Note that this method uses the '_'(underline) as the delimiter when
> splits the board make target.
> 
> Signed-off-by: Mingkai Hu <Mingkai.hu at freescale.com>
> ---
> 
> According to the comments from Wolfgang and Scott, I modified
> the patch and made some modification over v1:
> 
>  - remove the sectence thats puts the splited variables to the
>    config.mk, we can use the CONFIG_MK_* in the board config file
>    to override the variable in the board config file.
> 
>  - change CONFIG_OPT_* to CONFIG_MK_*
> 
>  mkconfig |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)

Arghhh... It seems this patch has not been well tested. Obviously
never ever ran a MAKEALL over all PPC boards. It causes tons of
messages like that:

include/config.h:2:25: warning: missing whitespace after the macro name

This patch breaks building for a number of boards / configurations,
especially:

xilinx-ppc405-generic_flash
xilinx-ppc405-generic
xilinx-ppc440-generic_flash
xilinx-ppc440-generic
VoVPN-GW_66MHz
VoVPN-GW_100MHz
EB+MCF-EV123
EB+MCF-EV123_internal
microblaze-generic
favr-32-ezkit


Shall I revert that patch?

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
"Nature is very un-American.  Nature never hurries."
- William George Jordan