On Fri, 22 Feb 2008, Misha Zhilin wrote:

> That's exactly what I did in the patch I've sent you. In addition to
> that, last_status carried -EREMOTEIO instead of 0. But I guess it's a
> matter of preference. Important thing for the fix is to carry an error
> status through all USB's qtds.
> 
> Here is the amended patch:
> 
> 
> [PATCH] USB: ehci: Fixes completion for multi-qtd URB the short read case
> 
> Signed-off-by: Misha Zhilin <[EMAIL PROTECTED]>
> diff -ur a/drivers/usb/host/ehci-q.c b/drivers/usb/host/ehci-q.c
> --- a/drivers/usb/host/ehci-q.c       2008-02-11 00:51:11.000000000 -0500
> +++ b/drivers/usb/host/ehci-q.c       2008-02-22 13:46:43.000000000 -0500
> @@ -315,10 +315,10 @@
>                       if (likely (last->urb != urb)) {
>                               ehci_urb_done(ehci, last->urb, last_status);
>                               count++;
> +                             last_status = -EINPROGRESS;
>                       }
>                       ehci_qtd_free (ehci, last);
>                       last = NULL;
> -                     last_status = -EINPROGRESS;
>               }
>  
>               /* ignore urbs submitted during completions we reported */

Acked-by: Alan Stern <[EMAIL PROTECTED]>

-
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to