Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753181AbbGGDZB (ORCPT ); Mon, 6 Jul 2015 23:25:01 -0400 Received: from bear.ext.ti.com ([192.94.94.41]:57168 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752445AbbGGDYw (ORCPT ); Mon, 6 Jul 2015 23:24:52 -0400 Date: Mon, 6 Jul 2015 22:24:48 -0500 From: Felipe Balbi To: John Youn CC: "balbi@ti.com" , Subbaraya Sundeep Bhatta , "gregkh@linuxfoundation.org" , "linux-usb@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "stable@vger.kernel.org" , Subbaraya Sundeep Bhatta Subject: Re: [PATCH v2 3/3] usb: dwc3: gadget: return error if command sent to DEPCMD register fails Message-ID: <20150707032448.GB13135@saruman.tx.rr.com> Reply-To: References: <1432203408-5482-1-git-send-email-sbhatta@xilinx.com> <1432203408-5482-3-git-send-email-sbhatta@xilinx.com> <20150629214701.GK1019@saruman.tx.rr.com> <20150629214854.GL1019@saruman.tx.rr.com> <2B3535C5ECE8B5419E3ECBE3007729090175264F3B@US01WEMBX2.internal.synopsys.com> <20150702030009.GB29462@saruman.tx.rr.com> <2B3535C5ECE8B5419E3ECBE3007729090175267F20@US01WEMBX2.internal.synopsys.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="mxv5cy4qt+RJ9ypb" Content-Disposition: inline In-Reply-To: <2B3535C5ECE8B5419E3ECBE3007729090175267F20@US01WEMBX2.internal.synopsys.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4364 Lines: 127 --mxv5cy4qt+RJ9ypb Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Tue, Jul 07, 2015 at 02:10:26AM +0000, John Youn wrote: > On 7/1/2015 8:00 PM, Felipe Balbi wrote: > > On Thu, Jul 02, 2015 at 02:03:14AM +0000, John Youn wrote: > >> On 6/29/2015 2:48 PM, Felipe Balbi wrote: > >>> Hi again, > >>> > >>> On Mon, Jun 29, 2015 at 04:47:01PM -0500, Felipe Balbi wrote: > >>>> On Thu, May 21, 2015 at 03:46:48PM +0530, Subbaraya Sundeep Bhatta w= rote: > >>>>> We need to return error to caller if command is not sent to > >>>>> controller succesfully. > >>>>> > >>>>> Signed-off-by: Subbaraya Sundeep Bhatta > >>>>> Fixes: 72246da40f37 (usb: Introduce DesignWare USB3 DRD Driver) > >>>>> Cc: > >>>>> --- > >>>>> v2 changes: > >>>>> Added Fixes and Cc in commit message. > >>>> > >>>> I noticed that this breaks at least my AM437x silicon with DWC3 2.40a > >>>> when used with g_zero and testusb. As of now, it could be that silic= on > >>>> is mis-behaving because I got a Transfer Complete before the failing= Set > >>>> Endpoint Transfer Resource command. > >>>> > >>>> In any case, can you run on your setup with g_zero and test.sh/testu= sb > >>>> [1]/[2] just to verify that it really works for you ? > >>>> > >>>> Meanwhile, I'll continue testing on my end. > >>>> > >>>> cheers > >>>> > >>>> [1] https://gitorious.org/usb/usb-tools/source/47ef073d9b6c0eae81620= 4c81374aafb795c6e40:testusb.c > >>>> [2] https://gitorious.org/usb/usb-tools/source/47ef073d9b6c0eae81620= 4c81374aafb795c6e40:test.sh > >>> > >>> Adding John here. John, any chance you could fire up dwc3 on HAPS and > >>> see wether it works or fails for you ? > >>> > >>> cheers > >>> > >> > >> Hi Felipe, > >> > >> Just an update on this. > >> > >> I'm trying to get this working with our latest IP with dwc3 from your > >> testing/next branch. It fails the usbtest with a problem unrelated to > >> this patch. > >> > >> It passes on 4.1.1. > >> > >> I'll have to look into the failure but I won't get to it until next > >> week as I'm off the rest of this week. > >=20 > > interesting... If you could post failure signature, I can help looking > > at it, but I guess it's too late to ask :-) > >=20 > > thanks for helping though > >=20 >=20 >=20 > Hi Felipe, >=20 > Nevermind about my issue, it ended up being a setup-related > problem. >=20 > I actually do see the same error as you due to this series of > patches. Except I see it happening before even the first > iteration. I get a completion status of 1 for the Set Endpoint > Transfer Resources command. I'm not sure why this is. >=20 > I don't see any conflict with any previous Transfer Complete. >=20 > I will have to consult with some hardware engineers tomorrow to > look into it further. cool, thanks. Just let me know if it ends up being something larger, then we can revert that commit for the time being until we come to a conclusion. Thanks a lot for helping with testing. > The trace is attached. thanks a lot :-) Do you mind letting me know which version are you using? 3.00a ? --=20 balbi --mxv5cy4qt+RJ9ypb Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJVm0aAAAoJEIaOsuA1yqREkXkP/1kx23UDvkjkpCq3xqwa46N4 /0fGc2sCqRjQpt5kz1VGnvHPHXGyY8n/+wli1D8k13DuvX8XT7N6UM9mOIbL7B4Y Jn9rbmQBOELeXBahThmKYpBfTcGRL35lucP5A73i9neClZLigNDzyQZw/rynLXX/ G0aA2Z2mGJBsJCmUBBpNoCVKMPSgYCzdY0TsfWnKB+zdaOAcowMI3pzv5AhxwxGL DIl/D54sysVZgK7mYWSJNcHiePWfoWFj4iz02ZpP45yB2Kd0+/oathCJtUtNP3wT TPvR3EkUGMGjF/0uHMr/A1w/qlc8pC26EotTHAUNVXeZ3djAg2A4CznzLd4/EEa5 NGApaFaDx4S3uarK/TgoCh+asISfwYkbnzt3Pusvp4lunyfVOnbDDY3pYZKkW6kg kCF7XYVupc9/nqgTixzmYXiJyeuAfZ1gx2RoLySV26tNs/rFzMC7PcaA0s+V6mdy U81S/SeujFu0fkJCfucgo9nTVJOFMMB6BLT85fylC/FNFOoGGU13Ol7YSJ79CNdj 1s04mdQSIQjLGZNoO3SRGrNNl2xK/H8DfrWI+jItlmgvy6o4eL4DmyG2R5KtJf7y kpzkz0Tl2joCGK/jWibguEGAXx1H3d+A0aDE/yjXyINqOUaPKYMcgcKlZPvdnXEY r1iYnRK32GEE9+27b0K2 =xtYB -----END PGP SIGNATURE----- --mxv5cy4qt+RJ9ypb-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/