Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933025AbdDENAS (ORCPT ); Wed, 5 Apr 2017 09:00:18 -0400 Received: from us01smtprelay-2.synopsys.com ([198.182.60.111]:39986 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933126AbdDEM6g (ORCPT ); Wed, 5 Apr 2017 08:58:36 -0400 From: Minas Harutyunyan To: John Stultz , Felipe Balbi CC: Minas Harutyunyan , John Youn , lkml , YongQin Liu , "linux-usb@vger.kernel.org" Subject: Re: dwc2_hc_chhltd_intr_dma - ChHltd set errors? Thread-Topic: dwc2_hc_chhltd_intr_dma - ChHltd set errors? Thread-Index: AQHSk4+c0FmhkyX5hEq/7QuV9W5RZw== Date: Wed, 5 Apr 2017 12:58:24 +0000 Message-ID: <410670D7E743164D87FA6160E7907A56FD9F832E@am04wembxb.internal.synopsys.com> References: <2B3535C5ECE8B5419E3ECBE300772909021B4A47FF@US01WEMBX2.internal.synopsys.com> <410670D7E743164D87FA6160E7907A56FD9F691E@am04wembxb.internal.synopsys.com> <410670D7E743164D87FA6160E7907A56FD9F6F17@am04wembxb.internal.synopsys.com> <87inmkipwc.fsf@linux.intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: yes X-MS-TNEF-Correlator: x-originating-ip: [10.116.70.205] Content-Type: multipart/mixed; boundary="_002_410670D7E743164D87FA6160E7907A56FD9F832Eam04wembxbinter_" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5596 Lines: 126 --_002_410670D7E743164D87FA6160E7907A56FD9F832Eam04wembxbinter_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi,=0A= =0A= On 4/4/2017 11:03 PM, John Stultz wrote:=0A= > On Tue, Apr 4, 2017 at 12:38 AM, Felipe Balbi=0A= > wrote:=0A= >>=0A= >> Hi,=0A= >>=0A= >> Minas Harutyunyan writes:=0A= >>>>>>>> We've noticed that when using usb ethernet adapters on HiKey, we= =0A= >>>>>>>> occasionally see errors like:=0A= >>>>>>>>=0A= >>>>>>>> dwc2 f72c0000.usb: dwc2_hc_chhltd_intr_dma: Channel 0 - ChHltd set= ,=0A= >>>>>>>> but reason is unknown=0A= >>>>>>>> dwc2 f72c0000.usb: hcint 0x00000002, intsts 0x06200029=0A= >>>>>>>>=0A= >>>>>>>> dwc2 f72c0000.usb: dwc2_hc_chhltd_intr_dma: Channel 11 - ChHltd se= t,=0A= >>>>>>>> but reason is unknown=0A= >>>>>>>> dwc2 f72c0000.usb: hcint 0x00000002, intsts 0x04200029=0A= >>>>>>>>=0A= >>>>>>>> Sometimes followed up by a usb error in the driver, something like= :=0A= >>>>>>>> asix 1-1.2:1.0 eth0: asix_rx_fixup() Bad Header Length 0x36000807,= offset 68=0A= >>>>>>>>=0A= >>>>>>>> Curious if you've seen any reports like this?=0A= >>>>>=0A= >>>>> The core version is less than 2.71a, am I right?=0A= >>>>=0A= >>>> So it looks like its reporting 0x4f54300a for hsotg->regs + GSNPSID=0A= >>>> which looks like DWC2_CORE_REV_3_00a=0A= >>>>=0A= >>>>> Please send full debug log to do more investigation.=0A= >>>>=0A= >>>> Full dmesg, or is there special debugging you want me to enable?=0A= >>>=0A= >>> Full dmesg around issue.=0A= >>>>=0A= >>>>> Also send us regdump after connecting ethernet adapter.=0A= >>>>=0A= >>>> Sorry, can you clarify how to generate this?=0A= >>>=0A= >>> cat regdump. To locate dwc2 regdump file: cd /; find -name regdump=0A= >>=0A= >> this won't work if his distro doesn't mount debugfs. Please give=0A= >> complete instructions ;-)=0A= >>=0A= >> # mkdir -p /d=0A= >> # mount -t debugfs none /d=0A= >> # cd /d=0A= >> # find . -name regdump=0A= >>=0A= >> The directory name is the same name as the dwc2 device name, AFAICT. So,= =0A= >> check your DTS for the name of the device.=0A= >=0A= > Thanks for the extra details! I didn't have DEBUG_FS built in, so this=0A= > helped clue me in.=0A= >=0A= > Attached are dmesg including the issue and the regdump.=0A= >=0A= > I did notice when cating the regdump file, I saw:=0A= > dwc2 f72c0000.usb: Mode Mismatch Interrupt: currently in Host mode=0A= > twice. (You'll see it 4 times in the dmesg around 1077 as I cat'ed=0A= > regdump again to verify it wasn't just chance).=0A= >=0A= > Let me know if there is anything else you need!=0A= >=0A= > thanks=0A= > -john=0A= >=0A= =0A= Could you please apply attached patch and try again.=0A= =0A= Thanks,=0A= Minas=0A= --_002_410670D7E743164D87FA6160E7907A56FD9F832Eam04wembxbinter_ Content-Type: text/plain; name="0001-usb-dwc2-hcd-Fix-host-channel-halt-flow.patch" Content-Description: 0001-usb-dwc2-hcd-Fix-host-channel-halt-flow.patch Content-Disposition: attachment; filename="0001-usb-dwc2-hcd-Fix-host-channel-halt-flow.patch"; size=1687; creation-date="Wed, 05 Apr 2017 12:58:24 GMT"; modification-date="Wed, 05 Apr 2017 12:58:24 GMT" Content-Transfer-Encoding: base64 RnJvbSA0MTJiZjdlM2RhZDhhMjBhZjkzN2YzNjgyYTA4MTE2YjY1NGRlYjAyIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpNZXNzYWdlLUlkOiA8NDEyYmY3ZTNkYWQ4YTIwYWY5MzdmMzY4MmEwODEx NmI2NTRkZWIwMi4xNDkxNDEwMjI0LmdpdC5obWluYXNAc3lub3BzeXMuY29tPgpGcm9tOiBNaW5h cyBIYXJ1dHl1bnlhbiA8aG1pbmFzQHN5bm9wc3lzLmNvbT4KRGF0ZTogTW9uLCAxMSBKdWwgMjAx NiAwMzozMToxNyAtMDcwMApTdWJqZWN0OiBbUEFUQ0hdIHVzYjogZHdjMjogaGNkOiBGaXggaG9z dCBjaGFubmVsIGhhbHQgZmxvdwoKQWNjb3JkaW5nIGRhdGFib29rIGluIEJ1ZmZlciBhbmQgRXh0 ZXJuYWwgRE1BIG1vZGUKbm9uLXNwbGl0IHBlcmlvZGljIGNoYW5uZWxzIGNhbid0IGJlIGhhbHRl ZC4KCkNoYW5nZS1JZDogSTdjN2M0ZTE2MzA5ZGRhN2EzYjVhZjM0MDIwZDQ2MzY2YzM1N2VkNDkK U2lnbmVkLW9mZi1ieTogTWluYXMgSGFydXR5dW55YW4gPGhtaW5hc0BzeW5vcHN5cy5jb20+Ci0t LQogZHJpdmVycy91c2IvZHdjMi9oY2QuYyB8IDE5ICsrKysrKysrKysrKysrKysrKysKIDEgZmls ZSBjaGFuZ2VkLCAxOSBpbnNlcnRpb25zKCspCgpkaWZmIC0tZ2l0IGEvZHJpdmVycy91c2IvZHdj Mi9oY2QuYyBiL2RyaXZlcnMvdXNiL2R3YzIvaGNkLmMKaW5kZXggZjRlZjE1OWI1MzhlLi40ZTZl ZTM5OGVmYjcgMTAwNjQ0Ci0tLSBhL2RyaXZlcnMvdXNiL2R3YzIvaGNkLmMKKysrIGIvZHJpdmVy cy91c2IvZHdjMi9oY2QuYwpAQCAtOTc5LDYgKzk3OSwyNSBAQCB2b2lkIGR3YzJfaGNfaGFsdChz dHJ1Y3QgZHdjMl9oc290ZyAqaHNvdGcsIHN0cnVjdCBkd2MyX2hvc3RfY2hhbiAqY2hhbiwKIAog CWlmIChkYmdfaGMoY2hhbikpCiAJCWRldl92ZGJnKGhzb3RnLT5kZXYsICIlcygpXG4iLCBfX2Z1 bmNfXyk7CisKKwkvKgorCSAqIEluIGJ1ZmZlciBETUEgb3IgZXh0ZXJuYWwgRE1BIG1vZGUgY2hh bm5lbCBjYW4ndCBiZSBoYWx0ZWQKKwkgKiBmb3Igbm9uLXNwbGl0IHBlcmlvZGljIGNoYW5uZWxz LiBBdCB0aGUgZW5kIG9mIHRoZSBuZXh0CisJICogdWZyYW1lL2ZyYW1lIChpbiB0aGUgd29yc3Qg Y2FzZSksIHRoZSBjb3JlIGdlbmVyYXRlcyBhIGNoYW5uZWwKKwkgKiBoYWx0ZWQgYW5kIGRpc2Fi bGVzIHRoZSBjaGFubmVsIGF1dG9tYXRpY2FsbHkuCisJICovCisJaWYgKChoc290Zy0+Y29yZV9w YXJhbXMtPmRtYV9lbmFibGUgPiAwICYmCisJICAgICBoc290Zy0+Y29yZV9wYXJhbXMtPmRtYV9k ZXNjX2VuYWJsZSA8PSAwKSB8fAorCSAgICAoaHNvdGctPmh3X3BhcmFtcy5hcmNoID09IEdIV0NG RzJfRVhUX0RNQV9BUkNIKSkgeworCQlpZiAoIWNoYW4tPmRvX3NwbGl0ICYmCisJCSAgICAoY2hh bi0+ZXBfdHlwZSA9PSBVU0JfRU5EUE9JTlRfWEZFUl9JU09DIHx8CisJCSAgICAgY2hhbi0+ZXBf dHlwZSA9PSBVU0JfRU5EUE9JTlRfWEZFUl9JTlQpKSB7CisJCQlkZXZfZXJyKGhzb3RnLT5kZXYs ICIlcygpIENoYW5uZWwgY2FuJ3QgYmUgaGFsdGVkXG4iLAorCQkJCV9fZnVuY19fKTsKKwkJCXJl dHVybjsKKwkJfQorCX0KKwogCWlmIChoYWx0X3N0YXR1cyA9PSBEV0MyX0hDX1hGRVJfTk9fSEFM VF9TVEFUVVMpCiAJCWRldl9lcnIoaHNvdGctPmRldiwgIiEhISBoYWx0X3N0YXR1cyA9ICVkICEh IVxuIiwgaGFsdF9zdGF0dXMpOwogCi0tIAoyLjExLjAKCg== --_002_410670D7E743164D87FA6160E7907A56FD9F832Eam04wembxbinter_--