]> git.proxmox.com Git - mirror_qemu.git/commit
usb-hub: clear PORT_STAT_SUSPEND on wakeup
authorLadi Prosek <lprosek@redhat.com>
Thu, 11 May 2017 12:53:14 +0000 (14:53 +0200)
committerGerd Hoffmann <kraxel@redhat.com>
Fri, 12 May 2017 10:26:40 +0000 (12:26 +0200)
commit66849dcfbed4be2ba012fd55631558c623c43f89
tree16d954bd3c27eea3b9d64423f7653a7d16b59910
parentee56264af818dd6c0a97cb9c6e44e5b7d394c0fc
usb-hub: clear PORT_STAT_SUSPEND on wakeup

The spec says:

  Suspend: (PORT_SUSPEND) This field indicates whether or not the device
  on this port is suspended. Setting this field causes the device to
  suspend by not propagating bus traffic downstream. This field may be
  reset by a request or by resume signaling from the device attached to
  the port.

I can't find any specific statement like "the PORT_SUSPEND field is reset
automatically on remote wakeup", but without this patch, the only way to
reset it is via the ClearPortFeature request so the ".. or by resume
signaling from the device" clause is clearly not implemented on the remote
wakeup path.

The default xhci Windows driver does not issue the ClearPortFeature request
and suspended devices attached to a hub don't properly get out of the
suspended state. Interestingly, the default uhci Windows driver *does*
issue the ClearPortFeature request and does not exhibit this problem.

Signed-off-by: Ladi Prosek <lprosek@redhat.com>
Message-id: 20170511125314.24549-3-lprosek@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
hw/usb/dev-hub.c