Oliver Neukum <[email protected]> wrote:

>Hi,
>
>I am no longer sure about my test procedure for cdc-wdm.
>Do you have a test script?

No, I don't. I probably should have a better system, but I usually just try 
some operations I consider "normal usage".

>I am also including what I currently have for your review.

Looks fine to me. I am not able to test it right now, as I am on vacation as 
usual :) This time skiing in the mountains. 

Please submit it if it works for you, and I will test when I am back next week. 

Bjørn


>@@ -478,6 +493,7 @@ retry:
>                       spin_unlock_irq(&desc->iuspin);
>                       goto retry;
>               }
>+
>               if (!desc->reslength) { /* zero length read */
>                       dev_dbg(&desc->intf->dev, "%s: zero length - clearing 
> WDM_READ\n",
>__func__);
>                       clear_bit(WDM_READ, &desc->flags);

This looks spurious though. You may want to drop that line.






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