diff options
author | Patrick Georgi <pgeorgi@chromium.org> | 2016-01-28 15:00:54 +0100 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2016-01-29 16:37:02 +0100 |
commit | dcc3ecc940e95cb6a97dbea203d3680f67910b59 (patch) | |
tree | 7e11e670f27fd8e4fedac6d985bbf53de9a13b6a | |
parent | 988ee17fd2486415e8be4ebf49f57b5a1b97e685 (diff) |
arch/arm64: Override bl31 timestamp with coreboot build timestamp
If coreboot's build process is reproducible (eg. using the latest git
timestamp as source), bl31 is, too.
This requires an arm-trusted-firmware side merge first (in progress) and
an update of our reference commit for the submodule, but it also doesn't
hurt anything because it merely sets a variable that currently goes
unused.
Change-Id: If139538a2fab5b3a70c67f4625aa2596532308f7
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-on: https://review.coreboot.org/13497
Tested-by: build bot (Jenkins)
Reviewed-by: Alexander Couzens <lynxis@fe80.eu>
-rw-r--r-- | src/arch/arm64/Makefile.inc | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/arch/arm64/Makefile.inc b/src/arch/arm64/Makefile.inc index 33755d3035..ec559fa42d 100644 --- a/src/arch/arm64/Makefile.inc +++ b/src/arch/arm64/Makefile.inc @@ -155,12 +155,18 @@ endif # CONFIG_CONSOLE_SERIAL # Avoid build/release|build/debug distinction by overriding BUILD_PLAT directly BL31_MAKEARGS += BUILD_PLAT="$(abspath $(obj)/3rdparty/arm-trusted-firmware)" +# Set a consistent build timestamp: the same coreboot has +# The \# \" complications exist to satisfy both gnu make's parser and editors +# with non-semantic quote-handling (that would assume that this line starts a +# multi line string. +BL31_MAKEARGS += BUILD_MESSAGE_TIMESTAMP="$(shell grep "\#define COREBOOT_BUILD\>" $(obj)/build.h |cut -d\" -f2 \# \")" + BL31_CFLAGS := -fno-pic -fno-stack-protector BL31_LDFLAGS := --emit-relocs BL31 := $(obj)/bl31.elf -$(BL31): +$(BL31): $(obj)/build.h @printf " MAKE $(subst $(obj)/,,$(@))\n" CROSS_COMPILE="$(CROSS_COMPILE_arm64)" \ CFLAGS="$(BL31_CFLAGS)" \ |