diff options
| author | Patrick Georgi <pgeorgi@chromium.org> | 2016-01-28 21:57:29 +0100 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2016-01-29 17:19:16 +0100 |
| commit | ddc30767dce49da80c10cdb981a8c06a533bac28 (patch) | |
| tree | 786e90cb7034c09187022415b9ddebf637dc0d43 /Makefile.inc | |
| parent | 60fad89091dec4f34803a3320de73198b1124002 (diff) | |
| download | coreboot-ddc30767dce49da80c10cdb981a8c06a533bac28.tar.gz coreboot-ddc30767dce49da80c10cdb981a8c06a533bac28.tar.bz2 coreboot-ddc30767dce49da80c10cdb981a8c06a533bac28.zip | |
build system: deduplicate users of cbfs-add-cmd
When adding the cbfstool remove requirement of the UPDATE_IMAGE path to
cbfs-add-cmd, prebuil[dt]-files become identical in both cases.
Change-Id: I80faaf1c83368b9dd00a9f247bf89e6d596be996
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-on: https://review.coreboot.org/13503
Tested-by: build bot (Jenkins)
Tested-by: Raptor Engineering Automated Test Stand <noreply@raptorengineeringinc.com>
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'Makefile.inc')
| -rw-r--r-- | Makefile.inc | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/Makefile.inc b/Makefile.inc index 3fcddbc1d3e4..84d7a2de7b5b 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -651,6 +651,7 @@ endif define cbfs-add-cmd printf " CBFS $(call extract_nth,2,$(1))\n" + $(if $(2),-$(CBFSTOOL) $@.tmp remove -n $(call extract_nth,2,$(file)) 2>/dev/null) $(CBFSTOOL) $@.tmp \ add$(if $(filter stage,$(call extract_nth,3,$(1))),-stage)$(if $(filter payload,$(call extract_nth,3,$(1))),-payload) \ -f $(call extract_nth,1,$(1)) \ @@ -665,10 +666,10 @@ define cbfs-add-cmd endef cbfs-files=$(cbfs-files-fixed) $(cbfs-files-aligned) $(cbfs-files-regular) -ifneq ($(CONFIG_UPDATE_IMAGE),y) -prebuild-files = $(foreach file,$(cbfs-files), $(call cbfs-add-cmd,$(file))) prebuilt-files = $(foreach file,$(cbfs-files), $(call extract_nth,1,$(file))) +prebuild-files = $(foreach file,$(cbfs-files), $(call cbfs-add-cmd,$(file),$(CONFIG_UPDATE_IMAGE))) +ifneq ($(CONFIG_UPDATE_IMAGE),y) ifeq ($(CONFIG_FMDFILE),) # For a description of the flash layout described by these variables, check # the $(DEFAULT_FLASHMAP) .fmd files. @@ -760,13 +761,6 @@ endif # ifeq ($(CONFIG_ARCH_X86),y) $(prebuild-files) true mv $@.tmp $@ else # ifneq ($(CONFIG_UPDATE_IMAGE),y) -prebuilt-files = $(foreach file,$(cbfs-files), $(call extract_nth,1,$(file))) -# Add all cbfs files to image of the form: CONFIG_CBFS_PREFIX/<filename> -prebuild-files = \ - $(foreach file,$(cbfs-files), \ - $(CBFSTOOL) $@.tmp remove -n $(call extract_nth,2,$(file)) 2>/dev/null ; \ - $(call cbfs-add-cmd,$(file))) - .PHONY: $(obj)/coreboot.pre $(obj)/coreboot.pre: $$(prebuilt-files) $(CBFSTOOL) mv $(obj)/coreboot.rom $@.tmp || \ |