[coreboot-gerrit] Patch merged into coreboot/master: libpayload: usb: xhci: Fix list of cleared port change bits

gerrit at coreboot.org gerrit at coreboot.org
Fri Aug 14 15:18:34 CEST 2015


the following patch was just integrated into master:
commit 6f4c7a6fded85c4687386b857687af4b9a953851
Author: Julius Werner <jwerner at chromium.org>
Date:   Fri Aug 7 15:29:41 2015 -0700

    libpayload: usb: xhci: Fix list of cleared port change bits
    
    The xhci_rh_port_status_changed() function tries to always clear all
    port status bits, even though most of them don't interest us. This is
    generally a smart thing to do since not clearing a status bit may cause
    the controller to not generate any more Port Status Change Events.
    However, the bitmask we currently use doesn't cover bit 23 (Port Config
    Error Change) and instead covers bit 16 (Port Link State Write Strobe)
    which is not really related to this and not a W1C bit. Probably a typo,
    so let's fix that.
    
    BRANCH=None
    BUG=None
    TEST=Plugged/unplugged a bunch of USB devices on an XHCI Falco.
    
    Original-Change-Id: Ia83f5b72cce094859c0f0e730752d7b5cfa6e1c6
    Original-Signed-off-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/291842
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    
    Change-Id: I11f5fe38cb70055daf6e866a8ee84ca80488e3bf
    Signed-off-by: Julius Werner <jwerner at chromium.org>
    Reviewed-on: http://review.coreboot.org/11194
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list