On Tue, Apr 2, 2013 at 11:04 AM, Vivek Gautam <gautamvivek1...@gmail.com> wrote: > Hi Marek, > > > On Tue, Apr 2, 2013 at 10:42 AM, Marek Vasut <ma...@denx.de> wrote: >> Dear Vivek Gautam, >> >>> Hi, >>> >>> On Tue, Apr 2, 2013 at 1:43 AM, Tom Rini <tr...@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. > The patches i could see in the 'next' are the originally posted ones. > >> >> Best regards, >> Marek Vasut > > > > -- > Thanks & Regards > Vivek -- Thanks & Regards Vivek _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
