Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934151AbdDFHmv convert rfc822-to-8bit (ORCPT ); Thu, 6 Apr 2017 03:42:51 -0400 Received: from us01smtprelay-2.synopsys.com ([198.182.47.9]:45194 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932973AbdDFHml (ORCPT ); Thu, 6 Apr 2017 03:42:41 -0400 From: Minas Harutyunyan To: John Stultz , Minas Harutyunyan CC: Felipe Balbi , 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: Thu, 6 Apr 2017 07:36:53 +0000 Message-ID: <410670D7E743164D87FA6160E7907A56FD9F8F15@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> <410670D7E743164D87FA6160E7907A56FD9F832E@am04wembxb.internal.synopsys.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.116.70.205] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2004 Lines: 55 On 4/6/2017 1:03 AM, John Stultz wrote: > > > On Wed, Apr 5, 2017 at 5:58 AM, Minas Harutyunyan > > > wrote: >> On 4/4/2017 11:03 PM, John Stultz wrote: >>> >>> I did notice when cating the regdump file, I saw: >>> dwc2 f72c0000.usb: Mode Mismatch Interrupt: currently in Host mode >>> twice. (You'll see it 4 times in the dmesg around 1077 as I cat'ed >>> regdump again to verify it wasn't just chance). >>> >>> Let me know if there is anything else you need! >>> >> >> Could you please apply attached patch and try again. > > Thanks for sending this out! > > So it didn't build against mainline, but I tweaked it a bit: > - if ((hsotg->core_params->dma_enable > 0 && > - hsotg->core_params->dma_desc_enable <= 0) || > + if ((hsotg->params.host_dma > 0 && > + hsotg->params.dma_desc_enable <= 0) || > > > But I'm still seeing similar behavior: > [ 91.517417] dwc2 f72c0000.usb: dwc2_hc_chhltd_intr_dma: Channel 5 - > ChHltd set, but reason is unknown > [ 91.526693] dwc2 f72c0000.usb: hcint 0x00000002, intsts 0x04000029 > [ 91.533613] asix 1-1.2:1.0 eth0: asix_rx_fixup() Bad Header Length > 0x8003a0cc, offset 1302 > [ 91.534102] asix 1-1.2:1.0 eth0: asix_rx_fixup() Bad Header Length > 0x73ff5a7d, offset 4 > [ 169.116866] dwc2 f72c0000.usb: dwc2_hc_chhltd_intr_dma: Channel 1 - > ChHltd set, but reason is unknown > [ 169.126146] dwc2 f72c0000.usb: hcint 0x00000002, intsts 0x06200029 > [ 170.699334] asix 1-1.2:1.0 eth0: asix_rx_fixup() Bad Header Length > 0x36000807, offset 68 > > And I'm not seeing the "Channel can't be halted" error from the new logic. > > Full dmesg and regdump attached. > > Let me know if there is something else I should try. > > thanks > -john Ok. To enable full dwc2 debug messages, please set USB_DWC2_DEBUG and USB_DWC2_VERBOSE in Kernel configuration file. Also provide the topology of connected devices(class, speed) to dwc2 root hub. Thanks, Minas