Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp322214imm; Thu, 26 Jul 2018 04:20:33 -0700 (PDT) X-Google-Smtp-Source: AAOMgpeFdHqNxoax8O2Y+FvbYTGsIy92JbUOz+WWl5kz5/TlJ4jow+rr7d8yoQmzdxDd6pHxlTbC X-Received: by 2002:a62:1314:: with SMTP id b20-v6mr1715254pfj.230.1532604033545; Thu, 26 Jul 2018 04:20:33 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1532604033; cv=none; d=google.com; s=arc-20160816; b=dSi9IsO14vpn8MJlXXQ/SvLDliwO1sATUwNX5xQRQ6/xefhm6E7BKXUGknNFCztgFU ZZvldY0zvO197b3vD2tz/UO1ddrAZHeUN+Jlpy51Yj25fEAH6LyZKR0FIZEO7uIJrQsq cow/zJgGfpN7HDBdHyG1q+JJZ6ZhZKodky5ZH+skIhZYbItsnV63ACaFKpclbAGPVdQV z9p7RjECPnPcvjZKjoZUVHodKgApRsY+cGbDm8C7nW48rTvPyBBE+XBzqy7MSxTdR3Tu 5BaZUId1gShyW/uYY8rMRL3GcGrce4qONZJeKfqrra7dpRoOkt2lz3+geydtnrCtC7jL 5O/w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:message-id:date:references :in-reply-to:subject:cc:to:from:arc-authentication-results; bh=9PbOOSgHQeXq5gsrV4Hvv1PtZO9bZqOUdKSD3o9jV2g=; b=NlI7fQbAbNyMEbycb0HHAF4mlnsOsQTpfzed7EEoeN27z1fGflMYTYLKvUyLTWHDbe kHI4pV+1S2PBrLQwhzge7aItoa6+4rg3ygKjXQ9/BYNVzRybPVLltT/lcadHeQUoGJjx XOMKiUBz2WxE4Jix+vaIBkKW9/dXOTK/t0bqRQyPOSgchXR7Gcs8ngwkNfNwk8f4CDX4 tzWMuEuVOgUcpjNHB+WtalouyyVMp3whBhVaNnZ4yxiqF1jPBCH/5pZtFMexXp4iSIbB 2x7pR9Ft0mjoq0Fmuhv4L/Q0YJLqELDH+bj3q8pr0MJItoKU0mapc1d/c9eq1F5Y48nq A25A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id g6-v6si924385plq.242.2018.07.26.04.19.54; Thu, 26 Jul 2018 04:20:33 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729479AbeGZMdw (ORCPT + 99 others); Thu, 26 Jul 2018 08:33:52 -0400 Received: from mga18.intel.com ([134.134.136.126]:32987 "EHLO mga18.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729097AbeGZMdw (ORCPT ); Thu, 26 Jul 2018 08:33:52 -0400 X-Amp-Result: UNSCANNABLE X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 26 Jul 2018 04:17:29 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,404,1526367600"; d="asc'?scan'208";a="57318803" Received: from pipin.fi.intel.com (HELO localhost) ([10.237.68.37]) by fmsmga007.fm.intel.com with ESMTP; 26 Jul 2018 04:17:26 -0700 From: Felipe Balbi To: Anurag Kumar Vulisha , Rob Herring Cc: "gregkh\@linuxfoundation.org" , "mark.rutland\@arm.com" , "v.anuragkumar\@gmail.com" , "linux-usb\@vger.kernel.org" , "devicetree\@vger.kernel.org" , "linux-kernel\@vger.kernel.org" Subject: RE: [PATCH] usb: dwc3: core: Add quirk for enabling AutoRetry feature in the controller In-Reply-To: References: <1532168920-3269-1-git-send-email-anurag.kumar.vulisha@xilinx.com> <20180725200325.GA21007@rob-hp-laptop> Date: Thu, 26 Jul 2018 14:14:09 +0300 Message-ID: <877eli1c72.fsf@linux.intel.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable hi, Anurag Kumar Vulisha writes: > Hi Rob, > > Thanks for your comments, please see my comments inline > >>-----Original Message----- >>From: Rob Herring [mailto:robh@kernel.org] >>Sent: Thursday, July 26, 2018 1:33 AM >>To: Anurag Kumar Vulisha >>Cc: gregkh@linuxfoundation.org; mark.rutland@arm.com; balbi@kernel.org; >>v.anuragkumar@gmail.com; linux-usb@vger.kernel.org; devicetree@vger.kerne= l.org; >>linux-kernel@vger.kernel.org >>Subject: Re: [PATCH] usb: dwc3: core: Add quirk for enabling AutoRetry fe= ature in the >>controller >> >>On Sat, Jul 21, 2018 at 03:58:40PM +0530, Anurag Kumar Vulisha wrote: >>> By default when core sees any transaction error(CRC or overflow) >>> it replies with terminating retry ACK (Retry=3D1 and Nump =3D=3D 0). >>> Enabling this Auto Retry feature in controller, on seeing any >>> transaction errors makes the core to send an non-terminating ACK >>> transaction packet (that is, ACK TP with Retry=3D1 and Nump !=3D 0). >>> Doing so will give controller a chance to recover from the error >>> condition. >>> >>> Signed-off-by: Anurag Kumar Vulisha >>> --- >>> Documentation/devicetree/bindings/usb/dwc3.txt | 5 +++++ >>> drivers/usb/dwc3/core.c | 16 ++++++++++++++++ >>> drivers/usb/dwc3/core.h | 6 ++++++ >>> 3 files changed, 27 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/usb/dwc3.txt >>b/Documentation/devicetree/bindings/usb/dwc3.txt >>> index 7f13ebe..2ba2bc2 100644 >>> --- a/Documentation/devicetree/bindings/usb/dwc3.txt >>> +++ b/Documentation/devicetree/bindings/usb/dwc3.txt >>> @@ -94,6 +94,11 @@ Optional properties: >>> this and tx-thr-num-pkt-prd to a valid, non-zero value >>> 1-16 (DWC_usb31 programming guide section 1.2.3) to >>> enable periodic ESS TX threshold. >>> + - snps,enable_auto_retry: Set to enable Auto retry Feature to make the >> >>s/_/-/ > Sorry, I am not able to understand what needs to fixed here. Please help = me in > understanding, so that I can fix it in v2. replace _ with - >>> + controller operating in Host mode on seeing transaction >>> + errors(CRC errors or internal overrun scenerios) on IN >>> + transfers to reply to the device with a non-terminating >>> + retry ACK (i.e, an ACK TP with Retry=3D1 & Nump !=3D 0) >> >>Seems like the property is unnecessary. When would you not want this >>retry behavior? Why not just enable unconditionally in the driver? >> > There is no harm in adding this fix always but I think this Retry > feature should be added depending on the user and type of the > application. For example, applying this feature in a streaming > application where isochronous transfers are used might end up in read the docs. Auto Retry in only for non-isochronous IN endpoints. You don't need any quirk for this. Just enable it unconditionally. Actually, you wanna make sure the core can run in Host mode before setting this, but that's it. No need for a quirk. =2D-=20 balbi --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEElLzh7wn96CXwjh2IzL64meEamQYFAltZrQEACgkQzL64meEa mQZwtw/9HYBSUy/EQHw0y2JadXUWbjKRD4DwbHHteudftN7ytcGWlKbb9ymhKMSX KhUk3NkEkXxlB13G4KCBKUaaowKkgWRDjZwK4viXGKiI6mp1gl5vI8vh6xtEtUHD YdZv8kU0qJ78bRCKbcP8t0xdr9zjCN5YbzkIPolIYhjDDvVeWhm2bTY7IzcfDxwJ Xtkc/Vox72M62Zk2fHiaRPRai/xDE5NDuzwFJYCIwKMX2PmhB7s+Hd7eNOJLtMga x7DdITFdHoR9eRa4FlMpWpzVSK/oSmNIYAMX30moqiS5TQJ3pD1SE0hFbHYH7MKk D7OqIS6ATrE2Jbt+aj58OupLp1rWOGAtMev47luzz4EEn2YNOGwkvch8yBaLGNL7 f4xQsigwZZdeBUW2waxMQwYNYsJykZFYPi/ndGE6+Kckrl6OMdNXhGM6AO/Sydkd I398o1nAZIZMWTal7qq/KNOCR41Kr6X+zyBCKBy7vywQ9PY6zhu2ds5jc1pLGXOz TkZ1dRpolI0lGVfET1Kp+5SLDzR/2v06VaUVdmRdx2YNMmsQsH/RsCVbgIbPLpFx wMqbZgX5kVT15QmsEm6KpDkTqlCCj3D3kClCfJG5XJakTNUMbyyIPtglEpi59pO2 uekMGftXUMZ/CLblBTOsVt3IR41e5vEFOtNykyo7Puos/ywUMm0= =T4BK -----END PGP SIGNATURE----- --=-=-=--