lkml.org 
[lkml]   [2011]   [Dec]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v2] Input: bcm5974 - Fix USB autosuspend
Date
Am Montag, 19. Dezember 2011, 18:02:03 schrieb Henrik Rydberg:
> static int bcm5974_suspend(struct usb_interface *iface, pm_message_t message)
> @@ -870,6 +884,9 @@ static int bcm5974_probe(struct usb_interface *iface,
> dev->tp_data, dev->cfg.tp_datalen,
> bcm5974_irq_trackpad, dev, 1);
>
> + /* Required for autosuspend */
> + iface->needs_remote_wakeup = 1;
> +
> /* create bcm5974 device */
> usb_make_path(udev, dev->phys, sizeof(dev->phys));
> strlcat(dev->phys, "/input0", sizeof(dev->phys));

It is not nice to set needs_remote_wakeup on suspend() because
it influences the decision of usbcore to suspend at all, in particular
manually set quirks will not be fully honored if you do it this way.
In addition it will never be reset, even as the device is closed.

Please set it in open() and clear it in close()

Regards
Oliver


\
 
 \ /
  Last update: 2011-12-19 18:43    [W:6.388 / U:3.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site