Marshall Dawson has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/34477 )
Change subject: Makefile: Support HAVE_BOOTBLOCK=n case ......................................................................
Makefile: Support HAVE_BOOTBLOCK=n case
With HAVE_BOOTBLOCK=n build of bootblock-class is skipped.
Inserts an empty 64-byte bootblock-region to coreboot.rom file, cbfstool will fill in the CBFS master header relative location at the end.
Change-Id: Iaee9200f72f31175aca597865e3c74fc68bec8a6 Signed-off-by: Kyösti Mälkki kyosti.malkki@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/34477 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Paul Menzel paulepanter@users.sourceforge.net Reviewed-by: Aaron Durbin adurbin@chromium.org Reviewed-by: Marshall Dawson marshalldawson3rd@gmail.com --- M Makefile.inc 1 file changed, 5 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve Aaron Durbin: Looks good to me, approved Marshall Dawson: Looks good to me, approved
diff --git a/Makefile.inc b/Makefile.inc index 2cad230..c275d1e 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -705,6 +705,11 @@ @printf " OBJCOPY $(notdir $(@))\n" $(OBJCOPY_bootblock) -O binary $< $@
+ifneq ($(CONFIG_HAVE_BOOTBLOCK),y) +$(objcbfs)/bootblock.bin: + dd if=/dev/zero of=$@ bs=64 count=1 +endif + $(objcbfs)/%.bin: $(objcbfs)/%.raw.bin cp $< $@