On 12/19/2014 12:19 PM, Peter Chen wrote: > >> On 12/18/2014 06:47 PM, Matthieu CASTET wrote: >>> Le Wed, 17 Dec 2014 20:40:29 +0800, >>> Peter Chen <peter.chen@xxxxxxxxxxxxx> a écrit : >>> >>>> On Wed, Dec 17, 2014 at 05:31:40PM +0530, Sanchayan Maity wrote: >>>>> On 12/17/2014 02:05 PM, Peter Chen wrote: >>>>>> Hi all, >>>>>> >>>>>> At the first two patches, I add identification registers APIs, we >>>>>> can get controller's revision by using this APIs. Since I don't >>>>>> have enough spec, if there are any errors, tell me. >>>>>> >>>>>> At the third patch, I implement an errata for revision 2.40a, hope >>>>>> it can fix the problem Sanchayan Maity reported. >>>>>> >>>>>> http://www.spinics.net/lists/linux-usb/msg118544.html >>>>>> >>>>>> @Sanchayan, would you please have a test? >>>>>> >>>>>> Peter Chen (3): >>>>>> usb: chipidea: add identification registers access APIs >>>>>> usb: chipidea: add chipidea revision information >>>>>> usb: chipidea: udc: add errata for revision 2.40a >>>>>> >>>>>> drivers/usb/chipidea/bits.h | 10 +++++++++ >>>>>> drivers/usb/chipidea/ci.h | 53 >> +++++++++++++++++++++++++++++++++++++++++++++ >>>>>> drivers/usb/chipidea/core.c | 23 ++++++++++++++++++-- >>>>>> drivers/usb/chipidea/udc.c | 18 ++++++++++++++- >>>>>> 4 files changed, 101 insertions(+), 3 deletions(-) >>>>>> >>>>> >>>>> Thanks for the patches. >>>>> >>>>> I tested the patches with iperf and tried a few times to be sure. >>>>> The condition is still the same. >>>>> >>>> >>>> Oh, bad luck. Be sure the code I added for 2.40a is ran at your board. >>>> >>> >>> For information we workarounded this errata with the attached patched. >>> It is based on a old version of the driver. >>> We never had time to clean and submit the patch, but it solve the >>> errata in our case. >>> >>> >>> Matthieu >>> >> >> I came in the morning and saw this patch. Immediately modified it and applied. >> Glad to say that it works. I had iperf running for an hour with no problem and >> 48.7GBytes of transfer at 116MBits/sec :). Going to run a longer 3 hour iperf >> test now, but, I guess it should be fine. I will spin a combination patch of what >> Peter and Matthieu gave me and try sending it by today evening for me, >> before going on vacation. >> > > Please try to see " mEp->qh.ptr->cap |= QH_ZLT;" in Matthieu's patch is needed or not, > since the QH cap is initialized when the ep is enabled, and it should not be changed. > Yes, it works without it as well. > Peter > >> Thanks a lot Matthieu, Peter and Felipe for the inputs and help. >> >> -Cheers, >> Sanchayan. > - Sanchayan -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
