[coreboot-gerrit] Patch merged into coreboot/master: skylake: fix invalid GNVS base address

gerrit at coreboot.org gerrit at coreboot.org
Fri Aug 14 15:15:44 CEST 2015


the following patch was just integrated into master:
commit 04a066661df161426ab58964ac9e0c84e33e98d1
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Tue Aug 4 15:33:23 2015 -0500

    skylake: fix invalid GNVS base address
    
    Leaving a sentinel 0xC0DEBABE and fixing it up is
    is the old way of setting the correct base address
    for GNVS. One just needs to reference NVSA which is
    already filled in by the skylake ACPI code.
    
    BUG=chrome-os-partner:43611
    BUG=chrome-os-partner:43522
    BRANCH=None
    TEST=Built and booted glados. /sys/firmware/log shows
         up as well as ramoops using the correct address.
    
    Original-Change-Id: I1d4979b1bb65faa76316a4ec4c551a7b9b9eed32
    Original-Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/290338
    Original-Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
    
    Change-Id: I25efea73a383215f9365ce91230f79516b0201a6
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-on: http://review.coreboot.org/11185
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list