[U-Boot] [PATCH 18/42] Blackfin: make sure autoconf.mk is generated early enough
Mike Frysinger
vapier at gentoo.org
Tue Feb 10 20:49:23 CET 2009
On Tuesday 10 February 2009 14:42:20 Wolfgang Denk wrote:
> Dear Mike Frysinger,
>
> In message <200902101357.29507.vapier at gentoo.org> you wrote:
> > > > $(BFIN_BOARDS:%=%_config) : unconfig
> > > > @$(MKCONFIG) $(@:_config=) blackfin blackfin $(@:_config=)
> > > > + @$(MAKE) -s -B $(obj)include/autoconf.mk
> > > > + @$(MAKE) -s -B $(obj)include/autoconf.mk
> > >
> > > Do you really mean to do this twice?
> >
> > unfortunately, yes. since some settings in the board config are turned
> > into compiler flags and those compiler flags can in turn affect the board
> > config, we need to do it twice. first is to make sure the proper cpu
> > flags are propagated into the toplevel build env while the second is to
> > make sure the autoconf.mk fully reflects the board config.
>
> Sounds like a design problem to me.
not really. the point is to avoid duplication and considering the method to
attain that, sounds pretty good to me.
> > i guess i could add a one line comment above each one giving hints about
> > why each is needed ...
>
> That would be the minimum, but given the fact that the top level
> Makefile already includes rules to build autoconf.mk I really wonder
> if we must do this so often, and if so, then why this is only the
> case for blackfin.
the top level Makefile includes rules to build it, but it doesnt re-source it
once it's been generated. so anything in the top level cannot use things from
autoconf.mk (like $(arch)_config.mk).
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20090210/e9b4a053/attachment.pgp
More information about the U-Boot
mailing list