Re: kern/46696: uhub disables port where USB keyboard attached

[Matthias Drochner <m.drochner%googlemail.com@localhost>]

The following reply was made to PR kern/46696; it has been noted by GNATS.

From: Matthias Drochner <m.drochner%googlemail.com@localhost>
To: Andreas Gustafsson <gson%gson.org@localhost>
Cc: <gnats-bugs%netbsd.org@localhost>, <mishka%NetBSD.org@localhost>
Subject: Re: kern/46696: uhub disables port where USB keyboard attached
Date: Tue, 18 Sep 2012 09:53:41 +0200

 On Sat, 15 Sep 2012 16:40:05 +0300
 Andreas Gustafsson <gson%gson.org@localhost> wrote:
 > I have now identified the commit where kern/46696 appeared (on my
 > machine, at least) by means of a semi-automated binary search.
 > For testing, I have prepared a patch which effectively reverts this
 > commit:
 > 
 >   http://www.gson.org/netbsd/bugs/46696/usb.patch
 
 I just got a (my first, actually) USB3 device and it triggers an error
 on first descriptor fetch too:
 ehci0: handing over full speed device on port 5 to uhci2
 usbd_new_device: addr=2, getting first desc failed: 17
 uhub_explore: usbd_new_device failed, error=STALLED
 uhub0: device problem, disabling port 1
 
 This happens with and without your patch.
 
 It is also strange that it hands over the device to USB1 at all,
 should better stay on ehci... Need to check specs for compatibility
 requirements.
 
 > Matthias, can you identify the USB device that prompted the above
 > commit?  And if it is a full-speed device, would it be possible
 > to use the 64-byte request size for full-speed devices but not
 > low-speed ones?
 
 I'm not 100% sure. Probably some Atmel-based embedded design.
 This is most certainly a full speed one, so it is better to
 restrict the effect of your patch to low-speed.
 
 But in any case -- as my problem shows, the whole thing
 needs more research.
 
 best regards
 Matthias