[coreboot-gerrit] Patch merged into coreboot/master: sandybridge ivybridge: Treat native init as first class citizen

gerrit at coreboot.org gerrit at coreboot.org
Sat Oct 3 22:23:00 CET 2015


the following patch was just integrated into master:
commit ecf2eb463faff04ab6061eb5dfd8da26c5026a97
Author: Alexandru Gagniuc <mr.nuke.me at gmail.com>
Date:   Mon Sep 28 21:39:12 2015 -0700

    sandybridge ivybridge: Treat native init as first class citizen
    
    This is a sad story. We have three different code paths for
    sandybridge and ivybridge: proper native path, google MRC path, and,
    everyone's favorite: Intel FSP path. For the purpose of this patch,
    the FSP path lives in its own little world, and doesn't concern us.
    
    Since MRC was first, when native files and variables were added, they
    were suffixed with "_native" to separate them from the existing code.
    This can cause confusion, as the suffix might make the native files
    seem parasitical.
    
    This has been bothering me for many months. MRC should be the
    parasitical path, especially since we fully support native init, and
    it works more reliably, on a wider range of hardware. There have been
    a few board ports that never made it to coreboot.org because MRC would
    hang.
    
    gigabyte/ga-b75m-d3h is a prime example: it did not work with MRC, so
    the effort was abandoned at first. Once the native path became
    available, the effort was restarted and the board is now supported.
    
    In honor of the hackers and pioneers who made the native code
    possible, rename things so that their effort is the first class
    citizen.
    
    Change-Id: Ic86cee5e00bf7f598716d3d15d1ea81ca673932f
    Signed-off-by: Alexandru Gagniuc <mr.nuke.me at gmail.com>
    Reviewed-on: http://review.coreboot.org/11788
    Tested-by: build bot (Jenkins)
    Reviewed-by: Philipp Deppenwiese <zaolin at das-labor.org>


See http://review.coreboot.org/11788 for details.

-gerrit



More information about the coreboot-gerrit mailing list