Am Freitag 27 Juni 2008 22:20:27 schrieb Ville Syrjälä:
> On Fri, Jun 27, 2008 at 09:07:13PM +0200, Oliver Neukum wrote:
> > Am Freitag 27 Juni 2008 18:10:36 schrieb Ville Syrjälä:
> > > On Fri, Jun 27, 2008 at 10:50:00AM +0200, Oliver Neukum wrote:
> > > > @@ -820,20 +824,34 @@ exit:
> > > > static int usbtouch_open(struct input_dev *input)
> > > > {
> > > > struct usbtouch_usb *usbtouch = input_get_drvdata(input);
> > > > + int rv;
> > > >
> > > > + rv = usb_autopm_get_interface(usbtouch->intf);
> > > > + if (rv < 0)
> > > > + goto bail;
> > > > + mutex_lock(&usbtouch->lock);
> > > > usbtouch->irq->dev = usbtouch->udev;
> > > >
> > > > - if (usb_submit_urb(usbtouch->irq, GFP_KERNEL))
> > > > - return -EIO;
> > > > -
> > > > - return 0;
> > > > + if (usb_submit_urb(usbtouch->irq, GFP_KERNEL)) {
> > > > + rv = -EIO;
> > > > + usb_autopm_put_interface(usbtouch->intf);
> > > > + } else {
> > > > + usbtouch->open = 1;
> > > > + }
> > > > + mutex_unlock(&usbtouch->lock);
> > > > +bail:
> > > > + return rv;
> > > > }
> > >
> > > What if the device is already suspended when open() is called?
> > >
> >
> > It cannot be. usb_autopm_get_interface() is called, which will resume
> > the device.
>
> Ah, so it resumes the device even if it was not autosuspended but
> suspended for another reason?
The other reason is either system wide, in which case nobody can call open,
or it is forcibly suspended via sysfs, causing usb_submit_urb() to fail later
on.
Regards
Oliver
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
