Dear Mingkai Hu, In message <1252466603-25103-1-git-send-email-mingkai...@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...@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: w...@denx.de "Nature is very un-American. Nature never hurries." - William George Jordan _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
