Chuck:<div><br></div><div>Thanks for the information. Sorry that we missed the patch. It was not done out of specific reason.</div><div>As you have commented, it is a very large patch with alot of changes. We wanted to submit</div>
<div>the patch to make sure the fundamental structure of the driver align with the kernel. Once that</div><div>is in place, additional patch will be easier. Fushen will make sure this change is in place on</div><div>the next submission.</div>
<div><br></div><div>Thanks</div><div>Feng Kan</div><div><br><br><div class="gmail_quote">On Tue, Jul 13, 2010 at 3:16 PM, Chuck Meade <span dir="ltr"><<a href="mailto:chuck@theptrgroup.com">chuck@theptrgroup.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">On 07/12/2010 07:16 PM, Fushen Chen wrote:<br>
> The DWC OTG driver module provides the initialization and cleanup<br>
> entry points for the DWC OTG USB driver.<br>
><br>
> Signed-off-by: Fushen Chen <<a href="mailto:fchen@apm.com">fchen@apm.com</a>><br>
> Signed-off-by: Mark Miesfeld <<a href="mailto:mmiesfeld@apm.com">mmiesfeld@apm.com</a>><br>
> ---<br>
<br>
</div>This reply is to the patch series, not just this 1/9 patch section.<br>
<br>
Fushen, why did you pick and choose which fixes to incorporate from the Denx<br>
tree's version of the dwc_otg driver?<br>
<br>
I'm not taking the time here to go through this multipart patch and check that<br>
you incorporated every fix, but I *did* randomly pick one fix that I made to that<br>
driver, to see if you incorporated it, and it appears you did not.<br>
I would have expected that you would have incorporated the fixes that were made<br>
to this driver in the Denx tree.<br>
<br>
The one that I checked is in the data toggle error interrupt handling, in<br>
handle_hc_chhltd_intr_dma() (see your 5/9 email in this patch series).  It looks<br>
like you left out the fix I made to this logic that averts an interrupt storm.<br>
<br>
I assume that since I checked one particular fix, and it was missing from your<br>
patch series, that there are likely more fixes you omitted.  Can you explain why<br>
you would leave this out, after Stefan asked you to incorporate the code changes<br>
made in the Denx tree's version of the driver?<br>
<font color="#888888"><br>
Chuck<br>
</font><div><div></div><div class="h5">_______________________________________________<br>
Linuxppc-dev mailing list<br>
<a href="mailto:Linuxppc-dev@lists.ozlabs.org">Linuxppc-dev@lists.ozlabs.org</a><br>
<a href="https://lists.ozlabs.org/listinfo/linuxppc-dev" target="_blank">https://lists.ozlabs.org/listinfo/linuxppc-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Feng Kan<br>
</div>