From: Peter Chen <peter.chen@nxp.com>
To: Dmitry Osipenko <digetx@gmail.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 05:16:45 +0000 [thread overview]
Message-ID: <20201229051615.GA5823@b29397-desktop> (raw)
In-Reply-To: <20201218120246.7759-1-digetx@gmail.com>
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.
Peter
>
> arch/arm/configs/tegra_defconfig | 3 +-
> drivers/usb/chipidea/Kconfig | 3 +-
> drivers/usb/chipidea/ci_hdrc_tegra.c | 344 ++++++++++++---
> drivers/usb/chipidea/core.c | 10 +-
> drivers/usb/chipidea/host.c | 104 ++++-
> drivers/usb/host/Kconfig | 10 +-
> drivers/usb/host/Makefile | 1 -
> drivers/usb/host/ehci-tegra.c | 604 ---------------------------
> drivers/usb/phy/phy-tegra-usb.c | 103 ++++-
> include/linux/usb/chipidea.h | 6 +
> include/linux/usb/tegra_usb_phy.h | 2 +
> 11 files changed, 518 insertions(+), 672 deletions(-)
> delete mode 100644 drivers/usb/host/ehci-tegra.c
>
> --
> 2.29.2
>
--
Thanks,
Peter Chen
next prev parent reply other threads:[~2020-12-29 5:17 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-12-18 12:02 [PATCH v3 0/9] Support Runtime PM and host mode by Tegra ChipIdea USB driver Dmitry Osipenko
2020-12-18 12:02 ` [PATCH v3 1/9] usb: phy: tegra: Add delay after power up Dmitry Osipenko
2020-12-18 12:02 ` [PATCH v3 2/9] usb: phy: tegra: Support waking up from a low power mode Dmitry Osipenko
2020-12-18 12:02 ` [PATCH v3 3/9] usb: chipidea: tegra: Remove MODULE_ALIAS Dmitry Osipenko
2020-12-18 12:02 ` [PATCH v3 4/9] usb: chipidea: tegra: Rename UDC to USB Dmitry Osipenko
2020-12-18 12:02 ` [PATCH v3 5/9] usb: chipidea: tegra: Support host mode Dmitry Osipenko
2020-12-18 12:02 ` [PATCH v3 6/9] usb: chipidea: tegra: Support runtime PM Dmitry Osipenko
2020-12-18 12:02 ` [PATCH v3 7/9] usb: chipidea: tegra: Specify TX FIFO threshold in UDC SoC info Dmitry Osipenko
2020-12-18 12:02 ` [PATCH v3 8/9] usb: host: ehci-tegra: Remove the driver Dmitry Osipenko
2020-12-18 12:02 ` [PATCH v3 9/9] ARM: tegra_defconfig: Enable USB_CHIPIDEA_HOST and remove USB_EHCI_TEGRA Dmitry Osipenko
2020-12-29 5:16 ` Peter Chen [this message]
2020-12-29 14:26 ` [PATCH v3 0/9] Support Runtime PM and host mode by Tegra ChipIdea USB driver Dmitry Osipenko
2021-01-12 6:56 ` Dmitry Osipenko
2021-01-13 1:20 ` Peter Chen
2021-01-13 9:01 ` Dmitry Osipenko
2021-01-13 10:27 ` Greg Kroah-Hartman
2021-01-13 14:28 ` Dmitry Osipenko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201229051615.GA5823@b29397-desktop \
--to=peter.chen@nxp.com \
--cc=balbi@kernel.org \
--cc=digetx@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=ion@agorria.com \
--cc=jonathanh@nvidia.com \
--cc=kwizart@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=mattmerhar@protonmail.com \
--cc=pgwipeout@gmail.com \
--cc=stern@rowland.harvard.edu \
--cc=thierry.reding@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).