[coreboot-gerrit] Patch merged into coreboot/master: 16713d7 libpayload: EHCI: Fix transaction error for interrupt transfer

gerrit at coreboot.org gerrit at coreboot.org
Sat Mar 21 01:40:48 CET 2015


the following patch was just integrated into master:
commit 16713d7a23aa8cdf9ff432a881ba9066c9859992
Author: Jim Lin <jilin at nvidia.com>
Date:   Mon Dec 8 19:30:04 2014 +0800

    libpayload: EHCI: Fix transaction error for interrupt transfer
    
    Data toggle should be running like 0, 1, 0, 1, ...
    In the failed case (where a low-speed USB keyboard or km232 device
    is installed), data toggle will be running as 0, 1, 0, 1, ..., 1, 1.
    Therefore causing Halted or Transaction Error bit to be set in qTD
    Status field.
    
    BUG=None
    BRANCH=None
    TEST=Tested on nyan_kitty platform, firmware-kitty-5771.61.B branch.
    Attached USB keyboard or km232 device to root-hub port (same side as
    SD card slot).
    Made sure no transaction error after doing interrupt transfer.
    
    Change-Id: I576f3c583dae4c279a6e0e8ffdfce5abe463277d
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 64b0428aaab869e20f6720669e953acf82ecb846
    Original-Change-Id: Ic2c0f95cff2ae6e314967b0b82231a962255f1a7
    Original-Signed-off-by: Jim Lin <jilin at nvidia.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/233857
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
    Reviewed-on: http://review.coreboot.org/8747
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list