Re: [linux-usb-devel] [PATCH] visor: Fix Oops on disconnect

Greg KH
2004-05-21T22:32:29Z

On Fri, May 21, 2004 at 06:04:46PM -0400, nardelli wrote:
>
> Maybe I spoke too soon here. We have 1 bulk in, 2 bulk out, and 1 interrupt
> in endpoint, which by the logic in usb-serial, translates to 2 ports. Only
> one of those ports can have a read_urb associated with it, unless we want to
> do some really fancy juggling. This means that we're going to have a port
> that does not have a valid read_urb associated with it, even after open().

But the call to open() fails, which prevents any of the other tty calls
from happening on that port. That's why we don't need to make that
check anywhere else.

> I'm at a loss why this device has an uneven number of bulk in and bulk out
> endpoints.

Stupid hardware engineers? Who really knows...

thanks,

greg k-h