[coreboot-gerrit] Patch merged into coreboot/master: skylake: remove the redundant fspNotify in chip final.

gerrit at coreboot.org gerrit at coreboot.org
Wed Jul 29 19:13:39 CEST 2015


the following patch was just integrated into master:
commit bf0902eb83cad554e7bb4993dcfae648d68e821f
Author: robbie zhang <robbie.zhang at intel.com>
Date:   Thu Jul 23 17:31:56 2015 -0700

    skylake: remove the redundant fspNotify in chip final.
    
    The fspNotify(EnumInitPhaseAfterPciEnumeration) is already
    registered in fsp_util.c as a generic callback, this is some code
    left from early development.
    
    Also I don't see a need for the chip_final function, although we
    could keep it as a placeholder but i decided to remove it.
    
    BUG=chrome-os-partner:42979
    BRANCH=None
    TEST=build with current fsp and the coming fsp 1.3.0, boot on sklrvp3.
    
    Change-Id: Ia892f2021be324859c344b4cb8cdeaf75f7ee32f
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: ae22ad57ddbab787da000ae99f85fd2b3d4092e7
    Original-Change-Id: I41be566da71f80451ff70ddd8ada77bf9b8d5b1d
    Original-Signed-off-by: robbie zhang <robbie.zhang at intel.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/287991
    Original-Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
    Reviewed-on: http://review.coreboot.org/11054
    Tested-by: build bot (Jenkins)
    Reviewed-by: Alexandru Gagniuc <mr.nuke.me at gmail.com>


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

-gerrit



More information about the coreboot-gerrit mailing list