From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Warren Date: Tue, 10 Feb 2009 11:23:21 -0800 Subject: [U-Boot] [PATCH 18/42] Blackfin: make sure autoconf.mk is generated early enough In-Reply-To: <200902101357.29507.vapier@gentoo.org> References: <1234246880-32438-1-git-send-email-vapier@gentoo.org> <1234246880-32438-19-git-send-email-vapier@gentoo.org> <4991BF57.3010305@gmail.com> <200902101357.29507.vapier@gentoo.org> Message-ID: <4991D429.7000908@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Mike Frysinger wrote: > On Tuesday 10 February 2009 12:54:31 Ben Warren wrote: > >> Mike Frysinger wrote: >> >>> We use things from autoconf.mk in the top level Makefile, so it needs to >>> exist before doing a top level make. Otherwise, the top level make env >>> does not get the values in the autoconf.mk. >>> >>> Signed-off-by: Mike Frysinger >>> --- >>> Makefile | 2 ++ >>> 1 files changed, 2 insertions(+), 0 deletions(-) >>> >>> diff --git a/Makefile b/Makefile >>> index 787c5f2..db77d3b 100644 >>> --- a/Makefile >>> +++ b/Makefile >>> @@ -3293,6 +3293,8 @@ BFIN_BOARDS = bf533-ezkit bf533-stamp bf537-stamp >>> bf561-ezkit >>> >>> $(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. > > i guess i could add a one line comment above each one giving hints about why > each is needed ... > -mike > No problem. Just a sanity check. Turns out you're sane :) regards, Ben