Re: USB: change interface to usb_lock_device_for_reset()

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

 



On Fri, 9 Jan 2009, Mauro Carvalho Chehab wrote:
> On Thu, 8 Jan 2009 21:56:15 -0800 (PST)
> Trent Piepho <xyzzy@xxxxxxxxxxxxx> wrote:
> > On Thu, 8 Jan 2009, Mike Isely wrote:
> > > > Yes... Anyway, this is the real patch. I've added a small comment about this
> > > > change... I'll commit this tomorrow, if you don't have a better suggestion.
> > >
> > > Looks good.
> >
> > Or maybe like this?
> >
> > diff -r f01b3897d141 linux/drivers/media/video/pvrusb2/pvrusb2-hdw.c
> > --- a/linux/drivers/media/video/pvrusb2/pvrusb2-hdw.c	Fri Jan 09 00:27:32 2009 -0200
> > +++ b/linux/drivers/media/video/pvrusb2/pvrusb2-hdw.c	Fri Jan 09 02:45:48 2009 -0200
> > @@ -3747,7 +3747,12 @@
> >  	int ret;
> >  	pvr2_trace(PVR2_TRACE_INIT,"Performing a device reset...");
> >  	ret = usb_lock_device_for_reset(hdw->usb_dev,NULL);
> > - 	if (ret == 1) {
> > +#if LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 29)
> > +	/* Due to the API changes, the ret value for success changed */
> > +	ret = ret != 1;
> > +#endif
> > +	if (ret == 0) {
> >  		ret = usb_reset_device(hdw->usb_dev);
> >  		usb_unlock_device(hdw->usb_dev);
> >  	} else {
> >
>
> Seems better! Could you please provide your SOB? I'll apply just the backport, then your patch.

Signed-off-by: Trent Piepho <xyzzy@xxxxxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux