On Tuesday 30 June 2009, Greg KH wrote: > > --- a/drivers/usb/musb/musb_host.c > > +++ b/drivers/usb/musb/musb_host.c > > @@ -373,13 +373,13 @@ musb_giveback(struct musb_qh *qh, struct urb *urb, int > > musb_save_toggle(qh, is_in, urb); > > break; > > case USB_ENDPOINT_XFER_ISOC: > > - if (urb->error_count) > > + if (status == 0 && urb->error_count) > > status = -EXDEV; > > break; > > } > > Can you send it in a format that explains the change and it can be > applied, with proper signed-off-by and everything? The intent would seem to be: use the existing status, instead of always reporting -EXDEV for ISO errors. The resend of that one-liner can include my: Acked-by: David Brownell <dbrownell@xxxxxxxxxxxxxxxxxxxxx> ... and I don't recall seeing this fixlet before, but it's possible one of the resends didn't highlight what was different, so I didn't know to swap it in for the other patches. There were a few dozen such patches, and swapping them *all* in/out is just not gonna happen. - Dave -- 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