linux-usb.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

  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).