[U-Boot] [PATCH 4/7] usb: hub: Fix enumration timeout
Marek Vasut
marex at denx.de
Tue Apr 2 08:43:32 CEST 2013
Dear Vivek Gautam,
> On Tue, Apr 2, 2013 at 11:04 AM, Vivek Gautam <gautamvivek1987 at gmail.com>
wrote:
> > Hi Marek,
> >
> > On Tue, Apr 2, 2013 at 10:42 AM, Marek Vasut <marex at denx.de> wrote:
> >> Dear Vivek Gautam,
> >>
> >>> Hi,
> >>>
> >>> On Tue, Apr 2, 2013 at 1:43 AM, Tom Rini <trini at ti.com> wrote:
> >>> > On Wed, Mar 27, 2013 at 02:59:00PM +0530, Vivek Gautam wrote:
> >>> >> Patch b6d7852c increases timeout for enumeration, taking
> >>> >> worst case to be 10 sec.
> >>> >> get_timer() api returns timestamp in micro-seconds, which is
> >>> >> what we are checking in the do-while() loop in usb_hub_configure()
> >>> >> (get_timer(start) < CONFIG_SYS_HZ * 10).
> >>> >> This should give us a required check for 10 seconds, and thereby
> >>> >> we don't need to add additional mdelay of 100 microseconds in
> >>> >> each cycle.
> >>> >
> >>> > The wording here is not correct. get_timer operates in milliseconds
> >>> > not microseconds.
> >>>
> >>> Oops!! Yes of course, my mistake. Thanks
> >>
> >> I have the patches queued in usb/next, you can send me a fix and I'll
> >> squash it with what's already in there.
> >
> > Sure, i shall send a fix for this.
> > Should it be good actually to respin the complete patch-series with
> > the changes you have suggested ??
>
> I was out on weekend, so couldn't update the patch-series.
No problem, I'll drop it from next then, please send the whole series.
Thanks!
Best regards,
Marek Vasut
More information about the U-Boot
mailing list