On Wed, Mar 08, 2023 at 11:43:09PM +0100, Ricardo Ribalda wrote: > On Wed, 1 Mar 2023 at 10:04, Ricardo Ribalda <ribalda@xxxxxxxxxxxx> wrote: > > > > Hi Laurent > > > > We are back to this issue. > > > > > > On Thu, 29 Dec 2022 at 03:22, Laurent Pinchart > > <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote: > > > > > > Hi Ricardo, > > > > > > Thank you for the patch. > > > > > > On Fri, Dec 02, 2022 at 05:48:52PM +0100, Ricardo Ribalda wrote: > > > > When the device suspends, it keeps power-cycling. > > > > > > > > The user notices it because the LED constanct oscillate between > > > > blue (ready) and no LED (off). > > > > Signed-off-by: Ricardo Ribalda <ribalda@xxxxxxxxxxxx> > > > > --- > > > > media: uvcvideo: Disable autosuspend for Insta360 > > > > > > > > The device does not handle properly the USB suspend and makes it barely usable. > > > > > > Isn't this best handled with a quirk in the USB core ? Autosuspend is a > > > device feature, not an interface feature, so if the USB sound driver is > > > loaded but uvcvideo isn't, the kernel may still attempt to autosuspend > > > the device. > > > > > > > Seems like USB_QUIRK_NO_AUTOSUSPEND was gone for a long time > > > > https://lore.kernel.org/lkml/20071115064457.GU19218@xxxxxxxxx/ > > > > under the assumption that autosuspend was off by default and user > > space should only enable autosuspend on the devices that support it > > (if I understand it correctly). > > > > There are two other quirks still available: USB_QUIRK_RESET_RESUME and > > USB_QUIRK_DISCONNECT_SUSPEND, but they do not seem to work for this > > device (Yunke, thanks for looking into this) > > > > If we are explicitly enabling autosuspend on the driver, shouldn't we > > make sure that the device supports it? > > > > Alan, Greg, any idea about what is the best way to proceed here from a > USB perspective? How is autosuspend getting enabled for this device? It is disabled by default for non-hub USB devices. If the uvcvideo or USB sound driver is enabling autosuspend, the driver should be fixed. Perhaps by adding a quirk bit for this purpose. If userspace is enabling autosuspend, then any misbehavior isn't the kernel's fault. :-) Alan Stern
