On Wed, Jul 27, 2011 at 11:23:50AM -0700, Prafulla Wadaskar wrote:
> 
> 
> > -----Original Message-----
> > From: Jason Cooper [mailto:u-b...@lakedaemon.net]
> > Sent: Wednesday, July 27, 2011 2:49 AM
> > To: cl...@debian.org; w...@denx.de; Prafulla Wadaskar
> > Cc: u-boot@lists.denx.de; Prabhanjan Sarnaik; Ashish Karkare; Siddarth
> > Gore; bd...@gag.com; Jason Cooper
> > Subject: RFC [PATCH 3/5] usb: Some EHCI chipsets are slow to respond.
> > 
> > This fixes 'EHCI timed out on TD...' on the dreamplug board.
> > 
> > Signed-off-by: Jason Cooper <u-b...@lakedaemon.net>
> > ---
> >  include/usb.h |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/include/usb.h b/include/usb.h
> > index 53603a5..168e2b2 100644
> > --- a/include/usb.h
> > +++ b/include/usb.h
> > @@ -46,7 +46,7 @@
> >   * This is the timeout to allow for submitting an urb in ms. We allow
> > more
> >   * time for a BULK device to react - some are slow.
> >   */
> > -#define USB_TIMEOUT_MS(pipe) (usb_pipebulk(pipe) ? 5000 : 100)
> > +#define USB_TIMEOUT_MS(pipe) (usb_pipebulk(pipe) ? 5000 : 500)
> 
> This looks global change, how will it affect other implementations?

I dunno, that's why it's RFC. ;-)  It's a timeout, so it *shouldn't*
adversely affect other systems with well behaved usb chipsets.  And
boards will ill-tempered usb chipsets should work better.  But I'm
not familiar enough with the usb code to say that definitively.

I could try doing a dreamplug-specific hack to make it work, but if
there's no adverse affect to other systems, then this is a cleaner
approach.

thx,

Jason.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to