diff options
author | Nico Huber <nico.h@gmx.de> | 2020-05-27 18:08:56 +0200 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2020-10-19 07:04:43 +0000 |
commit | e16971ac64dc36375e5d606cd51950f2807c5bdb (patch) | |
tree | 6a634824693b13b7fc9d2ccc2bed7dbafb1e6a6c | |
parent | 5c80519466863ae7b8579f1edb90829f7f9ee03d (diff) | |
download | coreboot-e16971ac64dc36375e5d606cd51950f2807c5bdb.tar.xz |
util/supermicro: Always include commonlib/bsd/compiler.h
We rely on `compiler.h` for definitions like `__packed`. Without it,
`smcbiosinfo.c` simply declared a global struct with that name, but
nothing was packed.
Found-by: reproducibility test
Change-Id: Ide055317115fc374a63812bcd3791445ca4f2dcc
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41784
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Michael Niewöhner <foss@mniewoehner.de>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
-rw-r--r-- | util/supermicro/Makefile.inc | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/util/supermicro/Makefile.inc b/util/supermicro/Makefile.inc index 1aa5bcb437..316cb48864 100644 --- a/util/supermicro/Makefile.inc +++ b/util/supermicro/Makefile.inc @@ -1,9 +1,11 @@ +TOOLCPPFLAGS += -include $(top)/src/commonlib/bsd/include/commonlib/bsd/compiler.h + SMCBIOSINFOTOOL:= $(objutil)/supermicro/smcbiosinfo $(SMCBIOSINFOTOOL): $(dir)/smcbiosinfo/smcbiosinfo.c printf " HOSTCC Creating SMCBIOSINFO tool\n" mkdir -p $(objutil)/supermicro - $(HOSTCC) $< -o $@ + $(HOSTCC) $(TOOLCPPFLAGS) $< -o $@ ifeq ($(CONFIG_VENDOR_SUPERMICRO),y) ifneq ($(call strip_quotes, $(CONFIG_SUPERMICRO_BOARDID)),) |