[coreboot-gerrit] Patch merged into coreboot/master: intel/skylake: Init variable so GCC knows it's set

gerrit at coreboot.org gerrit at coreboot.org
Fri Jan 15 22:41:18 CET 2016


the following patch was just integrated into master:
commit 19d9fe9ebc81478b9e7074e9ed6fa8f2070f0004
Author: Martin Roth <martinroth at google.com>
Date:   Thu Jan 7 15:35:45 2016 -0700

    intel/skylake: Init variable so GCC knows it's set
    
    Even though the data32 variable was getting written by
    pch_pcr_read(), GCC still flagged it as being used while
    uninitialized and failed the build.
    
    Note that pch_pcr_read() may only set 1 or 2 bytes of data32 in the
    successful path, depending on the size of the read.
    
    Change-Id: Icd6e80d06b9bf4af506d62d55ffe4c5e98634b2b
    Signed-off-by: Martin Roth <martinroth at google.com>
    Reviewed-on: https://review.coreboot.org/12860
    Tested-by: build bot (Jenkins)
    Reviewed-by: Nico Huber <nico.h at gmx.de>
    Reviewed-by: Ben Gardner <gardner.ben at gmail.com>


See https://review.coreboot.org/12860 for details.

-gerrit



More information about the coreboot-gerrit mailing list