On Fri, Jun 27, 2008 at 10:50:00AM +0200, Oliver Neukum wrote: > Hi, > > this patch should fix everything you complained about. As I don't have > the hardware, please test. I don't have the hardware either so can't help you there. > static void usbtouch_close(struct input_dev *input) > { > struct usbtouch_usb *usbtouch = input_get_drvdata(input); > > + mutex_lock(&usbtouch->lock); > + usbtouch->open = 0; > + mutex_unlock(&usbtouch->lock); > usb_kill_urb(usbtouch->irq); usb_kill_urb() is still not protected by the mutex but a resume happening here wouldn't resubmit the urb so I suppose it's OK to leave it like that. -- Ville Syrjälä syrjala@xxxxxx http://www.sci.fi/~syrjala/ -- 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