On Wed, Feb 13, 2002 at 11:30:35PM +0100, Martin Diehl wrote:
> 
> Yes, I do totally agree with David: there is absolutely _no_ debounce
> interval (connect-detect-to-port-reset) implemented in 2.4.18-pre9 hub.c
> According to USB 7.1.7.1 this is mandatory and must be TATTDB > 100 ms.
> It is also shown in fig. 7-19 of the USB 1.1 specs (I'm validating
> vs. 1.1). The mentioned 400ms delay for low speed devices is just were the
> debounce delay should be.

Good catch.  I'll send out a patch in a bit to fix this.

> Btw. another concern wrt. to this code from hub.c is, there are several
> places calling usb_get_port_status passing a pointer to a stack-located
> struct usb_port_status - which becomes the transfer_buffer for the
> usb_control_msg) call. Mapping stack locations for PCI-DMA however is
> invalid. I'm wondering whether we could simply have shared buffer for this
> allocated from khubd instead of spending much effort to alloc/check/free
> such stuff at many places.

Bleah, yes this should be fixed.  Patches _gladly_ accepted for this
problem.

thanks,

greg k-h

_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to