Re: [PATCH] USB:add RESET_RESUME for webcams shown to be quirky

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am Samstag, 27. August 2011, 22:16:00 schrieb Alan Stern:
> On Sat, 27 Aug 2011, Oliver Neukum wrote:
> 
> > From 6005022e0cd3cfaf0826c9784bea584c63d30498 Mon Sep 17 00:00:00 2001
> > From: Oliver Neukum <oliver@xxxxxxxxxx>
> > Date: Sat, 27 Aug 2011 19:59:17 +0200
> > Subject: [PATCH] USB:add RESET_RESUME for webcams shown to be quirky
> > 
> > The new runtime PM code has shown that many webcams suffer
> > from a race condition that may crash them upon resume.
> > Runtime PM is especially prone to show the problem because
> > it retains power to the cameras at all times. However
> > system suspension may also crash the devices and retain
> > power to the devices.
> > The only way to solve this problem without races is in
> > usbcore with the RESET_RESUME quirk.
> 
> Has this been discussed on the linux-media list or with the webcam 
> maintainers?

No. As far as I can tell, the problem arises in the audio parts of the cameras.

> Is this related in any way to Bugzilla #40352?

Again, no. It is an audio problem. That said, of course RESET_RESUME
might also fix issues with video devices acting up.
Though it lacks a certain ellegance.

> > @@ -65,6 +83,9 @@ static const struct usb_device_id usb_quirk_list[] = {
> >  	{ USB_DEVICE(0x0638, 0x0a13), .driver_info =
> >  	  USB_QUIRK_STRING_FETCH_255 },
> >  
> > +	/* Guillemot Webcam Hercules Dualpix Exchange*/
> > +	{ USB_DEVICE(0x06f8, 0x0804), .driver_info = USB_QUIRK_RESET_RESUME },
> > +
> >  	/* Saitek Cyborg Gold Joystick */
> >  	{ USB_DEVICE(0x06a3, 0x0006), .driver_info =
> >  			USB_QUIRK_CONFIG_INTF_STRINGS },
> 
> Come on, Oliver!  0x06f8 > 0x06a3.

OK, I am officially dyslexic.

	Regards
		Oliver
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux