On Wed, 24 May 2006, Franck Bui-Huu wrote:

> and use completion timeout instead. It also put the task
> in interruptible state instead of an uninterruptible one
> while waiting for the completion.

Don't make the email content a continuation of the subject line.  When
your patch is accepted the content will become a changelog entry, so it
should be self-contained.

> +     clear_bit(US_FLIDX_URB_ACTIVE, &us->flags);
> +
> +     if (timeleft <= 0) {
> +             US_DEBUGP("%s -- cancelling URB\n",
> +                       timeout == 0 ? "Timeout" : "Signal");

Shouldn't the test on the last line be "timeleft == 0"?

Alan Stern



-------------------------------------------------------
All the advantages of Linux Managed Hosting--Without the Cost and Risk!
Fully trained technicians. The highest number of Red Hat certifications in
the hosting industry. Fanatical Support. Click to learn more
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=107521&bid=248729&dat=121642
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to