diff options
author | Julius Werner <jwerner@chromium.org> | 2017-03-09 14:14:07 -0800 |
---|---|---|
committer | Julius Werner <jwerner@chromium.org> | 2017-03-15 00:54:23 +0100 |
commit | 5fae8294104ae62353466298fae493e717056a28 (patch) | |
tree | 301a0261ceaabef0d2de224f378f177e250a184d /util/msrtool/COPYING | |
parent | a50ced2eba20a007fa5b486c251c252ad09868cf (diff) |
libpayload: usbhub: Force enumeration of all connected ports on init
We have found a non-compliant USB hub (RealTek RTS 5413) that does not
set a port's Connect Status Change bit on its USB 3.0 half if the port
had already been connected while the hub was being reset. To work around
this bug, this patch adds code to initially request the status of every
port after a hub was enumerated, clear the Connect Status Change bit if
set, and then enumerate the port iff it is currently connected,
regardless of whether the change bit was set. A similar behavior can
also be found in the Linux kernel.
BRANCH=oak
BUG=b:35929438
TEST=Booted Elm with this change, my USB 3.0 sticks enumerate now even
if they had been plugged in since boot.
Change-Id: I8a28252eb94f005f04866d06e4fc61ea265cee89
Signed-off-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: https://review.coreboot.org/18729
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'util/msrtool/COPYING')
0 files changed, 0 insertions, 0 deletions