Re: [PATCH 2/3] usbtouchscreen: Implement runtime power management

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

 



Am Montag, 7. Juni 2010 18:23:16 schrieb Dmitry Torokhov:
> On Mon, Jun 07, 2010 at 03:13:42PM +0200, Oliver Neukum wrote:
> >  static int usbtouch_open(struct input_dev *input)
> >  {
> >  	struct usbtouch_usb *usbtouch = input_get_drvdata(input);
> > +	int r;
> >  
> >  	usbtouch->irq->dev = interface_to_usbdev(usbtouch->interface);
> >  
> > +	r = usb_autopm_get_interface(usbtouch->interface) ? -EIO : 0;
> 
> Why are we clobbering error code with -EIO? We should propagate te code
> returned to us (here and below).

usb_autopm_get_interface() uses internal USB error codes which with
a few exceptions, do not have the meaning error codes in user space
have.

	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