diff options
author | Felix Held <felix.held@amd.corp-partner.google.com> | 2021-08-28 00:36:17 +0200 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2021-09-05 17:52:53 +0000 |
commit | c958f84d404b3337c93e0457259dc7a235b868dc (patch) | |
tree | e17ac7738bff2fa02e5b291a6d328b27c3e8af07 | |
parent | a6274992ff5bcc76be3e6eb18467bab22d6cfa9b (diff) |
drivers/intel/fsp/Makefile: error out when FSP files aren't specified
Error out when the FSP binaries that are supposed to be added aren't
specified.
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Suggested-by: Nico Huber <nico.h@gmx.de>
Change-Id: Ie5f2d75d066f0b4e491e9c8420b7a0cbd4ba9e28
Reviewed-on: https://review.coreboot.org/c/coreboot/+/57219
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
-rw-r--r-- | src/drivers/intel/fsp2_0/Makefile.inc | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/src/drivers/intel/fsp2_0/Makefile.inc b/src/drivers/intel/fsp2_0/Makefile.inc index b472721550..80dc93b75e 100644 --- a/src/drivers/intel/fsp2_0/Makefile.inc +++ b/src/drivers/intel/fsp2_0/Makefile.inc @@ -97,6 +97,21 @@ ifneq ($(call strip_quotes,$(CONFIG_FSP_HEADER_PATH)),) CPPFLAGS_common+=-I$(CONFIG_FSP_HEADER_PATH) endif +# check if the FSP files that are supposed to be added are specified +ifeq ($(CONFIG_ADD_FSP_BINARIES),y) +ifeq ($(CONFIG_FSP_CAR),y) +ifeq ($(call strip_quotes,$(CONFIG_FSP_T_FILE)),) +$(error No FSP-T binary file specified.) +endif # CONFIG_FSP_T_FILE +endif # CONFIG_FSP_CAR +ifeq ($(call strip_quotes,$(CONFIG_FSP_M_FILE)),) +$(error No FSP-M binary file specified.) +endif # CONFIG_FSP_M_FILE +ifeq ($(call strip_quotes,$(CONFIG_FSP_S_FILE)),) +$(error No FSP-S binary file specified.) +endif # CONFIG_FSP_S_FILE +endif # CONFIG_ADD_FSP_BINARIES + subdirs-y += ppi endif |