[coreboot-gerrit] Patch merged into coreboot/master: edid: Don't half parse (and wrongly print) more detailed timings

gerrit at coreboot.org gerrit at coreboot.org
Thu Nov 19 21:37:45 CET 2015


the following patch was just integrated into master:
commit bca67fb7dc02a6342f6a19738bfd25da21369057
Author: Douglas Anderson <dianders at chromium.org>
Date:   Wed Oct 28 09:18:28 2015 -0700

    edid: Don't half parse (and wrongly print) more detailed timings
    
    The EDID parsing code continued to update _some_ fields of the output
    edid but not others if "did_detailed_timing" was already set.  It also
    then went on to print out this halfway mix of modes each time, despite
    the fact that it didn't really update everything.
    
    Let's fix that.  We'll reduce code changes by using a temporary copy of
    data in detailed_block() and then we'll copy it back if we decide we
    should update.
    
    BRANCH=none
    BUG=chrome-os-partner:46998
    TEST=No more bogus printouts
    
    Change-Id: Idbfa233e0997244c22ef21c892c4473a91621821
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 4d69999cdd7ce3cd2c9332ab3f22ea8eb4b6f2e9
    Original-Change-Id: Ia72cac7fda2772f26477e43237678fa30feca584
    Original-Signed-off-by: Douglas Anderson <dianders at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/309541
    Original-Reviewed-on: https://chromium-review.googlesource.com/309609
    Original-Commit-Ready: David Hendricks <dhendrix at chromium.org>
    Original-Tested-by: David Hendricks <dhendrix at chromium.org>
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
    Reviewed-on: http://review.coreboot.org/12444
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list