> -----Original Message----- > From: Sergei Shtylyov [mailto:sergei.shtyl...@cogentembedded.com] > Sent: Tuesday, September 22, 2015 9:42 PM > To: Kaukab, Yousaf; linux-usb@vger.kernel.org; ba...@ti.com; > john.y...@synopsys.com; Herrero, Gregory > Cc: he...@sntech.de; diand...@chromium.org; r.bald...@samsung.com; > dingu...@opensource.altera.com; zhangfei....@linaro.org; > david.a.co...@linux.intel.com > Subject: Re: [PATCH v2 14/32] usb: dwc2: host: wait 3ms for controller > stabilization > > On 09/22/2015 04:16 PM, Mian Yousaf Kaukab wrote: > > > From: Gregory Herrero <gregory.herr...@intel.com> > > > > Some high speed mass storage devices fail to enumerate with following > > error: > > > > Cannot enable port %i. Maybe the USB cable is bad? > > > > This happens only when the device is plugged while the controller is > > in hibernation state. After exiting hibernation, the controller > > detects the device as a low speed device and fail to enumerate it. > > > > Problem occurs only if HPRT0.PWR bit is programmed in a too short > > delay after exiting hibernation. Dumping hprt register in > > _dwc2_hcd_resume() directly after dwc2_exit_hibernation() shows that > > HPRT0.LNSTS (D+/D- level) becomes valid approximately 2ms after > > exiting hibernation. > > > Since dwc2_exit_hibernation() is called from atomic context, move the > > delay out of this function. > > Hm, I don't see Gregory moving anything in this patch... > Perhaps keep is more appropriate word. I'll replace move with keep if I end up doing another revision of this series.
BR, Yousaf -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
