[coreboot-gerrit] Patch merged into coreboot/master: libpayload: xhci: Add delay to get reset working more reliably

gerrit at coreboot.org gerrit at coreboot.org
Thu Nov 5 17:40:33 CET 2015


the following patch was just integrated into master:
commit 1528ffa57cdc8bfbea80ce6d23b9acddef68614e
Author: Rajmohan Mani <rajmohan.mani at intel.com>
Date:   Fri Oct 30 17:00:24 2015 -0700

    libpayload: xhci: Add delay to get reset working more reliably
    
    Existing Intel xHCI controllers require a delay of 1 ms,
    after setting the CMD_RESET bit in command register, before
    accessing any HC registers. This allows the HC to complete
    the reset operation and be ready for HC register access.
    Without this delay, the subsequent HC register access,
    may result in a system hang, very rarely.
    
    Verified CherryView / Braswell platforms go through over
    1000 warm reboot cycles (which was not possible without
    this patch), without any xHCI reset hang in depthcharge.
    
    BRANCH=None
    BUG=None
    TEST=Verified CherryView / Braswell platforms go through
    over 1000 warm reboot cycles, without any xHCI reset hang
    in depthcharge.
    
    Change-Id: I8eff5115ca52738bdcf8bc65fbfb2a5f60a0abe1
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 3e7ea70df36e3bf35a6ee1297640900ee76bfdac
    Original-Change-Id: Id681a19d0eedb0e2c29e259c5467bcde577e3460
    Original-Signed-off-by: Rajmohan Mani <rajmohan.mani at intel.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/310022
    Original-Reviewed-by: Patrick Georgi <pgeorgi at chromium.org>
    Reviewed-on: http://review.coreboot.org/12325
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
    Tested-by: build bot (Jenkins)
    Reviewed-by: Nico Huber <nico.h at gmx.de>


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

-gerrit



More information about the coreboot-gerrit mailing list