On Tue, Apr 2, 2013 at 11:04 AM, Vivek Gautam gautamvivek1987@gmail.com wrote:
Hi Marek,
On Tue, Apr 2, 2013 at 10:42 AM, Marek Vasut marex@denx.de wrote:
Dear Vivek Gautam,
Hi,
On Tue, Apr 2, 2013 at 1:43 AM, Tom Rini trini@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