[coreboot-gerrit] Patch merged into coreboot/master: google/veyron*: Pulse the i2c clock once if sda was low

gerrit at coreboot.org gerrit at coreboot.org
Wed Nov 18 16:29:26 CET 2015


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

    google/veyron*: Pulse the i2c clock once if sda was low
    
    On one particular TV the TV was holding SDA low when it came up.  It
    would release the SDA when the SCL went low the first time.
    Unfortunately the HDMI i2c port wouldn't transmit until the SDA was
    released.
    
    Let's detect this case and insert a bogus clock pulse to try to get the
    other side to release SDA.
    
    It's unclear why the kernel doesn't have this problem.
    
    BRANCH=none
    BUG=chrome-os-partner:46256
    TEST=Insignia TV works now
    
    Change-Id: Ic9d27eb69bdc9c5fb11a68258e0c755cdc8b79d7
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 356ee7503f04e741a41be37ad573b588067b7114
    Original-Change-Id: I4b6361877e0576cc4ea2f643f073f1aab660e434
    Original-Signed-off-by: Douglas Anderson <dianders at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/309258
    Original-Reviewed-by: Agnes Cheng <agnescheng at google.com>
    Original-Commit-Queue: Agnes Cheng <agnescheng at google.com>
    Original-Trybot-Ready: Agnes Cheng <agnescheng at google.com>
    Original-Tested-by: Agnes Cheng <agnescheng at google.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/309546
    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/12451
    Tested-by: build bot (Jenkins)
    Reviewed-by: Martin Roth <martinroth at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list