2017-12-31 17:13:35

by Leif Liddy

Subject: [PATCH] usb: quirks: Add reset-resume quirk for QCA6174 Rome Bluetooth

This is a rework of reverted commit fd865802c66bc451dc515ed89360f84376ce1a56
The issue is that some QCA Rome bluetooth controllers stop functioning upon resume from suspend.
These devices seem to be losing power during suspend. This patch will enable
reset_resume in usb core (instead of btusb) and will target the specific device 0x0cf3:0xe300

Signed-off-by: Leif Liddy <[email protected]>
---
drivers/usb/core/quirks.c | 3 +++
1 file changed, 3 insertions(+)

diff --git a/drivers/usb/core/quirks.c b/drivers/usb/core/quirks.c
index a10b346b9777..d94e6658d054 100644
--- a/drivers/usb/core/quirks.c
+++ b/drivers/usb/core/quirks.c
@@ -248,6 +248,9 @@ static const struct usb_device_id usb_quirk_list[] = {
/* INTEL VALUE SSD */
{ USB_DEVICE(0x8086, 0xf1a5), .driver_info = USB_QUIRK_RESET_RESUME },

+ /* QCA Rome Bluetooth in QCA6174 wireless module */
+ { USB_DEVICE(0x0cf3, 0xe300), .driver_info = USB_QUIRK_RESET_RESUME },
+
{ } /* terminating entry must be last */
};

--
2.14.3

2017-12-31 20:09:34

by Marcel Holtmann

Subject: Re: [PATCH] usb: quirks: Add reset-resume quirk for QCA6174 Rome Bluetooth

Hi Leif,

> This is a rework of reverted commit fd865802c66bc451dc515ed89360f84376ce1a56
> The issue is that some QCA Rome bluetooth controllers stop functioning upon resume from suspend.
> These devices seem to be losing power during suspend. This patch will enable
> reset_resume in usb core (instead of btusb) and will target the specific device 0x0cf3:0xe300
>
> Signed-off-by: Leif Liddy <[email protected]>
> ---
> drivers/usb/core/quirks.c | 3 +++
> 1 file changed, 3 insertions(+)

patch has been applied to bluetooth-next tree.

Regards

Marcel

2018-01-01 09:51:20

by Sergei Shtylyov

Subject: Re: [PATCH] usb: quirks: Add reset-resume quirk for QCA6174 Rome Bluetooth

Hello!

On 12/31/2017 8:13 PM, Leif Liddy wrote:

> This is a rework of reverted commit fd865802c66bc451dc515ed89360f84376ce1a56

You need to also specify the commit summry enclosed in (""). Alos, 12
digits of SHA1 is enough.

> The issue is that some QCA Rome bluetooth controllers stop functioning upon resume from suspend.
> These devices seem to be losing power during suspend. This patch will enable
> reset_resume in usb core (instead of btusb) and will target the specific device 0x0cf3:0xe300
>
> Signed-off-by: Leif Liddy <[email protected]>
[...]

MBR, Sergei