the following patch was just integrated into master:
commit 8346aca83a88f31781f190c1988d90a05b31425b
Author: Martin Roth <martinroth(a)google.com>
Date: Wed Nov 4 11:52:41 2015 -0700
Kconfig: Remove obsolete Kconfig symbols from google/intel boards
- CACHE_ROM is no longer used in the coreboot code. It was removed in
commit 4337020b (Remove CACHE_ROM.)
- CAR_MIGRATION is also no longer used in coreboot code - it was removed
in commit cbf5bdfe (CBMEM: Always select CAR_MIGRATION)
- MARK_GRAPHICS_MEM_WRCOMB was removed in
commit 30fe6120 (MTRR: Mark all prefetchable resources as WRCOMB)
Change-Id: I8b33a08c256f6b022e57e9af60d0629d9a3ffac8
Signed-off-by: Martin Roth <martinroth(a)google.com>
Reviewed-on: http://review.coreboot.org/12327
Tested-by: build bot (Jenkins)
Reviewed-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Reviewed-by: Nico Huber <nico.h(a)gmx.de>
See http://review.coreboot.org/12327 for details.
-gerrit
Martin Roth (martinroth(a)google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/12337
-gerrit
commit f755ec40f70a360cc0241e26cb9e288db096df14
Author: Martin Roth <martinroth(a)google.com>
Date: Thu Nov 5 12:30:06 2015 -0700
fsp1_0: fix microcode loading
This should fix microcode loading by the FSP.
- Update the name of the microcode file used to find the size to allow
the file generated from the .h file to be loaded.
- Add CPU_MICROCODE_CBFS_LOC back to FSP 1.0 Kconfig. This is still
needed for loading microcode by the FSP.
Change-Id: Ia6de09f7f4d18b03fb415eaa09dc755d3b25cdbc
Signed-off-by: Martin Roth <martinroth(a)google.com>
---
src/drivers/intel/fsp1_0/Kconfig | 7 +++++++
src/drivers/intel/fsp1_0/Makefile.inc | 4 ++--
2 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/src/drivers/intel/fsp1_0/Kconfig b/src/drivers/intel/fsp1_0/Kconfig
index 36bfa7c..5610eba 100644
--- a/src/drivers/intel/fsp1_0/Kconfig
+++ b/src/drivers/intel/fsp1_0/Kconfig
@@ -36,6 +36,13 @@ config DCACHE_RAM_SIZE
if HAVE_FSP_BIN
+config CPU_MICROCODE_CBFS_LOC
+ hex "Microcode update base address in CBFS"
+ default 0
+ help
+ The location (base address) in CBFS that contains the microcode update
+ binary.
+
config FSP_FILE
string "Intel FSP binary path and filename"
help
diff --git a/src/drivers/intel/fsp1_0/Makefile.inc b/src/drivers/intel/fsp1_0/Makefile.inc
index aac832b..e62047e 100644
--- a/src/drivers/intel/fsp1_0/Makefile.inc
+++ b/src/drivers/intel/fsp1_0/Makefile.inc
@@ -21,8 +21,8 @@ romstage-$(CONFIG_ENABLE_MRC_CACHE) += fastboot_cache.c
CPPFLAGS_common += -Isrc/drivers/intel/fsp1_0 -I$(objgenerated)
-ifneq ($(cpu_microcode_bins),)
-$(objgenerated)/microcode_size.h: $(obj)/cpu_microcode_blob.bin
+ifneq ($(cpu_microcode_blob.bin-file),)
+$(objgenerated)/microcode_size.h: $(cpu_microcode_blob.bin-file)
printf "#define MICROCODE_REGION_LENGTH $(call file-size,$<)" > $@.tmp \
&& cmp $@.tmp $@ 2>/dev/null || mv $@.tmp $@
else