[RFC] [PATCH] usbatm.[ch]: logic changes + error handling
rkagan at mail.ru
Sat Apr 16 13:46:40 EDT 2005
On Fri, Apr 15, 2005 at 01:21:22PM +0200, castet.matthieu at free.fr wrote:
> Selon Roman Kagan <rkagan at mail.ru>:
> > OK so here goes another update of my patch to the current CVS, in case
> > it gets somebody interested.
> For the iso part, I think we should get the size of packet of the selected
> alternativate on the endpoint : this will allow the user of useratm to choose
> the right alternative in it's bind without changing the iso packet size.
Do you want it different from how it's done in my patch? There it
initializes the rx_channel.endpoint to a bulk pipe from subdriver's
->in. Then the subdriver in its ->bind() is free to claim the right
interface, choose the appropriate altsetting, and then update
rx_channel.endpoint. Later on usbatm_usb_probe(), in case the endpoint
is an iso pipe, uses usb_maxpacket() on the given pipe to obtain the iso
packet size, and adjusts other values accordingly.
> For errors produced by the modem, I haven't too much time to investigate, but an
> adi developper gave us a documentation  of a part of commands we could send
> to the modem and there is a paragraph were they talk about crc errors in some
Uff it's a 224 page document...
More information about the Usbatm