[coreboot-gerrit] Patch set updated for coreboot: intel/fsp1.0: Get size of microcode during build time

Alexandru Gagniuc (mr.nuke.me@gmail.com) gerrit at coreboot.org
Tue Sep 29 17:58:05 CET 2015


Alexandru Gagniuc (mr.nuke.me at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11709

-gerrit

commit 47d4cb3b2db9944e85038d20b9486ceb7b0293b8
Author: Werner Zeh <werner.zeh at siemens.com>
Date:   Fri Sep 25 07:54:59 2015 +0200

    intel/fsp1.0: Get size of microcode during build time
    
    Avoid specifying the size of the microcode in microcode_size.h.
    Instead, the size will be determined during build time and
    microcode_size.h will be generated. This way, the size does
    not need to be adjusted by hand.
    
    Change-Id: I868f02b0cc03af12464a6a87c59761c200eb2502
    Signed-off-by: Werner Zeh <werner.zeh at siemens.com>
---
 src/drivers/intel/fsp1_0/Makefile.inc | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/drivers/intel/fsp1_0/Makefile.inc b/src/drivers/intel/fsp1_0/Makefile.inc
index 11ff31a..0ec52ec 100644
--- a/src/drivers/intel/fsp1_0/Makefile.inc
+++ b/src/drivers/intel/fsp1_0/Makefile.inc
@@ -23,8 +23,15 @@ romstage-y += fsp_util.c hob.c
 ramstage-$(CONFIG_ENABLE_MRC_CACHE) += fastboot_cache.c
 romstage-$(CONFIG_ENABLE_MRC_CACHE) += fastboot_cache.c
 
-CPPFLAGS_common += -Isrc/drivers/intel/fsp1_0
+CPPFLAGS_common += -Isrc/drivers/intel/fsp1_0 -I$(objgenerated)
 
+ifneq ($(cpu_microcode-objs),)
+$(objgenerated)/microcode_size.h: $(obj)/cpu_microcode_blob.bin
+	printf "#define MICROCODE_REGION_LENGTH $(call file-size,$<)" > $@.tmp \
+	&& cmp $@.tmp $@ 2>/dev/null || mv $@.tmp $@
+
+cpu_incs-$(CONFIG_PLATFORM_USES_FSP1_0) += $(objgenerated)/microcode_size.h
+endif
 cpu_incs-$(CONFIG_USE_GENERIC_FSP_CAR_INC) += $(src)/drivers/intel/fsp1_0/cache_as_ram.inc
 
 ifeq ($(CONFIG_HAVE_FSP_BIN),y)
@@ -45,3 +52,4 @@ mrc.cache-file := $(obj)/mrc.cache
 mrc.cache-align := 0x10000
 mrc.cache-type := mrc_cache
 endif
+



More information about the coreboot-gerrit mailing list