Skip to content

usb: chipidea: add one errata for revision 2.40a

Sanchayan Maity lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 16 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3usb: chipidea: Add identification registers access APIsdrivers/usb/chipidea/ci.h+53 -06 replies
2/3usb: chipidea: Add chipidea revision informationdrivers/usb/chipidea/bits.h drivers/usb/chipidea/core.c+31 -22 replies
3/3usb: chipidea: Add errata for revision 2.40adrivers/usb/chipidea/udc.c+20 -00 replies

Reviews trailers per patch

None of the 3 patches has a Reviewed-by or Acked-by.

PatchStefan Agner
SeriesT-b
1/3
2/3
3/3

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/3] usb: chipidea: add one errata for revision 2.40a

The first two patches add identification register API's. These can be used to get controller's revision.

The third patch implements an errata for revision 2.40a. Not sure which other SOCs implement this version of the Chipidea core but this fixes the usb client issue observed on Vybrids. The patch was tested on a Toradex Colibri VF61 module with the 3.18 kernel. iperf tests ran for three hours plus, with these patches applied have found the USB client connection to be now reliable.

This patchset is based off on Shawn Guo's for-next branch
https://git.kernel.org/cgit/linux/kernel/git/shawnguo/linux.git/tree/?h=for-next

The credit for the patches and fix goes to Matthieu Castet and Peter Chen. First two patches are by Peter Chen and the third patch which fixed the bug we observed was reported by Matthieu Castet.

The discussion of the problem and the relevant testing details can be found at this link: http://www.spinics.net/lists/linux-usb/msg118544.html

The first version of this patchset originally send by Peter Chen can be found at this link: http://www.spinics.net/lists/linux-usb/msg118753.html

Comments for review are welcome :).
Note: I am going on a vacation so will not be able to reply or do any further tests till Monday. Will attend and take care of any comments/requests for further changes/testing from Tuesday. Apologize for the delay in advance.

Sanchayan Maity (3):
usb: chipidea: Add identification registers access APIs
usb: chipidea: Add chipidea revision information
usb: chipidea: 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  |   20 ++++++++++++++++
 4 files changed, 104 insertions(+), 2 deletions(-)
Signature · 1 line
1.7.9.5