diff options
| author | Angel Pons <th3fanbus@gmail.com> | 2018-09-11 13:49:45 +0200 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2018-09-13 08:25:59 +0000 |
| commit | a52016cc46cdf0672448d9f25060c3c35e8919c4 (patch) | |
| tree | 9bb156a650d7a3e2bf12e7318e277efce3c1b73d /src/southbridge/intel/common/firmware | |
| parent | ef8b95745f4b72439fb5f108acf0a62361f64101 (diff) | |
src/*/intel: introduce warning when building with no IFD
Add a warning as suggested in patch CB:28233 with the
"CONFIG_INTEL_DESCRIPTOR_MODE_REQUIRED" option.
Change-Id: I42b6b336bb519f3d18b5a41eb20b380636ff5819
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/28382
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'src/southbridge/intel/common/firmware')
| -rw-r--r-- | src/southbridge/intel/common/firmware/Makefile.inc | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/src/southbridge/intel/common/firmware/Makefile.inc b/src/southbridge/intel/common/firmware/Makefile.inc index 426863c658..01a80614e3 100644 --- a/src/southbridge/intel/common/firmware/Makefile.inc +++ b/src/southbridge/intel/common/firmware/Makefile.inc @@ -22,6 +22,8 @@ ifeq ($(CONFIG_HAVE_INTEL_FIRMWARE),y) ifeq ($(CONFIG_HAVE_IFD_BIN),y) INTERMEDIATE+=add_intel_firmware +else ifeq ($(CONFIG_INTEL_DESCRIPTOR_MODE_REQUIRED),y) +INTERMEDIATE += warn_intel_firmware endif IFD_BIN_PATH := $(CONFIG_IFD_BIN_PATH) @@ -85,6 +87,13 @@ ifeq ($(CONFIG_EM100),y) mv $(obj)/coreboot.pre.new $(obj)/coreboot.pre endif -PHONY+=add_intel_firmware +warn_intel_firmware: + printf "\n\t** WARNING **\n" + printf "coreboot will be built without an Intel Firmware Descriptor.\n" + printf "Never write a complete coreboot.rom without an IFD to your\n" + printf "board's flash chip! You can use flashrom's IFD or layout\n" + printf "parameters to flash only to the BIOS region.\n\n" + +PHONY+=add_intel_firmware warn_intel_firmware endif |