[coreboot-gerrit] Patch merged into coreboot/master: 1c8ee21 edid: Change static variables to auto variables.

gerrit at coreboot.org gerrit at coreboot.org
Wed Dec 17 04:51:02 CET 2014


the following patch was just integrated into master:
commit 1c8ee21fd31195be6a0473d10b1e7717f6adc5c2
Author: Hung-Te Lin <hungte at chromium.org>
Date:   Thu Apr 17 15:21:37 2014 +0800

    edid: Change static variables to auto variables.
    
    To support parsing multiple EDID blobs, the static "decode results" flags should
    be changed to auto variables inside decode_edid.
    
    This is done by packaging static variables into a structure inside decode_edid.
    We also revised some functions (manufacturer_name, do_checksum) to avoid
    accessing global variables directly. Extension (and detail block) parsing may
    need to access and return all parsed context so we pass the whole structure to
    it.
    
    BRANCH=none
    BUG=none
    TEST=emerge-nyan coreboot chromeos-bootimage
         # See EDID parsed correctly on Nyan.
    
    Original-Change-Id: Ieca93d446bacf655c145dffdfa6cc6f5dc87ac26
    Original-Signed-off-by: Hung-Te Lin <hungte at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/195372
    Original-Reviewed-by: Gabe Black <gabeblack at chromium.org>
    (cherry picked from commit ed45909df24c05a0cb8b2ff662fdd2d7a39012f0)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: I17cdfa770181a6eaac9d1050d340c8e052572b4a
    Reviewed-on: http://review.coreboot.org/7834
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list