summaryrefslogtreecommitdiff
path: root/src/cpu/Kconfig
diff options
context:
space:
mode:
authorNico Huber <nico.huber@secunet.com>2018-11-29 17:05:32 +0100
committerNico Huber <nico.h@gmx.de>2019-01-10 09:24:02 +0000
commitf5ca922c879995a151f4ffefd52bfff34fee21ca (patch)
tree7e5af873e0a5cd4176e886a88531807508d1d3d7 /src/cpu/Kconfig
parentcda27c2492929fb4bb734211d9dc3a85202ec1e4 (diff)
downloadcoreboot-f5ca922c879995a151f4ffefd52bfff34fee21ca.tar.xz
Untangle CBFS microcode updates
The option to specify a binary file name was added later for platforms that do not provide microcode updates in our blobs repository. Alas, it wasn't visible what platforms these are. And if you specified a file for a platform that already had one, they were all included together. Make it visible which platforms don't provide binaries with the new con- figs MICROCODE_BLOB_NOT_IN_BLOB_REPO, MICROCODE_BLOB_NOT_HOOKED_UP and MICROCODE_BLOB_UNDISCLOSED. Based on that we can decide if we want to include binaries by default or explicitly show that no files are inclu- ded (default to CPU_MICROCODE_CBFS_NONE). Also split CPU_MICROCODE_CBFS_GENERATE into the more explicit CPU_MICROCODE_CBFS_DEFAULT_BINS and CPU_MICROCODE_CBFS_EXTERNAL_BINS. And clean up the visibility of options: Don't show CBFS related options on platforms that don't support it and don't show external file options if the platform uses special rules for multiple files (CPU_MICROCODE_ MULTIPLE_FILES). Change-Id: Ib403402e240d3531640a62ce93b7a93b4ef6ca5e Signed-off-by: Nico Huber <nico.huber@secunet.com> Reviewed-on: https://review.coreboot.org/c/29934 Reviewed-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/cpu/Kconfig')
-rw-r--r--src/cpu/Kconfig70
1 files changed, 54 insertions, 16 deletions
diff --git a/src/cpu/Kconfig b/src/cpu/Kconfig
index a7cb99e4b3..f98ce6d0eb 100644
--- a/src/cpu/Kconfig
+++ b/src/cpu/Kconfig
@@ -84,14 +84,43 @@ config USES_MICROCODE_HEADER_FILES
This is selected by a board or chipset to set the default for the
microcode source choice to a list of external microcode headers
+config MICROCODE_BLOB_NOT_IN_BLOB_REPO
+ bool
+ help
+ Selected by platforms that don't maintain microcode updates in the
+ blobs repo yet.
+
+config MICROCODE_BLOB_NOT_HOOKED_UP
+ bool
+ help
+ Selected by platforms that haven't hooked microcode updates up yet.
+
+config MICROCODE_BLOB_UNDISCLOSED
+ bool
+ help
+ Selected by work-in-progress platforms that don't have microcode
+ updates available yet.
+
+config USE_CPU_MICROCODE_CBFS_BINS
+ bool
+ help
+ Automatically selected below to add binary microcode files
+ (`cpu_microcode_bins` in the makefiles) to CBFS.
+
choice
prompt "Include CPU microcode in CBFS" if ARCH_X86
default CPU_MICROCODE_CBFS_EXTERNAL_HEADER if USES_MICROCODE_HEADER_FILES
- default CPU_MICROCODE_CBFS_GENERATE if SUPPORT_CPU_UCODE_IN_CBFS && USE_BLOBS
- default CPU_MICROCODE_CBFS_NONE if !SUPPORT_CPU_UCODE_IN_CBFS
+ default CPU_MICROCODE_CBFS_NONE if MICROCODE_BLOB_NOT_IN_BLOB_REPO || \
+ MICROCODE_BLOB_NOT_HOOKED_UP || \
+ MICROCODE_BLOB_UNDISCLOSED
+ depends on SUPPORT_CPU_UCODE_IN_CBFS
-config CPU_MICROCODE_CBFS_GENERATE
+config CPU_MICROCODE_CBFS_DEFAULT_BINS
bool "Generate from tree"
+ select USE_CPU_MICROCODE_CBFS_BINS
+ depends on !(MICROCODE_BLOB_NOT_IN_BLOB_REPO || \
+ MICROCODE_BLOB_NOT_HOOKED_UP || \
+ MICROCODE_BLOB_UNDISCLOSED)
help
Select this option if you want microcode updates to be assembled when
building coreboot and included in the final image as a separate CBFS
@@ -102,8 +131,27 @@ config CPU_MICROCODE_CBFS_GENERATE
If unsure, select this option.
+config CPU_MICROCODE_CBFS_EXTERNAL_BINS
+ bool "Include external microcode binary"
+ select USE_CPU_MICROCODE_CBFS_BINS
+ depends on !CPU_MICROCODE_MULTIPLE_FILES
+ help
+ Select this option if you want to include external binary files
+ in the CPUs native format. They 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, and applicable, select "Generate from tree"
+
config CPU_MICROCODE_CBFS_EXTERNAL_HEADER
bool "Include external microcode header files"
+ depends on !CPU_MICROCODE_MULTIPLE_FILES
help
Select this option if you want to include external c header files
containing the CPU microcode. This will be included as a separate
@@ -116,25 +164,17 @@ config CPU_MICROCODE_CBFS_EXTERNAL_HEADER
The microcode file may be removed from the ROM image at a later
time with cbfstool, if desired.
- If unsure, select "Generate from tree"
+ If unsure, and applicable, select "Generate from tree"
config CPU_MICROCODE_CBFS_NONE
bool "Do not include microcode updates"
help
Select this option if you do not want CPU microcode included in CBFS.
- Note that for some CPUs, the microcode is hard-coded into the source
- tree and is not loaded from CBFS. In this case, microcode will still
- be updated. There is a push to move all microcode to CBFS, but this
- change is not implemented for all CPUs.
-
- This option currently applies to:
- - Intel SandyBridge/IvyBridge
- - VIA Nano
Microcode may be added to the ROM image at a later time with cbfstool,
if desired.
- If unsure, select "Generate from tree"
+ If unsure, and applicable, select "Generate from tree"
The GOOD:
Microcode updates intend to solve issues that have been discovered
@@ -164,8 +204,6 @@ endchoice
config CPU_MICROCODE_MULTIPLE_FILES
bool
- default n
- depends on CPU_MICROCODE_CBFS_GENERATE
help
Select this option to install separate microcode container files into
CBFS instead of using the traditional monolithic microcode file format.
@@ -179,7 +217,7 @@ config CPU_MICROCODE_HEADER_FILES
config CPU_UCODE_BINARIES
string "Microcode binary path and filename"
- depends on CPU_MICROCODE_CBFS_GENERATE
+ depends on CPU_MICROCODE_CBFS_EXTERNAL_BINS
default ""
help
Some platforms have microcode in the blobs directory, and these can