| Message ID | 1320883438-12274-1-git-send-email-yanok@emcraft.com |
|---|---|
| State | Rejected |
| Delegated to: | Marek Vasut |
| Headers | show |
On Wednesday 09 November 2011 19:03:58 Ilya Yanok wrote: > In our setup (OMAP3 EHCI host, keyboard, mouse mass storage and BT > dongle devices connected via several hubs) keyboard fails to initialize > during the bus scan (stalled endpoint while get_descriptor/64) and after > that keyboard is unusable (even in Linux) until power off/on. > > Increasing the delay after port reset from 200ms to 260ms solves this > problem. ugh. this doesn't sound like the right way to fix things. looks like Linux has added a dynamic polling state that automatically deals with broken devices. look at its hub_port_debounce. seems like we should implement something like that instead of delaying longer and longer and making the situation needlessly worse for most systems. -mike
Hi Mike, On 10.11.2011 05:09, Mike Frysinger wrote: > On Wednesday 09 November 2011 19:03:58 Ilya Yanok wrote: >> In our setup (OMAP3 EHCI host, keyboard, mouse mass storage and BT >> dongle devices connected via several hubs) keyboard fails to initialize >> during the bus scan (stalled endpoint while get_descriptor/64) and after >> that keyboard is unusable (even in Linux) until power off/on. >> >> Increasing the delay after port reset from 200ms to 260ms solves this >> problem. > > ugh. this doesn't sound like the right way to fix things. looks like Linux > has added a dynamic polling state that automatically deals with broken > devices. look at its hub_port_debounce. seems like we should implement > something like that instead of delaying longer and longer and making the > situation needlessly worse for most systems. Thanks for the pointer. Unfortunately I don't have time to look into this deeper and we will go with a workaround for now. Maybe we will return to this later. Regards, Ilya.
Dear Ilya Yanok, > In our setup (OMAP3 EHCI host, keyboard, mouse mass storage and BT > dongle devices connected via several hubs) keyboard fails to initialize > during the bus scan (stalled endpoint while get_descriptor/64) and after > that keyboard is unusable (even in Linux) until power off/on. > > Increasing the delay after port reset from 200ms to 260ms solves this > problem. Do we still have this problem? > Signed-off-by: Ilya Yanok <yanok@emcraft.com> > --- > common/usb.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/common/usb.c b/common/usb.c > index bed5116..5e282c6 100644 > --- a/common/usb.c > +++ b/common/usb.c > @@ -1147,7 +1147,7 @@ void usb_hub_port_connect_change(struct usb_device > *dev, int port) return; > } > > - wait_ms(200); > + wait_ms(260); > > /* Allocate a new device struct for it */ > usb = usb_alloc_new_device(); Best regards, Marek Vasut
Hi Marek, Stefano, On 02.04.2012 21:29, Marek Vasut wrote: >> In our setup (OMAP3 EHCI host, keyboard, mouse mass storage and BT >> dongle devices connected via several hubs) keyboard fails to initialize >> during the bus scan (stalled endpoint while get_descriptor/64) and after >> that keyboard is unusable (even in Linux) until power off/on. >> >> Increasing the delay after port reset from 200ms to 260ms solves this >> problem. > > Do we still have this problem? Sorry, I'm not working with this board any more. I'm ccing Stefano, probably he can answer. Regards, Ilya.
On 02/04/2012 21:00, Ilya Yanok wrote: > Hi Marek, Stefano, Hi Marek, > > On 02.04.2012 21:29, Marek Vasut wrote: >>> In our setup (OMAP3 EHCI host, keyboard, mouse mass storage and BT >>> dongle devices connected via several hubs) keyboard fails to initialize >>> during the bus scan (stalled endpoint while get_descriptor/64) and after >>> that keyboard is unusable (even in Linux) until power off/on. >>> >>> Increasing the delay after port reset from 200ms to 260ms solves this >>> problem. >> >> Do we still have this problem? > > Sorry, I'm not working with this board any more. I'm ccing Stefano, > probably he can answer. USB seems not working on this board, but the patch does not help. No peripheral are found after bus scan, even if the delay is increased by the patch. I will suggest to not apply this patch for the release and I will try to investigate what is gone wrong on this board. Best regards, Stefano Babic
Dear Stefano Babic, > On 02/04/2012 21:00, Ilya Yanok wrote: > > Hi Marek, Stefano, > > Hi Marek, > > > On 02.04.2012 21:29, Marek Vasut wrote: > >>> In our setup (OMAP3 EHCI host, keyboard, mouse mass storage and BT > >>> dongle devices connected via several hubs) keyboard fails to initialize > >>> during the bus scan (stalled endpoint while get_descriptor/64) and > >>> after that keyboard is unusable (even in Linux) until power off/on. > >>> > >>> Increasing the delay after port reset from 200ms to 260ms solves this > >>> problem. > >> > >> Do we still have this problem? > > > > Sorry, I'm not working with this board any more. I'm ccing Stefano, > > probably he can answer. > > USB seems not working on this board, but the patch does not help. No > peripheral are found after bus scan, even if the delay is increased by > the patch. I will suggest to not apply this patch for the release and I > will try to investigate what is gone wrong on this board. > > Best regards, > Stefano Babic Ok, discarding from patchwork. Thanks! Best regards, Marek Vasut
diff --git a/common/usb.c b/common/usb.c index bed5116..5e282c6 100644 --- a/common/usb.c +++ b/common/usb.c @@ -1147,7 +1147,7 @@ void usb_hub_port_connect_change(struct usb_device *dev, int port) return; } - wait_ms(200); + wait_ms(260); /* Allocate a new device struct for it */ usb = usb_alloc_new_device();
In our setup (OMAP3 EHCI host, keyboard, mouse mass storage and BT dongle devices connected via several hubs) keyboard fails to initialize during the bus scan (stalled endpoint while get_descriptor/64) and after that keyboard is unusable (even in Linux) until power off/on. Increasing the delay after port reset from 200ms to 260ms solves this problem. Signed-off-by: Ilya Yanok <yanok@emcraft.com> --- common/usb.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)