Received: by 2002:a05:6a10:2785:0:0:0:0 with SMTP id ia5csp198393pxb; Wed, 13 Jan 2021 01:04:30 -0800 (PST) X-Google-Smtp-Source: ABdhPJx5MXopYRK7YYT78uRe0LoyAQG8JPFde2Q1cHgciwEWNbaH10IlowK/G5HaltH/R6LapvMr X-Received: by 2002:a17:906:4a14:: with SMTP id w20mr896605eju.192.1610528670770; Wed, 13 Jan 2021 01:04:30 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1610528670; cv=none; d=google.com; s=arc-20160816; b=tJ+qlCB4YTLyc0XyZ7bw5jff7c72meeo0qa5HBoop7Cd6+qA+yJooL+ZyCv1FTFX2r NjLk2nXXyKVXDtZ2YpnDIRAm7TLX3Lf2YgbZFTA2bSO++hhXUlS2dWH/TZKQ+i7VOuig Vt3nvCaKYAhinOT9EOPPeJcWOux8WPMI8mBhny7NSr4PrFErtm1zb6yFKUOrcHXcb0LX +xUnhthnE2X8xK+HNOwp1q/HS4oKG45dcajBp9QyT4f5pEtj/098EzIvhjkFv5vOI6rN EZHDWQboIY7s2kJjhYnwSdRRiAy9wToYDCFXuOZy4rh9MtWQS2YZUxse62zoCcPZ/GVx MlWA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject:dkim-signature; bh=zkyW24Nem9pNLyI2wkdK3KzUwBFzPFBOJwsBkINN3LQ=; b=UDp7gDJFe+6O4cgpJJ29dnlCUlqLTnWPh75MSAleJ0xVD5Rl7R0nS9/wYyGcJaGZId nxm77DQ2eD/FbHKpYdTcO/pMy/aKrxYxyDNs1eurFu6Alu3YcSu+HOCpYS+hyiUbxgnX 9APPzDBXMg8N5EgM84IuMnasbaZsd5fqu3xo4WovJ6iOv3gi4cJbarIzFN0j5TlAIjIw 967JkClKNwx/STm7onrcM9gTfOQN6uhfNy5mmm0TlW41/txuLoBGgF4Y/R6NhG63OGjk 0WlMHNrAy6dG4F4qAoZusQVQEhYJBrFvJbrodcfCpDlIdWePfkvQHcjtcIR50xOfsBWk ZtsA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=Hn426ULH; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id p4si608977ejy.509.2021.01.13.01.04.06; Wed, 13 Jan 2021 01:04:30 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=Hn426ULH; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727430AbhAMJCg (ORCPT + 99 others); Wed, 13 Jan 2021 04:02:36 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51276 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727067AbhAMJCg (ORCPT ); Wed, 13 Jan 2021 04:02:36 -0500 Received: from mail-lj1-x234.google.com (mail-lj1-x234.google.com [IPv6:2a00:1450:4864:20::234]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4A3E0C061575; Wed, 13 Jan 2021 01:01:55 -0800 (PST) Received: by mail-lj1-x234.google.com with SMTP id y22so1657412ljn.9; Wed, 13 Jan 2021 01:01:55 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=zkyW24Nem9pNLyI2wkdK3KzUwBFzPFBOJwsBkINN3LQ=; b=Hn426ULH1yMmh0ZdIKPuTDeQAShrud9SMoAHdN1a1K+5pPyYne1zavl4UhOf8Y5EHu vuz5wjnMKki/EvHzDH3QSMqVWaBLQHcBhOHguS5hFMIFrh0YT8hcLeUxpg9KpIns3+R0 2/ldFBLlVoLsVaWrDG+lcAwvTHZ20Sfzz4fICZJJ/tfN31cwx+7LGPdWVx/6CtgTcDfu iKPLXvjmJF50GWJptOBc6khiKZyyraNkuEsk4BV7fHxiEIzlly9n60atGxBb+pmJgdid z2JynXqDhSCODV5gwAtODbBLZxi0yu1JpPZKeNinaTMrsoPSSrey5NzvtHvjH20FnezA azNg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=zkyW24Nem9pNLyI2wkdK3KzUwBFzPFBOJwsBkINN3LQ=; b=mtJOgsDuoTkl9Em2P5UG+ODFkQ8y0fHgCG4AGEs3SLCI5NjyBNw6VnI483kX7gMmmN ikJRHU6OAJWYD8Wx5ClqvYwhq2xeBQnUkkpWH5y87mGFuZcKjqkVZE5Sm94xXeaXmSnZ H0CG+bOzWKZYQNqVFx38vOKgwwbAu+XnBNFGi3DgYbBNqwXXmTco+ZIQc5tDjQwZGjZf /lMNYJK0Wt5mSuh1Zkbt2eSVAgLMBHLVUg2omJoh3LxHKeitJUkTIM37W7+XyoifacMZ 7MXrhf6HWBTp4O3+BXZESFwL3mdw3qURhrrZ3QQjDpvz75oGf4wnvOA1NCigyZ0Y3Z+W 201g== X-Gm-Message-State: AOAM533gHwgxzhZtIh47RPioFiwW17Y8Ph2rMKS+diJFjCngKHF3JGc8 XJ8FBdscN706HLWpNtWUbbSNjeOUBNo= X-Received: by 2002:a2e:9b47:: with SMTP id o7mr484867ljj.99.1610528513514; Wed, 13 Jan 2021 01:01:53 -0800 (PST) Received: from [192.168.2.145] (109-252-192-57.dynamic.spd-mgts.ru. [109.252.192.57]) by smtp.googlemail.com with ESMTPSA id a8sm141538lfo.206.2021.01.13.01.01.52 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 13 Jan 2021 01:01:52 -0800 (PST) Subject: Re: [PATCH v3 0/9] Support Runtime PM and host mode by Tegra ChipIdea USB driver To: Peter Chen , Greg Kroah-Hartman Cc: Peter Chen , Felipe Balbi , Thierry Reding , Jonathan Hunter , Alan Stern , Matt Merhar , Nicolas Chauvet , Peter Geis , Ion Agorria , "linux-tegra@vger.kernel.org" , "linux-usb@vger.kernel.org" , "linux-kernel@vger.kernel.org" References: <20201218120246.7759-1-digetx@gmail.com> <20201229051615.GA5823@b29397-desktop> <713c4b1a-a4b3-41ad-7aad-c49e594f778b@gmail.com> <20210113012036.GA1560@b29397-desktop> From: Dmitry Osipenko Message-ID: <5cfc067a-0b3b-05fc-7567-4b2b475dd6e4@gmail.com> Date: Wed, 13 Jan 2021 12:01:51 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.4.2 MIME-Version: 1.0 In-Reply-To: <20210113012036.GA1560@b29397-desktop> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 13.01.2021 04:20, Peter Chen пишет: > On 21-01-12 09:56:37, Dmitry Osipenko wrote: >> 29.12.2020 17:26, Dmitry Osipenko пишет: >>> 29.12.2020 08:16, Peter Chen пишет: >>>> On 20-12-18 15:02:37, Dmitry Osipenko wrote: >>>>> This series implements Runtime PM support for the Tegra ChipIdea USB driver. >>>>> It also squashes the older ehci-tegra driver into the ChipIdea driver, hence >>>>> the RPM is supported by both UDC and host controllers, secondly this opens >>>>> opportunity for implementing OTG support in the future. >>>>> >>>>> Patchset was tested on various Tegra20, Tegra30 and Tegra124 devices. >>>>> Thanks to Peter Geis, Matt Merhar, Nicolas Chauvet and Ion Agorria for >>>>> helping with the extensive and productive testing! >>>>> >>>>> Changelog: >>>>> >>>>> v3: - Replaced "goto" with if-statements as was suggested by Thierry Reding. >>>>> >>>>> - Improved wording of the deprecated Kconfig entry as was suggested >>>>> by Alan Stern. >>>>> >>>>> - Added ACKs from Thierry Reding and Alan Stern. >>>>> >>>>> - Added a new minor patch "Specify TX FIFO threshold in UDC SoC info" >>>>> just for completeness, since we can now switch OTG to host mode in >>>>> the ChipIdea driver. Although, OTG support remains a work-in-progress >>>>> for now. >>>>> >>>>> v2: - Improved comments in the code as it was suggested by Peter Chen and >>>>> Sergei Shtylyov for v1. >>>>> >>>>> - Replaced mdelay() with fsleep() and made ci->hdc to reset to NULL in >>>>> a error code path, like it was suggested by Peter Chen. >>>>> >>>>> - Redirected deprecated USB_EHCI_TEGRA Kconfig entry to USB_CHIPIDEA_TEGRA >>>>> as was suggested by Alan Stern. >>>>> >>>>> - Improved commit message and added ACK from Thierry Reding to the patch >>>>> that removes MODULE_ALIAS. >>>>> >>>>> - Fixed UDC PHY waking up on ASUS TF201 tablet device by utilizing >>>>> additional VBUS sensor. This was reported and tested by Ion Agorria. >>>>> >>>>> - Added t-b from Ion Agorria. >>>>> >>>>> Dmitry Osipenko (8): >>>>> usb: phy: tegra: Add delay after power up >>>>> usb: phy: tegra: Support waking up from a low power mode >>>>> usb: chipidea: tegra: Remove MODULE_ALIAS >>>>> usb: chipidea: tegra: Rename UDC to USB >>>>> usb: chipidea: tegra: Support runtime PM >>>>> usb: chipidea: tegra: Specify TX FIFO threshold in UDC SoC info >>>>> usb: host: ehci-tegra: Remove the driver >>>>> ARM: tegra_defconfig: Enable USB_CHIPIDEA_HOST and remove >>>>> USB_EHCI_TEGRA >>>>> >>>>> Peter Geis (1): >>>>> usb: chipidea: tegra: Support host mode >>>> >>>> Chipidea related (patch 3-7) are applied, thanks. >>> >>> Hello Peter, >>> >>> Thank you for applying the patches. >>> >>> Who will apply the remaining patches? >>> >>> The Chipidea patch #6 depends on the PHY changes, otherwise USB will >>> suspend and never resume. >>> >> >> Peter, could you please apply the PHY and defconfig patches along with >> the CI patches to -next? I.e. the whole series. Preferentially in >> original ordering of patches should be preserved. >> > > Hi Dmitry, > > Usually, Greg could apply all USB patches, if I apply other USB related > patches, it may cause conflict with other patches in other's tree. > Greg, free feel to apply this series with > Acked-by: Peter Chen > for chipidea part. > > For ARM defconfig patch, I think it should go ARM's tree. > Thank you for the answer! It's not the ARM defconfig, but a local tegra_defconfig. It should be fine to take the patch via the USB tree since Thierry already acked it and asked to do so. Greg, please take this whole series. Thanks in advance!