From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Google-Smtp-Source: AIpwx4/6nDuo9L6nAZThSyU+pgLqPl5rCDd0MBgYmpPsFVAMZmTeoppds6Kpwe7G4DvdZPz6lvtV ARC-Seal: i=1; a=rsa-sha256; t=1522597789; cv=none; d=google.com; s=arc-20160816; b=JLCTMSDyUjR9dz4IiEoyjGJRbTytMB0aKC/fvXxg3FelfcmBpa0NUoqYcQHqA5PQCa tMczd3atyor22vrV8+TuwshJGmndJupZY6jJXXlAPJp/Kao2vz9TAgo3J/SA3xPhdamJ 8eCtK4rUiGhonD65dkLrWUmP8cDxOgorfR4y27mfBB7w1D3ft52srglpQZEUtzMR/BuG MDbahUJN7Y4qTToHx7iUwgZ2TX1RTkV2q+FcFua/K78RcaSN3a0Jp89tAGjbgyxHXkwx 02aorB0iTp7Yc3qJK4HOBBaAlVsmWekPGYYqyNBPgMJr4IFJ4LdLgJZ0ydKxc5TcBL9K beww== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:organization:references :in-reply-to:date:cc:to:from:subject:message-id :arc-authentication-results; bh=68IPQtkSV68f1brbVwv0HWg6lf98syaotLRwRmf8Dhg=; b=Qz6oSIMDRX7+Kx6LaUn2DPp9SnpmMaSYMLbRyAHF24SIjQh/qgDt0ztb7MjBb4Anib 0mHEl4JOFtoGhh1Nsb88gGDS5nr7vjwjtZWAw7s2VrIJ0qMO2NtNRjbGdhXBvKOAXAu/ k5MQoTDdMTM2lKNKf7rID6YcolRCCrVfmbTiz6mdR/FbcYV+zj/5MrQ9mnyveAg71eJ0 Xr6HWqc86j5leyLKVRSEw4vVfI5gDMJbg7YuhzTD2tzUwKXafQw/p5AHYYPsxEtA9hFB s4iZXErSJMx/t3+w7FNFf/0u8HKZC6bIQl2wd2lSGljD8Kji0K6E0WLjK0FzHNhh+zdy xomA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of ben.hutchings@codethink.co.uk designates 176.9.8.82 as permitted sender) smtp.mailfrom=ben.hutchings@codethink.co.uk; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=codethink.co.uk Authentication-Results: mx.google.com; spf=pass (google.com: domain of ben.hutchings@codethink.co.uk designates 176.9.8.82 as permitted sender) smtp.mailfrom=ben.hutchings@codethink.co.uk; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=codethink.co.uk Message-ID: <1522597785.2654.54.camel@codethink.co.uk> Subject: Re: [PATCH 4.4 068/134] usb: dwc2: Make sure we disconnect the gadget state From: Ben Hutchings To: Greg Kroah-Hartman , linux-kernel@vger.kernel.org Cc: stable@vger.kernel.org, Wei Xu , Guodong Xu , Amit Pundir , Rob Herring , John Youn , Douglas Anderson , Chen Yu , Felipe Balbi , linux-usb@vger.kernel.org, John Stultz , Sasha Levin Date: Sun, 01 Apr 2018 16:49:45 +0100 In-Reply-To: <20180319171859.057406471@linuxfoundation.org> References: <20180319171849.024066323@linuxfoundation.org> <20180319171859.057406471@linuxfoundation.org> Organization: Codethink Ltd. Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.22.6-1+deb9u1 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: =?utf-8?q?1595390862547164468?= X-GMAIL-MSGID: =?utf-8?q?1596559499183213694?= X-Mailing-List: linux-kernel@vger.kernel.org List-ID: On Mon, 2018-03-19 at 19:05 +0100, Greg Kroah-Hartman wrote: > 4.4-stable review patch.  If anyone has any objections, please let me know. > > ------------------ > > From: John Stultz > > > [ Upstream commit dad3f793f20fbb5c0c342f0f5a0bdf69a4d76089 ] Maybe we should also have: commit d2471d4a24dfbff5e463d382e2c6fec7d7e25a09 Author: John Stultz Date:   Mon Oct 23 14:32:48 2017 -0700     usb: dwc2: Improve gadget state disconnection handling Ben. > I had seen some odd behavior with HiKey's usb-gadget interface > that I finally seemed to have chased down. Basically every other > time I plugged in the OTG port, the gadget interface would > properly initialize. The other times, I'd get a big WARN_ON > in dwc2_hsotg_init_fifo() about the fifo_map not being clear. > > Ends up if we don't disconnect the gadget state, the fifo-map > doesn't get cleared properly, which causes WARN_ON messages and > also results in the device not properly being setup as a gadget > every other time the OTG port is connected. > > So this patch adds a call to dwc2_hsotg_disconnect() in the > reset path so the state is properly cleared. > > With it, the gadget interface initializes properly on every > plug in. > > Cc: Wei Xu > Cc: Guodong Xu > Cc: Amit Pundir > Cc: Rob Herring > Cc: John Youn > Cc: Douglas Anderson > Cc: Chen Yu > Cc: Felipe Balbi > Cc: Greg Kroah-Hartman > Cc: linux-usb@vger.kernel.org > Acked-by: John Youn > Signed-off-by: John Stultz > Signed-off-by: Felipe Balbi > Signed-off-by: Sasha Levin > Signed-off-by: Greg Kroah-Hartman > --- >  drivers/usb/dwc2/hcd.c |    1 + >  1 file changed, 1 insertion(+) > > --- a/drivers/usb/dwc2/hcd.c > +++ b/drivers/usb/dwc2/hcd.c > @@ -1385,6 +1385,7 @@ static void dwc2_conn_id_status_change(s >   dwc2_core_init(hsotg, false, -1); >   dwc2_enable_global_interrupts(hsotg); >   spin_lock_irqsave(&hsotg->lock, flags); > + dwc2_hsotg_disconnect(hsotg); >   dwc2_hsotg_core_init_disconnected(hsotg, false); >   spin_unlock_irqrestore(&hsotg->lock, flags); >  dwc2_hsotg_core_connect(hsotg); -- Ben Hutchings Software Developer, Codethink Ltd. From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: [4.4,068/134] usb: dwc2: Make sure we disconnect the gadget state From: Ben Hutchings Message-Id: <1522597785.2654.54.camel@codethink.co.uk> Date: Sun, 01 Apr 2018 16:49:45 +0100 To: Greg Kroah-Hartman , linux-kernel@vger.kernel.org Cc: stable@vger.kernel.org, Wei Xu , Guodong Xu , Amit Pundir , Rob Herring , John Youn , Douglas Anderson , Chen Yu , Felipe Balbi , linux-usb@vger.kernel.org, John Stultz , Sasha Levin List-ID: T24gTW9uLCAyMDE4LTAzLTE5IGF0IDE5OjA1ICswMTAwLCBHcmVnIEtyb2FoLUhhcnRtYW4gd3Jv dGU6Cj4gNC40LXN0YWJsZSByZXZpZXcgcGF0Y2guwqDCoElmIGFueW9uZSBoYXMgYW55IG9iamVj dGlvbnMsIHBsZWFzZSBsZXQgbWUga25vdy4KPiAKPiAtLS0tLS0tLS0tLS0tLS0tLS0KPiAKPiBG cm9tOiBKb2huIFN0dWx0eiA8am9obi5zdHVsdHpAbGluYXJvLm9yZz4KPiAKPiAKPiBbIFVwc3Ry ZWFtIGNvbW1pdCBkYWQzZjc5M2YyMGZiYjVjMGMzNDJmMGY1YTBiZGY2OWE0ZDc2MDg5IF0KCk1h eWJlIHdlIHNob3VsZCBhbHNvIGhhdmU6Cgpjb21taXQgZDI0NzFkNGEyNGRmYmZmNWU0NjNkMzgy ZTJjNmZlYzdkN2UyNWEwOQpBdXRob3I6IEpvaG4gU3R1bHR6IDxqb2huLnN0dWx0ekBsaW5hcm8u b3JnPgpEYXRlOsKgwqDCoE1vbiBPY3QgMjMgMTQ6MzI6NDggMjAxNyAtMDcwMAoKwqDCoMKgwqB1 c2I6IGR3YzI6IEltcHJvdmUgZ2FkZ2V0IHN0YXRlIGRpc2Nvbm5lY3Rpb24gaGFuZGxpbmcKCkJl bi4KCj4gSSBoYWQgc2VlbiBzb21lIG9kZCBiZWhhdmlvciB3aXRoIEhpS2V5J3MgdXNiLWdhZGdl dCBpbnRlcmZhY2UKPiB0aGF0IEkgZmluYWxseSBzZWVtZWQgdG8gaGF2ZSBjaGFzZWQgZG93bi4g QmFzaWNhbGx5IGV2ZXJ5IG90aGVyCj4gdGltZSBJIHBsdWdnZWQgaW4gdGhlIE9URyBwb3J0LCB0 aGUgZ2FkZ2V0IGludGVyZmFjZSB3b3VsZAo+IHByb3Blcmx5IGluaXRpYWxpemUuIFRoZSBvdGhl ciB0aW1lcywgSSdkIGdldCBhIGJpZyBXQVJOX09OCj4gaW4gZHdjMl9oc290Z19pbml0X2ZpZm8o KSBhYm91dCB0aGUgZmlmb19tYXAgbm90IGJlaW5nIGNsZWFyLgo+IAo+IEVuZHMgdXAgaWYgd2Ug ZG9uJ3QgZGlzY29ubmVjdCB0aGUgZ2FkZ2V0IHN0YXRlLCB0aGUgZmlmby1tYXAKPiBkb2Vzbid0 IGdldCBjbGVhcmVkIHByb3Blcmx5LCB3aGljaCBjYXVzZXMgV0FSTl9PTiBtZXNzYWdlcyBhbmQK PiBhbHNvIHJlc3VsdHMgaW4gdGhlIGRldmljZSBub3QgcHJvcGVybHkgYmVpbmcgc2V0dXAgYXMg YSBnYWRnZXQKPiBldmVyeSBvdGhlciB0aW1lIHRoZSBPVEcgcG9ydCBpcyBjb25uZWN0ZWQuCj4g Cj4gU28gdGhpcyBwYXRjaCBhZGRzIGEgY2FsbCB0byBkd2MyX2hzb3RnX2Rpc2Nvbm5lY3QoKSBp biB0aGUKPiByZXNldCBwYXRoIHNvIHRoZSBzdGF0ZSBpcyBwcm9wZXJseSBjbGVhcmVkLgo+IAo+ IFdpdGggaXQsIHRoZSBnYWRnZXQgaW50ZXJmYWNlIGluaXRpYWxpemVzIHByb3Blcmx5IG9uIGV2 ZXJ5Cj4gcGx1ZyBpbi4KPiAKPiBDYzogV2VpIFh1IDx4dXdlaTVAaGlzaWxpY29uLmNvbT4KPiBD YzogR3VvZG9uZyBYdSA8Z3VvZG9uZy54dUBsaW5hcm8ub3JnPgo+IENjOiBBbWl0IFB1bmRpciA8 YW1pdC5wdW5kaXJAbGluYXJvLm9yZz4KPiBDYzogUm9iIEhlcnJpbmcgPHJvYmgrZHRAa2VybmVs Lm9yZz4KPiBDYzogSm9obiBZb3VuIDxqb2hueW91bkBzeW5vcHN5cy5jb20+Cj4gQ2M6IERvdWds YXMgQW5kZXJzb24gPGRpYW5kZXJzQGNocm9taXVtLm9yZz4KPiBDYzogQ2hlbiBZdSA8Y2hlbnl1 NTZAaHVhd2VpLmNvbT4KPiBDYzogRmVsaXBlIEJhbGJpIDxmZWxpcGUuYmFsYmlAbGludXguaW50 ZWwuY29tPgo+IENjOiBHcmVnIEtyb2FoLUhhcnRtYW4gPGdyZWdraEBsaW51eGZvdW5kYXRpb24u b3JnPgo+IENjOiBsaW51eC11c2JAdmdlci5rZXJuZWwub3JnCj4gQWNrZWQtYnk6IEpvaG4gWW91 biA8am9obnlvdW5Ac3lub3BzeXMuY29tPgo+IFNpZ25lZC1vZmYtYnk6IEpvaG4gU3R1bHR6IDxq b2huLnN0dWx0ekBsaW5hcm8ub3JnPgo+IFNpZ25lZC1vZmYtYnk6IEZlbGlwZSBCYWxiaSA8ZmVs aXBlLmJhbGJpQGxpbnV4LmludGVsLmNvbT4KPiBTaWduZWQtb2ZmLWJ5OiBTYXNoYSBMZXZpbiA8 YWxleGFuZGVyLmxldmluQG1pY3Jvc29mdC5jb20+Cj4gU2lnbmVkLW9mZi1ieTogR3JlZyBLcm9h aC1IYXJ0bWFuIDxncmVna2hAbGludXhmb3VuZGF0aW9uLm9yZz4KPiAtLS0KPiDCoGRyaXZlcnMv dXNiL2R3YzIvaGNkLmMgfMKgwqDCoMKgMSArCj4gwqAxIGZpbGUgY2hhbmdlZCwgMSBpbnNlcnRp b24oKykKPiAKPiAtLS0gYS9kcml2ZXJzL3VzYi9kd2MyL2hjZC5jCj4gKysrIGIvZHJpdmVycy91 c2IvZHdjMi9oY2QuYwo+IEBAIC0xMzg1LDYgKzEzODUsNyBAQCBzdGF0aWMgdm9pZCBkd2MyX2Nv bm5faWRfc3RhdHVzX2NoYW5nZShzCj4gwqAJCWR3YzJfY29yZV9pbml0KGhzb3RnLCBmYWxzZSwg LTEpOwo+IMKgCQlkd2MyX2VuYWJsZV9nbG9iYWxfaW50ZXJydXB0cyhoc290Zyk7Cj4gwqAJCXNw aW5fbG9ja19pcnFzYXZlKCZoc290Zy0+bG9jaywgZmxhZ3MpOwo+ICsJCWR3YzJfaHNvdGdfZGlz Y29ubmVjdChoc290Zyk7Cj4gwqAJCWR3YzJfaHNvdGdfY29yZV9pbml0X2Rpc2Nvbm5lY3RlZCho c290ZywgZmFsc2UpOwo+IMKgCQlzcGluX3VubG9ja19pcnFyZXN0b3JlKCZoc290Zy0+bG9jaywg ZmxhZ3MpOwo+wqAJCWR3YzJfaHNvdGdfY29yZV9jb25uZWN0KGhzb3RnKTsK