Re: [PATCH v3 0/9] Support Runtime PM and host mode by Tegra ChipIdea USB driver

From: Dmitry Osipenko <digetx@gmail.com>
To: Peter Chen <peter.chen@nxp.com>
Cc: Thierry Reding <thierry.reding@gmail.com>,
	Jonathan Hunter <jonathanh@nvidia.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Alan Stern <stern@rowland.harvard.edu>,
	Felipe Balbi <balbi@kernel.org>,
	Matt Merhar <mattmerhar@protonmail.com>,
	Nicolas Chauvet <kwizart@gmail.com>,
	Peter Geis <pgwipeout@gmail.com>, Ion Agorria <ion@agorria.com>,
	"linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>,
	"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v3 0/9] Support Runtime PM and host mode by Tegra ChipIdea USB driver
Date: Tue, 29 Dec 2020 17:26:58 +0300	[thread overview]
Message-ID: <b2c21687-4cb9-ba0a-a724-3a82ddd8daff@gmail.com> (raw)
In-Reply-To: <20201229051615.GA5823@b29397-desktop>

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.