From: gregory.clement@free-electrons.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 0/3] Add EHCI support for Armada 37xx
Date: Thu, 09 Mar 2017 18:42:46 +0100 [thread overview]
Message-ID: <87shmmjqi1.fsf@free-electrons.com> (raw)
In-Reply-To: <20170309172827.17874-1-gregory.clement@free-electrons.com> (Gregory CLEMENT's message of "Thu, 9 Mar 2017 18:28:24 +0100")
Hi,
On jeu., mars 09 2017, Gregory CLEMENT <gregory.clement@free-electrons.com> wrote:
> Hi,
>
> The EHCI controller in the Armada 37xx SoCs is the one used on many
> other mvebu SoCs such as the orion5x, the kirkwood, or the
> armada. However, for Armada 37xx an extra initialization step is
> needed: this is the purpose of the first patch.
>
> The second patch allows to build the driver for the ARM64 Armada SoCs.
>
> The last one enables the EHCI in the device tree.
>
> This third version takes into account the review from Alan Stern.
>
> Thanks,
>
> Gregory
>
> Changelog:
> v2-> v3:
> - Remove the _SHIFT define and use their value directly, suggested by
> Alan Stern
> - Remove unnecessary error message in ehci_orion_drv_reset() ,
> suggested by Alan Stern
> - Exit ehci_orion_drv_reset() if ehci_setup() failed, suggested by
> Alan Stern
All the above changes are not in the series I've just sent. I will send
the correct series in a few minutes
sorry for the noise!
Gregory
> - Add Reviewed-by flag from Andrew Lunn
>
> v1 -> v2:
> - Fix typo in commit log of the patch 1, suggested by Andrew Lunn
> - Fix wording in the device tree binding documenation, suggested by
> Andrew Lunn
> - Make new define symbol name more clear by adding _SHIFT to the end,
> suggested by Andrew Lunn
> - Use a full first name + last name for the Signed-off-by and the
> Authorship i oatch 1, suggested by Thomas Petazzoni.
> - Improve the commit log of the patch 2, suggested by Andrew Lunn
> - Fix Kconfig logic, suggested by Andrew Lunn
>
> Gregory CLEMENT (2):
> usb: host: Allow to build ehci orion with mvebu SoCs
> ARM64: dts: marvell: armada-37xx: Add USB2 node
>
> Hua Jing (1):
> usb: orion-echi: Add support for the Armada 3700
>
> .../devicetree/bindings/usb/ehci-orion.txt | 4 ++-
> arch/arm64/boot/dts/marvell/armada-3720-db.dts | 6 ++++
> arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 7 ++++
> drivers/usb/host/Kconfig | 2 +-
> drivers/usb/host/ehci-orion.c | 39 ++++++++++++++++++++++
> 5 files changed, 56 insertions(+), 2 deletions(-)
>
> --
> 2.11.0
>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
WARNING: multiple messages have this Message-ID (diff)
From: Gregory CLEMENT <gregory.clement@free-electrons.com>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
linux-usb@vger.kernel.org, Jason Cooper <jason@lakedaemon.net>,
Andrew Lunn <andrew@lunn.ch>,
Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
Rob Herring <robh+dt@kernel.org>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
linux-arm-kernel@lists.infradead.org,
Nadav Haklai <nadavh@marvell.com>, Victor Gu <xigu@marvell.com>,
Marcin Wojtas <mw@semihalf.com>,
Wilson Ding <dingwei@marvell.com>, Hua Jing <jinghua@marvell.com>,
Neta Zur Hershkovits <neta@marvell.com>
Subject: Re: [PATCH v3 0/3] Add EHCI support for Armada 37xx
Date: Thu, 09 Mar 2017 18:42:46 +0100 [thread overview]
Message-ID: <87shmmjqi1.fsf@free-electrons.com> (raw)
In-Reply-To: <20170309172827.17874-1-gregory.clement@free-electrons.com> (Gregory CLEMENT's message of "Thu, 9 Mar 2017 18:28:24 +0100")
Hi,
On jeu., mars 09 2017, Gregory CLEMENT <gregory.clement@free-electrons.com> wrote:
> Hi,
>
> The EHCI controller in the Armada 37xx SoCs is the one used on many
> other mvebu SoCs such as the orion5x, the kirkwood, or the
> armada. However, for Armada 37xx an extra initialization step is
> needed: this is the purpose of the first patch.
>
> The second patch allows to build the driver for the ARM64 Armada SoCs.
>
> The last one enables the EHCI in the device tree.
>
> This third version takes into account the review from Alan Stern.
>
> Thanks,
>
> Gregory
>
> Changelog:
> v2-> v3:
> - Remove the _SHIFT define and use their value directly, suggested by
> Alan Stern
> - Remove unnecessary error message in ehci_orion_drv_reset() ,
> suggested by Alan Stern
> - Exit ehci_orion_drv_reset() if ehci_setup() failed, suggested by
> Alan Stern
All the above changes are not in the series I've just sent. I will send
the correct series in a few minutes
sorry for the noise!
Gregory
> - Add Reviewed-by flag from Andrew Lunn
>
> v1 -> v2:
> - Fix typo in commit log of the patch 1, suggested by Andrew Lunn
> - Fix wording in the device tree binding documenation, suggested by
> Andrew Lunn
> - Make new define symbol name more clear by adding _SHIFT to the end,
> suggested by Andrew Lunn
> - Use a full first name + last name for the Signed-off-by and the
> Authorship i oatch 1, suggested by Thomas Petazzoni.
> - Improve the commit log of the patch 2, suggested by Andrew Lunn
> - Fix Kconfig logic, suggested by Andrew Lunn
>
> Gregory CLEMENT (2):
> usb: host: Allow to build ehci orion with mvebu SoCs
> ARM64: dts: marvell: armada-37xx: Add USB2 node
>
> Hua Jing (1):
> usb: orion-echi: Add support for the Armada 3700
>
> .../devicetree/bindings/usb/ehci-orion.txt | 4 ++-
> arch/arm64/boot/dts/marvell/armada-3720-db.dts | 6 ++++
> arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 7 ++++
> drivers/usb/host/Kconfig | 2 +-
> drivers/usb/host/ehci-orion.c | 39 ++++++++++++++++++++++
> 5 files changed, 56 insertions(+), 2 deletions(-)
>
> --
> 2.11.0
>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2017-03-09 17:42 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-09 17:28 [PATCH v3 0/3] Add EHCI support for Armada 37xx Gregory CLEMENT
2017-03-09 17:28 ` Gregory CLEMENT
2017-03-09 17:28 ` Gregory CLEMENT
2017-03-09 17:28 ` [PATCH v3 1/3] usb: orion-echi: Add support for the Armada 3700 Gregory CLEMENT
2017-03-09 17:28 ` Gregory CLEMENT
2017-03-09 17:28 ` Gregory CLEMENT
2017-03-09 17:28 ` [PATCH v3 2/3] usb: host: Allow to build ehci orion with mvebu SoCs Gregory CLEMENT
2017-03-09 17:28 ` Gregory CLEMENT
2017-03-09 17:28 ` Gregory CLEMENT
2017-03-09 17:28 ` [PATCH v3 3/3] ARM64: dts: marvell: armada-37xx: Add USB2 node Gregory CLEMENT
2017-03-09 17:28 ` Gregory CLEMENT
2017-03-09 17:42 ` Gregory CLEMENT [this message]
2017-03-09 17:42 ` [PATCH v3 0/3] Add EHCI support for Armada 37xx Gregory CLEMENT
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=87shmmjqi1.fsf@free-electrons.com \
--to=gregory.clement@free-electrons.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.