On 3/27/2013 2:59 PM, 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.Signed-off-by: Vivek Gautam<gautam.vi...@samsung.com> CC: Vipin Kumar<vipin.ku...@st.com> --- common/usb_hub.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/common/usb_hub.c b/common/usb_hub.c index 0677004..d77f98d 100644 --- a/common/usb_hub.c +++ b/common/usb_hub.c @@ -439,7 +439,6 @@ static int usb_hub_configure(struct usb_device *dev) (portstatus& USB_PORT_STAT_CONNECTION)) break; - mdelay(100); } while (get_timer(start)< CONFIG_SYS_HZ * 10); if (ret< 0)
With this change, we are continuously reading the uhb status. Although this is also OK, but I feel 100 ms delay is better
Still, there is no harm even if this patch is added. So, Reviewed-by: Vipin Kumar <vipin.ku...@st.com> _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
