[coreboot-gerrit] New patch to review for coreboot: Kconfig: Add option for microcode filenames

Martin Roth (martinroth@google.com) gerrit at coreboot.org
Thu Sep 1 00:08:54 CEST 2016


Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16386

-gerrit

commit 69ddbe8b0c49c7a8551c7e7ff0649799c20e0797
Author: Martin Roth <martinroth at chromium.org>
Date:   Tue Aug 30 16:21:53 2016 -0600

    Kconfig: Add option for microcode filenames
    
    Hardcoding the microcode filenames into the makefiles is great when
    the microcode is in the blobs directory.  When the microcode isn't
    posted to the blobs directory, we need some method of supplying the
    microcode binary into the build.  This can of course be done manually
    after the build has completed, as can be done with everything that
    we're including in the ROM image.  Instead of making life hard for
    everyone though, let's just add a way to specify where the microcode
    rom comes from.
    
    BUG=chrome-os-partner:53013
    
    Change-Id: I7c5127234809e8515906efa56c04af6005eecf0b
    Signed-off-by: Martin Roth <martinroth at chromium.org>
---
 src/cpu/Kconfig                      | 15 +++++++++++++++
 src/cpu/intel/microcode/Makefile.inc |  2 ++
 2 files changed, 17 insertions(+)

diff --git a/src/cpu/Kconfig b/src/cpu/Kconfig
index a026b28..3f0e7a2 100644
--- a/src/cpu/Kconfig
+++ b/src/cpu/Kconfig
@@ -166,3 +166,18 @@ config CPU_MICROCODE_HEADER_FILES
 	help
 	  A list of one or more microcode header files with path from the
 	  coreboot directory.  These should be separated by spaces.
+
+config CPU_UCODE_BINARIES
+	string "path and filename of microcode binary files"
+	depends on CPU_MICROCODE_CBFS_GENERATE
+	default ""
+	help
+	  Some platforms have microcode in the blobs directory, and these can
+	  be hardcoded in the makefiles.  For platforms with microcode
+	  binaries that aren't in the makefile, set this option to pull
+	  in the microcode.
+
+	  This should contain the full path and filename of one or more
+	  microcode binary files to include, separated by spaces.
+
+	  If unsure, leave this blank.
diff --git a/src/cpu/intel/microcode/Makefile.inc b/src/cpu/intel/microcode/Makefile.inc
index f589430..b56e6a7 100644
--- a/src/cpu/intel/microcode/Makefile.inc
+++ b/src/cpu/intel/microcode/Makefile.inc
@@ -4,3 +4,5 @@
 ################################################################################
 ramstage-$(CONFIG_SUPPORT_CPU_UCODE_IN_CBFS) += microcode.c
 romstage-$(CONFIG_SUPPORT_CPU_UCODE_IN_CBFS) += microcode.c
+
+cpu_microcode_bins += $(call strip_quotes,$(CONFIG_CPU_UCODE_BINARIES))



More information about the coreboot-gerrit mailing list