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. Cheers, Mauro -- 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