* Re: [PATCH] USB: quirks: add NO_LPM quirk for Logitech Rally Camera
[not found] <CAAd214VGLYUSE10AEfk+ujjnXRDr9-HxhtucoLKLVR35vzH3Rw@mail.gmail.com>
@ 2018-12-05 9:35 ` Greg Kroah-Hartman
0 siblings, 0 replies; only message in thread
From: Greg Kroah-Hartman @ 2018-12-05 9:35 UTC (permalink / raw)
To: Kyle Williams
Cc: Kai-Heng Feng, Rajat Jain, Harry Pan, Julius Werner, Daniel Drake,
Maxence Duprès, Dmitry Fleytman Dmitry Fleytman,
Danilo Krummrich, linux-usb, linux-kernel
On Thu, Nov 29, 2018 at 07:59:09AM -0500, Kyle Williams wrote:
> Description: Some USB device / host controller combinations seem to have
> problems with Link Power management. In particular it is described that
> the combination of a Logitech Rally and Atrus device causes
> 'not enough bandwidth for new device state'error
>
> This patch creates quirk entries for the Logitech Rally Camera
> indicating LPM should remain disabled for the device.
>
> Signed-off-by: Kyle Williams <kdgwill@chromium.org>
> Reviewed-on: https://chromium-review.googlesource.com/1297535
> Commit-Ready: Kyle Williams <kdgwill@chromium.org>
> Tested-by: Kyle Williams <kdgwill@chromium.org>
> Reviewed-by: Julius Werner <jwerner@chromium.org>
> ---
> drivers/usb/core/quirks.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/drivers/usb/core/quirks.c b/drivers/usb/core/quirks.c
> index 2deac3a55957..49c44d89a394 100644
> --- a/drivers/usb/core/quirks.c
> +++ b/drivers/usb/core/quirks.c
> @@ -224,6 +224,11 @@ static const struct usb_device_id usb_quirk_list[] = {
> /* Logitech Logitech Screen Share */
> { USB_DEVICE(0x046d, 0x086c), .driver_info = USB_QUIRK_NO_LPM },
>
> + /* Logitech Rally Camera */
> + { USB_DEVICE(0x046d, 0x0881), .driver_info = USB_QUIRK_NO_LPM },
> + { USB_DEVICE(0x046d, 0x0888), .driver_info = USB_QUIRK_NO_LPM },
> + { USB_DEVICE(0x046d, 0x0889), .driver_info = USB_QUIRK_NO_LPM },
> +
> /* Logitech Quickcam Fusion */
> { USB_DEVICE(0x046d, 0x08c1), .driver_info = USB_QUIRK_RESET_RESUME },
>
This patch had all tabs stripped out of it, and was sent in html :(
^ permalink raw reply [flat|nested] only message in thread