[coreboot] Patch merged into coreboot/master: f9db41b buildsystem: Make CPU microcode updating more configurable

gerrit at coreboot.org gerrit at coreboot.org
Wed Sep 5 03:40:47 CEST 2012


the following patch was just integrated into master:
commit f9db41bf4575235910a6488d840127c2be7b449f
Author: Alexandru Gagniuc <mr.nuke.me at gmail.com>
Date:   Fri Jul 20 00:11:21 2012 -0500

    buildsystem: Make CPU microcode updating more configurable
    
    This patch aims to improve the microcode in CBFS handling that was
    brought by the last patches from Stefan and the Chromium team.
    
    Choices in Kconfig
      - 1) Generate microcode from tree (default)
      - 2) Include external microcode file
      - 3) Do not put microcode in CBFS
    
    The idea is to give the user full control over including non-free
    blobs in the final ROM image.
    
    MICROCODE_INCLUDE_PATH Kconfig variable is eliminated. Microcode
    is handled by a special class, cpu_microcode, as such:
    
    cpu_microcode-y += microcode_file.c
    
    MICROCODE_IN_CBFS should, in the future, be eliminated. Right now it is
    needed by intel microcode updating. Once all intel cpus are converted to
    cbfs updating, this variable can go away.
    
    These files are then compiled and assembled into a binary CBFS file.
    The advantage of doing it this way versus the current method is that
      1) The rule is CPU-agnostic
      2) Gives user more control over if and how to include microcode blobs
      3) The rules for building the microcode binary are kept in
       src/cpu/Makefile.inc, and thus would not clobber the other makefiles,
       which are already overloaded and very difficult to navigate.
    
    Change-Id: I38d0c9851691aa112e93031860e94895857ebb76
    Signed-off-by: Alexandru Gagniuc <mr.nuke.me at gmail.com>

Reviewed-By: Patrick Georgi <patrick at georgi-clan.de> at Thu Aug 30 11:27:17 2012, giving +2
Build-Tested: build bot (Jenkins) at Mon Aug 27 09:09:48 2012, giving +1
Reviewed-By: Anton Kochkov <anton.kochkov at gmail.com> at Mon Aug 27 09:13:56 2012, giving +1
Reviewed-By: Alexandru Gagniuc <mr.nuke.me at gmail.com> at Wed Aug 29 02:18:34 2012, giving +2
See http://review.coreboot.org/1245 for details.

-gerrit




More information about the coreboot mailing list