diff options
author | Julius Werner <jwerner@chromium.org> | 2021-07-08 21:30:08 +0000 |
---|---|---|
committer | Julius Werner <jwerner@chromium.org> | 2021-07-09 00:52:10 +0000 |
commit | 4676279151659922da8717712c523a103575a21a (patch) | |
tree | 6d4738753de75bf3573dfc36512e6d6555d506ec /Makefile.inc | |
parent | ec5a5d7abf32230450265d20857261b049fef2d5 (diff) |
Revert "Makefile.inc: Drop the cbfs master header from non-X86"
This reverts commit d109354c0f1d4b155c60701cd42e632213350d72.
Reason for revert: Breaks libpayload CBFS code when accessing
non-default CBFS.
BUG=b:193093750
Change-Id: Id7f47406e6126f19e1fd6bc9d33c8c9d0cb9450d
Signed-off-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/56130
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Diffstat (limited to 'Makefile.inc')
-rw-r--r-- | Makefile.inc | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/Makefile.inc b/Makefile.inc index fee73f5c06..b1c706709a 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -1101,15 +1101,22 @@ ifeq ($(CONFIG_ARCH_X86),y) -b -$(call file-size,$(objcbfs)/bootblock.bin) $(cbfs-autogen-attributes) \ $(TS_OPTIONS) \ $(CBFSTOOL_ADD_CMD_OPTIONS) - # the cbfs master header is a deprecated feature only used on x86 - $(CBFSTOOL) $@.tmp add-master-header $(TS_OPTIONS) $(CBFSTOOL_ADD_CMD_OPTIONS) else # ifeq ($(CONFIG_ARCH_X86),y) $(CBFSTOOL) $@.tmp write -u \ -r BOOTBLOCK \ -f $(objcbfs)/bootblock.bin # make space for the CBFS master header pointer. "ptr_" is just # arbitrary 4 bytes that will be overwritten by add-master-header. + printf "ptr_" > $@.tmp.2 + $(CBFSTOOL) $@.tmp add \ + -f $@.tmp.2 \ + -n "header pointer" \ + -t "cbfs header" \ + -b -4 \ + $(CBFSTOOL_ADD_CMD_OPTIONS) + rm -f $@.tmp.2 endif # ifeq ($(CONFIG_ARCH_X86),y) + $(CBFSTOOL) $@.tmp add-master-header $(TS_OPTIONS) $(CBFSTOOL_ADD_CMD_OPTIONS) $(prebuild-files) true mv $@.tmp $@ else # ifneq ($(CONFIG_UPDATE_IMAGE),y) |