[coreboot-gerrit] Patch merged into coreboot/master: Kconfig: Remove unnecessary and incorrect MRC_CACHE symbols

gerrit at coreboot.org gerrit at coreboot.org
Sat Jun 27 02:47:44 CEST 2015


the following patch was just integrated into master:
commit 0458370bf7f579966d42c6eb7fa067715b44b393
Author: Martin Roth <gaumless at gmail.com>
Date:   Fri Jun 19 23:39:38 2015 -0600

    Kconfig: Remove unnecessary and incorrect MRC_CACHE symbols
    
    Because of a misunderstanding of how Kconfig files are parsed, the
    OVERRIDE_MRC_CACHE_LOC symbol was added to make sure that the value
    was correctly set.  This is not needed unless for some reason the
    Kconfig parser is suddenly rewritten to parse everything differently.
    
    At some point, the value in the FSP's Kconfig file was updated to
    OVERRIDE_CACHE_CACHE_LOC, while the entries in the mainboard
    Kconfig files were not updated.  This resulted in the default values
    not getting set correctly by default on the FSP Bay Trail boards.
    
    This removes the whole bunch of incorrect and unnecessary symbols and
    just sets the default for the MRC cache location directly.
    
    Change-Id: I1cec758576866b7e0677272b8309bfde8d4a1ee4
    Signed-off-by: Martin Roth <gaumless at gmail.com>
    Reviewed-on: http://review.coreboot.org/10611
    Tested-by: build bot (Jenkins)
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list