[coreboot-gerrit] Patch set updated for coreboot: cpu/microcode: Remove unused EXTERNAL and ADDED_DURING_BUILD variables
Aaron Durbin (adurbin@chromium.org)
gerrit at coreboot.org
Thu Oct 15 00:46:57 CEST 2015
Aaron Durbin (adurbin at chromium.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11903
-gerrit
commit 93ee3cf0a8d9b9b8277c1ca4f825ecbdab4bd977
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date: Wed Oct 14 15:41:49 2015 -0500
cpu/microcode: Remove unused EXTERNAL and ADDED_DURING_BUILD variables
Change-Id: Ifea94c21e531a74953f5a0e2f489378c20ef3b5c
Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
Makefile.inc | 2 +-
src/cpu/Kconfig | 32 --------------------------------
src/cpu/Makefile.inc | 6 ------
3 files changed, 1 insertion(+), 39 deletions(-)
diff --git a/Makefile.inc b/Makefile.inc
index 94d33d3..785396b 100644
--- a/Makefile.inc
+++ b/Makefile.inc
@@ -656,7 +656,7 @@ ifeq ($(CONFIG_PXE_ROM),y)
$(CBFSTOOL) $@.tmp add -f $(CONFIG_PXE_ROM_FILE) -n pci$(CONFIG_PXE_ROM_ID).rom -t raw
endif
ifeq ($(CONFIG_CPU_INTEL_FIRMWARE_INTERFACE_TABLE),y)
-ifeq ($(CONFIG_CPU_MICROCODE_ADDED_DURING_BUILD),y)
+ifeq ($(CONFIG_CPU_MICROCODE_CBFS_GENERATE),y)
@printf " UPDATE-FIT \n"
$(CBFSTOOL) $@.tmp update-fit -n cpu_microcode_blob.bin -x $(CONFIG_CPU_INTEL_NUM_FIT_ENTRIES)
endif
diff --git a/src/cpu/Kconfig b/src/cpu/Kconfig
index 578bab2..9b9413c 100644
--- a/src/cpu/Kconfig
+++ b/src/cpu/Kconfig
@@ -67,17 +67,6 @@ config SUPPORT_CPU_UCODE_IN_CBFS
bool
default n
-# This variable is used to determine if we add CPU microcode to CBFS during the
-# build. Microcode can be added manually afterwards, or removed. As a result,
-# code should not rely on this to tell if a microcode update is present or not,
-# and should instead search CBFS.
-# This variable is useful in determining if certain automated post-processing
-# steps can be performed right after the build, such as automatically adding
-# a firmware interface table.
-config CPU_MICROCODE_ADDED_DURING_BUILD
- bool
- default y if CPU_MICROCODE_CBFS_GENERATE || CPU_MICROCODE_CBFS_EXTERNAL
-
choice
prompt "Include CPU microcode in CBFS" if ARCH_X86
default CPU_MICROCODE_CBFS_GENERATE if SUPPORT_CPU_UCODE_IN_CBFS && USE_BLOBS
@@ -95,20 +84,6 @@ config CPU_MICROCODE_CBFS_GENERATE
If unsure, select this option.
-config CPU_MICROCODE_CBFS_EXTERNAL
- bool "Include external microcode file"
- help
- Select this option if you want to include an external file containing
- the CPU microcode. This will be included as a separate file in CBFS.
- A word of caution: only select this option if you are sure the
- microcode that you have is newer than the microcode shipping with
- coreboot.
-
- The microcode file may be removed from the ROM image at a later
- time with cbfstool, if desired.
-
- If unsure, select "Generate from tree"
-
config CPU_MICROCODE_CBFS_NONE
bool "Do not include microcode updates"
help
@@ -160,10 +135,3 @@ config CPU_MICROCODE_MULTIPLE_FILES
help
Select this option to install separate microcode container files into
CBFS instead of using the traditional monolithic microcode file format.
-
-config CPU_MICROCODE_FILE
- string "Path and filename of CPU microcode"
- depends on CPU_MICROCODE_CBFS_EXTERNAL
- default "cpu_microcode.bin"
- help
- The path and filename of the file containing the CPU microcode.
diff --git a/src/cpu/Makefile.inc b/src/cpu/Makefile.inc
index e76308c..e92e2f7 100644
--- a/src/cpu/Makefile.inc
+++ b/src/cpu/Makefile.inc
@@ -17,12 +17,6 @@ $(eval $(call create_class_compiler,cpu_microcode,x86_32))
## Rules for building the microcode blob in CBFS
################################################################################
-# External microcode file, or are we generating one ?
-ifeq ($(CONFIG_CPU_MICROCODE_CBFS_EXTERNAL), y)
-cpu_ucode_cbfs_file = $(call strip_quotes,$(CONFIG_CPU_MICROCODE_FILE))
-cbfs_include_ucode = y
-endif
-
ifeq ($(CONFIG_CPU_MICROCODE_CBFS_GENERATE), y)
ifeq ($(CONFIG_CPU_MICROCODE_MULTIPLE_FILES), n)
cpu_ucode_cbfs_file = $(obj)/cpu_microcode_blob.bin
More information about the coreboot-gerrit
mailing list