> -----Original Message-----
> From: Wolfgang Denk [mailto:w...@denx.de]
> Sent: Monday, September 07, 2009 6:59 PM
> To: Hu Mingkai-B21284
> Cc: Kumar Gala; Wood Scott-B07421; U-Boot-Users ML; Mike Frysinger
> Subject: Re: [PATCH 01/10] mkconfig: parse top level makefile
> target to multiple config targets
>
> Dear "Hu Mingkai-B21284",
>
> In message
> <73839b4a0818e747864426270ac332c30447e...@zmy16exm20.fsl.frees
> cale.net> you wrote:
> >
> > > > > It should be enough to pass the make target name to
> the mkconfig
> > > > > script resp. the board config file, i. e. in this case either
> > > > > "CONFIG_MPC8536DS_NAND" or "CONFIG_MPC8536DS_NAND_36BIT".
> > > The rest
> > > > > of the scripting/decision making can then be done in
> the board
> > > > > config file.
> > > > >
> > > >
> > > > Thanks for your replay, but I'm not totally catch on you.
> > > > "the board config file" in your words refer to
> > > board/*/config.mk, right?
> > >
> > > No, with "board config file" I mean include/configs/*.h
> >
> > How can I parse the board name in a header file?
>
> I'm not sure I understand the question (or rather the problem
> you are seeing).
>
> You already know the board name, because the board config
> file is clearly related ot one (or eventually more) boards.
> The rest can be done with some trivial #ifdef'fery.
>
Oh... I complicated the matters, you means as the follows, right?
In the Makefile:
MPC8536DS_NAND_config \
MPC8536DS_NAND_36BIT_config \
MPC8536DS_36BIT_config \
MPC8536DS_config: unconfig
@echo "#define CONFIG_$(@:_config=) 1" >$(obj)include/config.h
@$(MKCONFIG) -a MPC8536DS ppc mpc85xx mpc8536ds freescale
then in the include/configs/*.h:
#ifdef MPC8536DS_NAND
blablabla
#endif
#ifdef MPC8536DS_NAND_36BIT
blablabla
#endif
#ifdef MPC8536DS_36BIT
blablabla
#endif
> > If config booting from NAND, we also need to override the
> TEXT_BASE in
> > the board/*/config.mk file, how could we do that?
>
> You can for example set CONFIG_SYS_MONITOR_BASE (or some
> other CONFIG_ variable - but CONFIG_SYS_MONITOR_BASE is used
> anyway) as needed in your include/configs/*.h, which then
> gets exported through include/autoconf.mk, so you can use
> some "TEXT_BASE = $(CONFIG_SYS_MONITOR_BASE)" in your config.mk
>
Thanks.
I'll intergate your comments, align the patchset to the latest U-Boot
and resend the patches, please comments then.
Many thanks,
Mingkai
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
