[coreboot-gerrit] Patch merged into coreboot/master: rockchip/rk3288: If we fail to read the EDID 5 times in a row, it's an error

gerrit at coreboot.org gerrit at coreboot.org
Thu Oct 29 21:07:25 CET 2015


the following patch was just integrated into master:
commit 6db10452b4ad5d99d160fd6f6c52c4c9a6c30635
Author: Douglas Anderson <dianders at chromium.org>
Date:   Tue Oct 27 16:27:29 2015 -0700

    rockchip/rk3288: If we fail to read the EDID 5 times in a row, it's an error
    
    Previously if we tried to read the HDMI EDID several times and failed
    each time then we're return from hdmi_read_edid() with no error.  Then
    we'd interpret whatever happened to be in memory at the time as an
    EDID--not so great.
    
    Let's actually look at the error.
    
    BRANCH=none
    BUG=chrome-os-partner:46256
    TEST=Monitor that can't read EDID not shows that in the log
    
    Change-Id: I6e64b13ae3f8c61bf1baaa1cfc8b24987bd75cf3
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 44bda7311f9ee677235e4dc8db669226518b3895
    Original-Change-Id: I9089755b75118499bec37bdb96d1635f66252e65
    Original-Signed-off-by: Douglas Anderson <dianders at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/309298
    Original-Commit-Ready: David Hendricks <dhendrix at chromium.org>
    Original-Tested-by: David Hendricks <dhendrix at chromium.org>
    Original-Reviewed-by: David Hendricks <dhendrix at chromium.org>
    Reviewed-on: http://review.coreboot.org/12231
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list