Felipe,<br><br><div class="gmail_quote">On Wed, Sep 19, 2012 at 11:04 AM, Felipe Balbi <span dir="ltr"><<a href="mailto:balbi@ti.com" target="_blank">balbi@ti.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<div><div class="h5"><br>
On Wed, Sep 19, 2012 at 10:02:48AM -0500, Sonasath, Moiz wrote:<br>
> Felipe,<br>
><br>
> On Wed, Sep 19, 2012 at 6:53 AM, Felipe Balbi <<a href="mailto:balbi@ti.com">balbi@ti.com</a>> wrote:<br>
><br>
> > Hi,<br>
> ><br>
> > On Wed, Sep 19, 2012 at 05:00:27PM +0530, Kishon Vijay Abraham I wrote:<br>
> > > From: Moiz Sonasath <<a href="mailto:m-sonasath@ti.com">m-sonasath@ti.com</a>><br>
> > ><br>
> > > For the gadget pullup functionality to work in<br>
> > > SS mode it requires a particular sequence of<br>
> > > toggling the run-stop bit. Here is the required<br>
> > > sequence:<br>
> > ><br>
> > > - Set DCTL[31]<br>
> > > - Clear DCTL[31]<br>
> > > - Clear OMAP5430_CONTROL_CORE__PHY_POWER_USB[14]<br>
> > > - Clear DCTL[8:5] = 0x00<br>
> > > - Set DCTL[8:5] = 0x05<br>
> > > - Wait 25 Ms<br>
> > > - Set DCTL[31]<br>
> > > - Set OMAP5430_CONTROL_CORE__PHY_POWER_USB[14]<br>
> > ><br>
> > > Tested rigourously the gadget pull-up functionality<br>
> > > in bot HS and SS modes.<br>
> > ><br>
> > > Signed-off-by: Moiz Sonasath <<a href="mailto:m-sonasath@ti.com">m-sonasath@ti.com</a>><br>
> > > Signed-off-by: Kishon Vijay Abraham I <<a href="mailto:kishon@ti.com">kishon@ti.com</a>><br>
> ><br>
> > this needs to split into three patches:<br>
> ><br>
> > add new poweron field, implement it on omap-usb3, use it on<br>
> > dwc3/gadget.c<br>
> ><br>
> > btw, I don't think the changes to run_stop bit are necessary and if they<br>
> > are, that'd either be a silicon errata or it would've been mentioned on<br>
> > the databook. I don't remember seeing that on the databook so I'm<br>
> > assuming that this is caused by a bad use of the PHY.<br>
> ><br>
> > Why that mdelay(25) ? why 25 ms ? That's quite a long time, actually.<br>
> ><br>
><br>
> Felipe, This is infact a HW bug that the Si-Val team did accept and gave us<br>
> this workaround sequence with the precise delay :-)<br>
><br>
> Supposedly this will be fixed in ES 2.0.<br>
<br>
</div></div>in that case this doesn't have to go to mainline since we're not<br>
supporting ES1.0 in mainline :-)<br>
<br>
at minimum this should've come with a proper revision check anyway.<br></blockquote><div><br>Actually most of it is under a rev check :) <br><br>Perhaps the last: usb_phy_shutdown(dwc->usb3_phy); <br>in the else part should be in <br>
if (dwc->revision <= DWC3_REVISION_187A) check<br><br>Kishon, can you resend please?<br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<span class="HOEnZb"><font color="#888888"><br>
--<br>
balbi<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>Regards<br>Moiz Sonasath<br>Android Kernel Team<br><br>