[PATCH v2 0/3] usb: implement AMD remote wakeup quirk

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sarah, Alan,

The following patches are required to resolve remote wake issues with
certain devices.

Issue description:
If the remote wake is issued from the device in a specific timing
condition while the system is entering sleep state then it may cause
system to auto wake on subsequent sleep cycle.

Root cause:
Host controller rebroadcasts the Resume signal > 100 µseconds after
receiving the original resume event from the device. For proper
function, some devices may require the rebroadcast of resume event
within the USB spec of 100µS.

Reproduce steps:
1. Enable remote wakeup for usb mouse.
2. Execute S3.
3. Click mouse _intensely_ (more than 10 times) to wake the system up.
4. Then execute S3 again.
5. Observe that the system auto wake up.

[Q] Why the special devices are only mice? Would high speed devices
such as 3G modem or USB Bluetooth adapter trigger this issue?
- Current this sensitivity is only confined to devices that use Pixart
  controllers. This controller is designed for use with LS mouse
devices only. We have not observed any other devices failing. There
may be a small risk for other devices also but this patch (reset
device in resume phase) will cover the cases if required.

[Q] Shouldn’t the resume signal be sent within 100 us for every
device?
- The Host controller may not send the resume signal within 100us,
  this our host controller specification change. This is why we
require the patch to prevent side effects on certain known devices.

[Q] Why would clicking mouse INTENSELY to wake the system up trigger
this issue?
- This behavior is specific to the devices that use Pixart controller.
  It is timing dependent on when the resume event is triggered during
the sleep state.

[Q] Is it a host controller issue or mouse?
- It is the host controller behavior during resume that triggers the
  device incorrect behavior on the next resume.


- Patch 1 is adding is_usb_mouse and quirks to avoid mixing with the
  xHCI changes, which is suggested by Alan.
- Patch 2 is the implementation for xHCI driver.
- Patch 3 is the implementation for OHCI driver.
They are generated on gregkh/usb usb-next.

Changes from v1 -> v2:
- Add reproduce steps.
- Add a patch to put the changes of core/hub.c and pci-quirks into one
  patch.
- Remove all the split strings.
- Refactor usb_amd_remote_wakeup_quirk to save a variable and less
  codes.
- Remove 10ms delay in spinlock during xhci reset port by mouse.
- Add an extra tab to algin to all quirks macros.
- Remove backport request.
- Add descriptions of the fixes and how they work.
- Summarized questions from Sarah and Alan.

Thanks,
Rui


Huang Rui (3):
  usb: add is_usb_mouse routine and remote wakeup quirk
  usb: xhci: implement AMD remote wakeup quirk
  usb: ohci: add AMD remote wakeup quirk into ohci driver

 drivers/usb/core/hub.c        | 27 ++++++++++++++++++++
 drivers/usb/host/ohci-hub.c   | 44 +++++++++++++++++++++++++++++++++
 drivers/usb/host/ohci-pci.c   | 14 +++++++++++
 drivers/usb/host/ohci.h       | 23 ++++++++---------
 drivers/usb/host/pci-quirks.c | 14 ++++++++++-
 drivers/usb/host/pci-quirks.h |  1 +
 drivers/usb/host/xhci-pci.c   |  4 +++
 drivers/usb/host/xhci.c       | 57 +++++++++++++++++++++++++++++++++++++++++++
 drivers/usb/host/xhci.h       | 35 +++++++++++++-------------
 include/linux/usb.h           |  1 +
 10 files changed, 191 insertions(+), 29 deletions(-)

-- 
1.7.11.7


--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux