[PATCH] iso pipe support for usbatm

castet.matthieu at free.fr castet.matthieu at free.fr
Fri Mar 25 06:23:06 EST 2005


Selon Roman Kagan <rkagan at mail.ru>:

> On Fri, Mar 25, 2005 at 10:10:28AM +0100, castet.matthieu at free.fr wrote:
> > yes urb->status say that there is at least an error, so you could have 15
> valid
> > frames and a frame with an error.
> > Where do you see that such urb are discarded ?
> > The lenght information is for each frame, and the urb->status is used just
> for
> > tasklet_schedule. And missing tasklet_schedule don't discard them, it just
> > delay the processing and resubmit.
>
> What's the point to delay it then?  I would think that the test for the
> urb status could be used to discard the data in this urb (by setting
> ->filled_cells to zero).  Delaying it until the next urb arrives doesn't
> seem to make any sense.
>
This is what I was trying to said...

So we finally agree that we should always call tasklet_schedule and use the
urb->status to force the filled_cells to 0 in case of errors ?


Matthieu




More information about the Usbatm mailing list