Alan Stern wrote:
> 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.
> 

ok

>> +    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"?
> 

sure it should...sorry for that :(

I'm going to send a new one, thanks.

        Franck


-------------------------------------------------------
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