On Mon, 2 Mar 2015, Aleksander Morgado wrote: > On Mon, Mar 2, 2015 at 4:02 PM, Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> wrote: > > On Mon, 2 Mar 2015, Aleksander Morgado wrote: > > > >> I think my patch also lacks the -EREMOTEIO return for the case when 0 > >> bytes are transferred. I'll try to update it today. > > > > It's not necessary. The USB core automatically sets the status to > > -EREMOTEIO if the status is equal to 0 and the transfer was short and > > URB_SHORT_NOT_OK was set. See __usb_hcd_giveback_urb() > > There are actually several places in xhci-ring.c where that check is > done in order to return -EREMOTEIO; does that mean that all of them > are redundant? e.g. not only the ones in process_ctrl_td() but also in > finish_td(). Yes, they are redundant. Alan Stern -- 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