[PATCH] USB: quirks: Ignore wakeup from the Valve Index HMD hub

[PATCH] USB: quirks: Ignore wakeup from the Valve Index HMD hub

[Curtis Vogt]

The Valve Index HMD's onboard USB hub (28de:2613) asserts remote wakeup a
few seconds after the host enters S3, so a system with the headset
attached does not stay suspended. Observed on 7.3-rc2: every suspend
attempt returned after ~6 seconds with wakeup events recorded from the
hub. Disabling wakeup on the hub through sysfs allows the host to stay
asleep until woken from the front panel. The headset most likely reacts to
the DisplayPort link being taken down as the host suspends.

Nothing behind this hub is a device that should be able to wake the host.
The headset includes a radio, microphone, and a downstream Microchip
USB2744 hub.

Link: https://lore.kernel.org/all/20260910170254.833871-1-mario.limonciello@amd.com
Signed-off-by: Curtis Vogt <curtis.vogt@gmail.com>
---
 drivers/usb/core/quirks.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/usb/core/quirks.c b/drivers/usb/core/quirks.c
index b5b577f0b931..7ededf8d9cfd 100644
--- a/drivers/usb/core/quirks.c
+++ b/drivers/usb/core/quirks.c
@@ -596,6 +596,10 @@ static const struct usb_device_id usb_quirk_list[] = {
 	/* ShanWan Wireless Gamepad */
 	{ USB_DEVICE(0x2563, 0x0575), .driver_info = USB_QUIRK_WINDOWS_CONFIG_REQ_SIZE },
 
+	/* Valve Index HMD hub */
+	{ USB_DEVICE(0x28de, 0x2613), .driver_info =
+			USB_QUIRK_IGNORE_REMOTE_WAKEUP },
+
 	/* UGREEN 35871 - BOS descriptor fetch hangs at SuperSpeed Plus */
 	{ USB_DEVICE(0x2b89, 0x5871), .driver_info = USB_QUIRK_NO_BOS },
 
-- 
2.55.0

Re: [PATCH] USB: quirks: Ignore wakeup from the Valve Index HMD hub

[Mario Limonciello]

On 9/23/26 22:41, Curtis Vogt wrote:
> The Valve Index HMD's onboard USB hub (28de:2613) asserts remote wakeup a
> few seconds after the host enters S3, so a system with the headset
> attached does not stay suspended. Observed on 7.3-rc2: every suspend
> attempt returned after ~6 seconds with wakeup events recorded from the
> hub. Disabling wakeup on the hub through sysfs allows the host to stay
> asleep until woken from the front panel. The headset most likely reacts to
> the DisplayPort link being taken down as the host suspends.
> 
> Nothing behind this hub is a device that should be able to wake the host.
> The headset includes a radio, microphone, and a downstream Microchip
> USB2744 hub.
> 
> Link: https://lore.kernel.org/all/20260910170254.833871-1-mario.limonciello@amd.com
> Signed-off-by: Curtis Vogt <curtis.vogt@gmail.com>

I guess this would make sense:

Closes: https://gitlab.freedesktop.org/drm/amd/-/work_items/4333

Thanks for the patch.

Reviewed-by: Mario Limonciello (AMD) <superm1@kernel.org>> ---
>   drivers/usb/core/quirks.c | 4 ++++
>   1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/usb/core/quirks.c b/drivers/usb/core/quirks.c
> index b5b577f0b931..7ededf8d9cfd 100644
> --- a/drivers/usb/core/quirks.c
> +++ b/drivers/usb/core/quirks.c
> @@ -596,6 +596,10 @@ static const struct usb_device_id usb_quirk_list[] = {
>   	/* ShanWan Wireless Gamepad */
>   	{ USB_DEVICE(0x2563, 0x0575), .driver_info = USB_QUIRK_WINDOWS_CONFIG_REQ_SIZE },
>   
> +	/* Valve Index HMD hub */
> +	{ USB_DEVICE(0x28de, 0x2613), .driver_info =
> +			USB_QUIRK_IGNORE_REMOTE_WAKEUP },
> +
>   	/* UGREEN 35871 - BOS descriptor fetch hangs at SuperSpeed Plus */
>   	{ USB_DEVICE(0x2b89, 0x5871), .driver_info = USB_QUIRK_NO_BOS },
>