From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752205AbdJXJre convert rfc822-to-8bit (ORCPT ); Tue, 24 Oct 2017 05:47:34 -0400 Received: from smtprelay2.synopsys.com ([198.182.60.111]:52615 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752026AbdJXJra (ORCPT ); Tue, 24 Oct 2017 05:47:30 -0400 From: Minas Harutyunyan To: John Stultz , Minas Harutyunyan CC: John Youn , lkml , Wei Xu , Guodong Xu , "Amit Pundir" , YongQin Liu , Douglas Anderson , Chen Yu , Felipe Balbi , Greg Kroah-Hartman , "linux-usb@vger.kernel.org" Subject: Re: [RESEND x2][PATCH 0/3] dwc2 fixes for edge cases on hikey Thread-Topic: [RESEND x2][PATCH 0/3] dwc2 fixes for edge cases on hikey Thread-Index: AQHTMkq1fSiD2TKttEaGjxxQhxASTg== Date: Tue, 24 Oct 2017 09:47:01 +0000 Message-ID: <410670D7E743164D87FA6160E7907A560113A323E2@am04wembxa.internal.synopsys.com> References: <1505937448-13475-1-git-send-email-john.stultz@linaro.org> <2B3535C5ECE8B5419E3ECBE3007729090269FF70AC@US01WEMBX2.internal.synopsys.com> <410670D7E743164D87FA6160E7907A560113A21CDB@am04wembxa.internal.synopsys.com> <410670D7E743164D87FA6160E7907A560113A2C8E9@am04wembxa.internal.synopsys.com> <410670D7E743164D87FA6160E7907A560113A2E38F@am04wembxa.internal.synopsys.com> <410670D7E743164D87FA6160E7907A560113A2ECC2@am04wembxa.internal.synopsys.com> <410670D7E743164D87FA6160E7907A560113A3036D@am04wembxa.internal.synopsys.com> <410670D7E743164D87FA6160E7907A560113A31032@am04wembxa.internal.synopsys.com> <410670D7E743164D87FA6160E7907A560113A31B67@am04wembxa.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.92] 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 On 10/24/2017 12:41 AM, John Stultz wrote: > On Mon, Oct 23, 2017 at 2:19 AM, Minas Harutyunyan > wrote: >> Could you please verify on your setup follow patches: >> 1. Vardan's patch. >> 2. Patch for TOUTCAL&USBTRDTIM programming (new version see below). >> 4. Your patch 2/3 to avoid "Mode Mismatch" interrupts. >> 5. Your patch 3/3 to set udc state to "not attached". >> 6. Your patch 1/3, but remove dwc2_hsotg_core_init_disconnected() >> function call from Host starting brnch, keep *only* >> dwc2_hsotg_disconnect() to change UDC state to "not attached". > > So yes, this set does seem to work ok for me. Though neither Vardan's > patch or the TOUTCAL/USBTRDTIM patch seem to have much effect either > way (I need to do more testing just to be sure, but for the use cases > I've had trouble with they don't seem to do much). > > I'm happy to rework my earlier patch #1/3 to remove > dwc2_hsotg_core_init_disconnected() and resend. > > thanks > -john > Hi John Stultz, 1. Vardan's patch required for cases when core switching from host mode to device mode. On host disconnect hsotg->lx_state set to DWC2_L2 as result dwc2_hsotg_enqueue_setup() failed because enqueuing should be done in DWC2_L0 state. This patch set lx_state to DWC2_L0 before enqueuing setup transfer. 2. TOUTCAL&USBTRDTIM patch adding missing USBCFG programming in host mode. These fields even if was programmed in device mode (see function dwc2_hsotg_core_init_disconnected()) will be resetting to POR values after core soft reset applied. Programming of these fields to correct values allow fix issues with lot of transaction errors due to timeouts and turnarrounds on USB bus. Previously on TOUTCAL patch you wrote: "So while using this patch and the earlier one from Vardan, I don't see the "Transaction Error" and " ChHltd set, but reason is unknown" messages". Thanks, Minas