From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.3 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id AF992ECE58E for ; Mon, 14 Oct 2019 10:03:07 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 762B42168B for ; Mon, 14 Oct 2019 10:03:07 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="U4miVBs5" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731258AbfJNKDE (ORCPT ); Mon, 14 Oct 2019 06:03:04 -0400 Received: from mail-wr1-f67.google.com ([209.85.221.67]:41998 "EHLO mail-wr1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730378AbfJNKDE (ORCPT ); Mon, 14 Oct 2019 06:03:04 -0400 Received: by mail-wr1-f67.google.com with SMTP id n14so18917679wrw.9; Mon, 14 Oct 2019 03:03:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=Ngln3AC9QLkh5gLsYDzuns7ztDutAhXRFp1aOInf5Jc=; b=U4miVBs5EV1+dtU0uxxHmfviusIPRGukon9NHCnlRGMyw8Tj8qbuekNpfalgePiFM8 LgGlnc1SkmylAifrQb4JDUMjwMzbIMWhz5NaRmcrqiaWJ3nTBaOYZyu1XlcLVfUruVxU 6vBAf+fTSyKlqT71aBuvfLxG3FvskNEZofQIvVb9T8iN+swLKL+ExWlByDaN3HtP2vx9 LKmFqB5/5w56z/sA0anQEE/CfPJEQmhQ0Gd82fYCCWYd7S6KRpUNzsghJZhpO/EmJdNS CIG4kUxH9I8c3YvQqHujwmD3whp8woLObSwrJ/iDQlsf7/HAGguvKdFU8DLO0TZUtgqr asAA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=Ngln3AC9QLkh5gLsYDzuns7ztDutAhXRFp1aOInf5Jc=; b=fIYSRn3HVY1QR9B9a739BUPA+qsNYdZX0ZD0clVQEc1JAamtP638OJdOw+D3jBP0bq P2xL1rxC0U199oxf99fZ1PUYyTNdPh6dsrXjAUCSkDj/UT7w2nGpMNMXGik5E9UtgfzY 7MPUr+yz1SnhmtYxiwSsFJljQAGzYHmCeqLy46WSU5cMCfBTj1hHHRxGRSEcr2ff3rgV nm0W0fNu2XHh/J8Ta2+4I9S8bdEle3aTuV2jnHhbrg2pX+rJ4JfUN+4W1qsozEo4m4mB ogOp4zoqX+RCMIuqWJR6EEINKZkrQT6QDZM1g+68Be7ePXFd9azjlDnB8P18ZeEHCevm CqXA== X-Gm-Message-State: APjAAAVP2rTsx1eeYCO1e3OMCcNVlcWD2Z/D9ZsLFg/XNZjAWClDRGSX CxZFL4Sji+Yl3LummulzxzM= X-Google-Smtp-Source: APXvYqwvBzjci18YXroHT3sen3HLefXdQ+pC5l/eE2rPeSrO/4FWoS+bguRI69Ie6dCXzsvO4roSSg== X-Received: by 2002:adf:82ad:: with SMTP id 42mr12109017wrc.209.1571047380209; Mon, 14 Oct 2019 03:03:00 -0700 (PDT) Received: from localhost (p2E5BE2CE.dip0.t-ipconnect.de. [46.91.226.206]) by smtp.gmail.com with ESMTPSA id m18sm27712327wrg.97.2019.10.14.03.02.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 14 Oct 2019 03:02:58 -0700 (PDT) Date: Mon, 14 Oct 2019 12:02:57 +0200 From: Thierry Reding To: Felipe Balbi , Kishon Vijay Abraham I , Greg Kroah-Hartman Cc: Nagarjuna Kristam , jonathanh@nvidia.com, mark.rutland@arm.com, robh+dt@kernel.org, devicetree@vger.kernel.org, linux-tegra@vger.kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [Patch V10 0/8] Tegra XUSB gadget driver support Message-ID: <20191014100257.GB419598@ulmo> References: <1569227152-3030-1-git-send-email-nkristam@nvidia.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="v9Ux+11Zm5mwPlX6" Content-Disposition: inline In-Reply-To: <1569227152-3030-1-git-send-email-nkristam@nvidia.com> User-Agent: Mutt/1.12.2 (2019-09-21) Sender: linux-usb-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-usb@vger.kernel.org --v9Ux+11Zm5mwPlX6 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Sep 23, 2019 at 01:55:44PM +0530, Nagarjuna Kristam wrote: > Patches 1-3 are phy driver changes to add support for device > mode. > Patches 4-7 are changes related to XUSB device mode > controller driver. > Patch 8 is to enable drivers for XUDC support in defconfig >=20 > Test Steps(USB 2.0): > - Enable "USB Gadget precomposed configurations" in defconfig > - Build, flash and boot Jetson TX1 > - Connect Jetson TX1 and Ubuntu device using USB A to Micro B > cable > - After boot on Jetson TX1 terminal usb0 network device should be > enumerated > - Assign static ip to usb0 on Jetson TX1 and corresponding net > device on ubuntu > - Run ping test and transfer test(used scp) to check data transfer > communication Hi Felipe, Kishon, Greg, Patches 1-3 provide new API that is required by patch 7, so I think patches 1, 2, 3, 4 and 7 should probably all go through a single tree to avoid having to model the dependencies using stable branches. Kishon, patches 1-3 have gone through several rounds of review already, but do you have any remaining concerns on them? If not, it'd be great if you could ack them. Felipe and Greg could then pick them up along with patches 4 and 7 into the USB tree. Felipe, Greg, does that sound like a reasonable plan? I should also mention that while waiting for review, Nagarjuna has been able to extend support for the XUDC driver to Tegra186 but has so far been holding back on sending them out so as to not needlessly hold up progress on this series. However, given the interdependencies, I've come to think that it may be preferable to merge everything in one go instead of revisiting this in a couple of weeks. If you guys prefer, Nagarjuna could send out v11 of the series and integrate Tegra186 support. Thanks, Thierry > SS mode is verified by enabling Type A port as peripheral > --- > v10: > * Patches 1,2,4,5,7,8 - No changes. > * Patch 3 has braces added to one single line else condition. > * Patch 6 has update on compatible string as per suggestion from Chunfeng. > --- > v9: > * Patches 1,2,3,4,5 - No changes. > * Patch 6 has update on compatible string as per suggestion from Chunfeng. > * Patch 7 has comment fixes as suggested by Chunfeng. > * Patch 8 has CONFIG_USB_GPIO enabled as module additionally. > --- > v8: > * Patches 1,2,3,4,5,8 - No changes. > * Patch 6 has update on compatible string as per change done in [1]. > * Patch 7 has issue fix, where device mode didnot got enabled after resume > from suspend. > --- > v7: > * Patches 1,2,3,4,5,6,8 - No changes. > * Patch 7 - Comments from Balbi and Chunfun adrresed. > Added COMPILE_TEST in Kconfig and updated dependencies. > --- > v6: > * Patches 1,2,3,7,8 - No changes. > * Patch 4,5,6 - Comments from Rob addressed, updated usb connector driver > compatibility string. > --- > v5: > * Patches 1-3 - Commit subject updated as per inputs from Thierry. > * Patch 4 - Added reg-names used on Tegra210 in the bindings doc > * Enabled xudc driver as module instead of part of kernel in patch 8. > * Patched 5-8 - No changes. > --- > v4: > * patch 1 - no changes. > * corrected companion device search based on inputs from Thierry in patch= 2. > * removed unneeded dev variable and corrected value read in > tegra210_utmi_port_reset function in patch 3. > * dt binding doc and dtb files are corrected for alignments. > Replaced extcon-usb-gpio with usb role switch. > * Added support for USB role switch instead of extcon-usb-gpio and other = minor > comments as suggested by Chunfeng. > * Enabled xudc driver as module instead of part of kernel in patch 8. > --- > V3: > * Rebased patch 1 to top of tree. > * Fixed bug in patch 2, where xudc interrupts dont get generated if USB h= ost > mode fails to probe. Moved fake port detection logic to generic xusb.c.= fake > usb port data is updated based on soc flag need_fake_usb3_port. > * Added extra lines whereever necessary to make code more readable in pat= ch 3 > and 7. > * dt binding doc is corrected for typos and extcon references. Also added > details for clocks and removed xusb_ references to clock and power-doma= in > names and accordingly patch 5 is updated. > * removed avdd-pll-utmip-supply in patch 6, as its now part of padctl dri= ver. > * Patch 8 has no changes. > --- > V2: > * Patches 1-3 are new patches in this series, which splits unified featur= es > patch to speprated features and removes need of port-fake entry in DT. > * Patch 4 is re-arragend dt-bindings patch which incorporates previous > patch comments to sort DT entries alphabetically, addresses name changes > and PM domain details added. > * Patch 5-6 are re-arranged DT patches with major changes - sort entries > alphabetically, and adds clock names. > * Patch 7 is UDC driver tegra XUSB device mode controller with major > changes - remove un-used module params, lockinng for device_mode flag, > moving un-needed info logs to debug level, making changes feature flag > dependent rather than SOC based macros and other error handling in prob= e. > * Patch 8 has no changes. >=20 > Nagarjuna Kristam (8): > phy: tegra: xusb: Add XUSB dual mode support on Tegra210 > phy: tegra: xusb: Add usb3 port fake support on Tegra210 > phy: tegra: xusb: Add vbus override support on Tegra210 > dt-bindings: usb: Add NVIDIA Tegra XUSB device mode controller binding > arm64: tegra: Add xudc node for Tegra210 > arm64: tegra: Enable xudc on Jetson TX1 > usb: gadget: Add UDC driver for tegra XUSB device mode controller > arm64: defconfig: Enable tegra XUDC support >=20 > .../devicetree/bindings/usb/nvidia,tegra-xudc.txt | 110 + > arch/arm64/boot/dts/nvidia/tegra210-p2597.dtsi | 31 +- > arch/arm64/boot/dts/nvidia/tegra210.dtsi | 19 + > arch/arm64/configs/defconfig | 2 + > drivers/phy/tegra/xusb-tegra210.c | 134 +- > drivers/phy/tegra/xusb.c | 87 + > drivers/phy/tegra/xusb.h | 4 + > drivers/usb/gadget/udc/Kconfig | 11 + > drivers/usb/gadget/udc/Makefile | 1 + > drivers/usb/gadget/udc/tegra-xudc.c | 3787 ++++++++++++++= ++++++ > include/linux/phy/tegra/xusb.h | 4 +- > 11 files changed, 4186 insertions(+), 4 deletions(-) > create mode 100644 Documentation/devicetree/bindings/usb/nvidia,tegra-xu= dc.txt > create mode 100644 drivers/usb/gadget/udc/tegra-xudc.c >=20 > --=20 > 2.7.4 >=20 --v9Ux+11Zm5mwPlX6 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAl2kR88ACgkQ3SOs138+ s6HcqBAAi6UX8jUkVwpS+m7EFLB8/o3WWuKoXwy4KUgj8b/RxjJJY1McNFuBE+b7 1bZYWwORUA1v1fttqvXb6IbyMU5pyX9szaIlUw89B/p8hN0zXNdSsXi+d+Zq1mKo 13zp6K4cel9FgTJZFWCRgP8ZfOplg6u3nNwP5JmPo/y8I9/iG4b5y01dsUpUJu5Q eniPgOflNqULclwdL0wcHRfVO2c3IyEE9U/qN+HDpxIY5/S1y9xDpwzjQikQSykN cOl/p2q00+EG/0onZcBGhuzGy9S8I9mHLGQhksmvcZDOIGMGotcwaur5xFGAv4IB 7Ji548LYVfBT2P3vGkwnHJPm849jUHmoogEisHZZopqS3tQEUArIeTV//KQmKQdt XVyYcyq0nzw5KdF0Su+FCmDZUIaXJf1NVy+hsjhk9CyXbTWlEabUV6B5blafpqNF iAcBvToW8OndAMmLXkgkXpcct2QmgA9ic0JQbNtbxw5UQ7MwU5Tj30M+7f86q2K+ z5LyNeIf+X2xsztv+FgeNNNQbtfp0NlpBxRPQ5qFaykgjOzzg1FvljxQpfQ29UCu 4QvfBe2wIClKTwcO8o3wQ+JURKnt5YRXyzljYEkbPSu7MgnB86jsxHueGDm1j1AA kf6rTlSilLLaPMuSFh2vjb3GR/rDEsZEkYmBDhDSIh2dce/DHEg= =CTFt -----END PGP SIGNATURE----- --v9Ux+11Zm5mwPlX6--