From: Antoine Tenart <antoine.tenart@free-electrons.com>
To: sebastian.hesselbarth@gmail.com, Peter.Chen@freescale.com,
balbi@ti.com, p.zabel@pengutronix.de, arnd@arndb.de
Cc: Antoine Tenart <antoine.tenart@free-electrons.com>,
alexandre.belloni@free-electrons.com,
thomas.petazzoni@free-electrons.com, zmxu@marvell.com,
jszhang@marvell.com, linux-arm-kernel@lists.infradead.org,
linux-usb@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v6 07/12] usb: chipidea: add a usb2 driver for ci13xxx
Date: Wed, 1 Oct 2014 14:39:14 +0200 [thread overview]
Message-ID: <20141001123914.GA25307@kwain> (raw)
In-Reply-To: <20140929150837.GD8912@kwain>
Peter, Arnd, Felipe,
On Mon, Sep 29, 2014 at 05:08:37PM +0200, Antoine Tenart wrote:
> On Tue, Sep 23, 2014 at 12:28:03PM +0200, Antoine Tenart wrote:
> > Add a USB2 ChipIdea driver for ci13xxx, with optional PHY, clock
> > and DMA mask, to support USB2 ChipIdea controllers that don't need
> > specific functions.
>
> Did we agree on the modifications needed to get this accepted? If so,
> can one of you sum up what's need to be done, I got a bit lost reading
> all the discussions on this thread.
>
> Anyway, that would be good to get this series accepted as the Berlin USB
> support depends on it. This series has been out since the beginning of
> June and non-critical changes can be added by future series.
I'm seriously considering sending a new version with dt-only support as I
don't need the other case. There are still many discussions between the
three of you regarding DMA support, which is not a problem for my use
case. As this is not a modification of an existing driver, future
additions can be done to support more cases later and we do not need
this driver to handle all possible cases right now as this won't be a
regression.
Please, can one of you sum up the modifications needed to get this
accepted, if any. If the dt part is fine for you, I'll resend this
series with dt-only support. The platform compatibility can surely be
added later, when needed by someone, and when an agreement on how to do
it is reached.
I'm afraid this series is currently blocked by implementation related
discussions not linked with the USB Berlin support, while this could be
done as a follow up series.
Thanks,
Antoine
--
Antoine Ténart, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
WARNING: multiple messages have this Message-ID (diff)
From: antoine.tenart@free-electrons.com (Antoine Tenart)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v6 07/12] usb: chipidea: add a usb2 driver for ci13xxx
Date: Wed, 1 Oct 2014 14:39:14 +0200 [thread overview]
Message-ID: <20141001123914.GA25307@kwain> (raw)
In-Reply-To: <20140929150837.GD8912@kwain>
Peter, Arnd, Felipe,
On Mon, Sep 29, 2014 at 05:08:37PM +0200, Antoine Tenart wrote:
> On Tue, Sep 23, 2014 at 12:28:03PM +0200, Antoine Tenart wrote:
> > Add a USB2 ChipIdea driver for ci13xxx, with optional PHY, clock
> > and DMA mask, to support USB2 ChipIdea controllers that don't need
> > specific functions.
>
> Did we agree on the modifications needed to get this accepted? If so,
> can one of you sum up what's need to be done, I got a bit lost reading
> all the discussions on this thread.
>
> Anyway, that would be good to get this series accepted as the Berlin USB
> support depends on it. This series has been out since the beginning of
> June and non-critical changes can be added by future series.
I'm seriously considering sending a new version with dt-only support as I
don't need the other case. There are still many discussions between the
three of you regarding DMA support, which is not a problem for my use
case. As this is not a modification of an existing driver, future
additions can be done to support more cases later and we do not need
this driver to handle all possible cases right now as this won't be a
regression.
Please, can one of you sum up the modifications needed to get this
accepted, if any. If the dt part is fine for you, I'll resend this
series with dt-only support. The platform compatibility can surely be
added later, when needed by someone, and when an agreement on how to do
it is reached.
I'm afraid this series is currently blocked by implementation related
discussions not linked with the USB Berlin support, while this could be
done as a follow up series.
Thanks,
Antoine
--
Antoine T?nart, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2014-10-01 12:39 UTC|newest]
Thread overview: 147+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-23 10:27 [PATCH v6 00/12] ARM: berlin: USB support Antoine Tenart
2014-09-23 10:27 ` Antoine Tenart
2014-09-23 10:27 ` Antoine Tenart
2014-09-23 10:27 ` [PATCH v6 01/12] reset: add the Berlin reset controller driver Antoine Tenart
2014-09-23 10:27 ` Antoine Tenart
2014-09-23 10:27 ` Antoine Tenart
2014-09-23 10:27 ` [PATCH v6 02/12] Documentation: bindings: add reset bindings docs for Marvell Berlin SoCs Antoine Tenart
2014-09-23 10:27 ` Antoine Tenart
2014-09-23 10:27 ` Antoine Tenart
2014-09-23 10:27 ` [PATCH v6 03/12] ARM: Berlin: select the reset controller Antoine Tenart
2014-09-23 10:27 ` Antoine Tenart
2014-09-23 10:27 ` Antoine Tenart
2014-09-23 10:28 ` [PATCH v6 04/12] ARM: dts: berlin: add a required reset property in the chip controller node Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` [PATCH v6 05/12] phy: add the Berlin USB PHY driver Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` [PATCH v6 06/12] Documentation: bindings: add doc for the Berlin USB PHY Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` [PATCH v6 07/12] usb: chipidea: add a usb2 driver for ci13xxx Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:39 ` Arnd Bergmann
2014-09-23 10:39 ` Arnd Bergmann
2014-09-23 10:39 ` Arnd Bergmann
2014-09-23 13:36 ` Antoine Tenart
2014-09-23 13:36 ` Antoine Tenart
2014-09-23 13:36 ` Antoine Tenart
2014-09-23 16:44 ` Arnd Bergmann
2014-09-23 16:44 ` Arnd Bergmann
2014-09-23 16:55 ` Felipe Balbi
2014-09-23 16:55 ` Felipe Balbi
2014-09-23 16:55 ` Felipe Balbi
2014-09-23 17:37 ` Arnd Bergmann
2014-09-23 17:37 ` Arnd Bergmann
2014-09-23 17:37 ` Arnd Bergmann
2014-09-23 17:43 ` Felipe Balbi
2014-09-23 17:43 ` Felipe Balbi
2014-09-23 17:43 ` Felipe Balbi
2014-09-24 2:27 ` Peter Chen
2014-09-24 2:27 ` Peter Chen
2014-09-24 2:27 ` Peter Chen
2014-09-24 7:44 ` Arnd Bergmann
2014-09-24 7:44 ` Arnd Bergmann
2014-09-24 7:44 ` Arnd Bergmann
2014-09-24 8:30 ` Arnd Bergmann
2014-09-24 8:30 ` Arnd Bergmann
2014-09-24 11:29 ` Peter Chen
2014-09-24 11:29 ` Peter Chen
2014-09-24 11:29 ` Peter Chen
2014-09-24 12:23 ` Arnd Bergmann
2014-09-24 12:23 ` Arnd Bergmann
2014-09-25 0:56 ` Peter Chen
2014-09-25 0:56 ` Peter Chen
2014-09-25 0:56 ` Peter Chen
2014-09-25 14:15 ` Felipe Balbi
2014-09-25 14:15 ` Felipe Balbi
2014-09-25 14:15 ` Felipe Balbi
2014-09-25 23:39 ` Peter Chen
2014-09-25 23:39 ` Peter Chen
2014-09-25 23:39 ` Peter Chen
2014-09-26 0:37 ` Felipe Balbi
2014-09-26 0:37 ` Felipe Balbi
2014-09-26 0:37 ` Felipe Balbi
2014-09-26 0:39 ` Felipe Balbi
2014-09-26 0:39 ` Felipe Balbi
2014-09-26 0:39 ` Felipe Balbi
2014-09-26 7:20 ` Arnd Bergmann
2014-09-26 7:20 ` Arnd Bergmann
2014-09-26 15:43 ` Felipe Balbi
2014-09-26 15:43 ` Felipe Balbi
2014-09-26 15:43 ` Felipe Balbi
2014-09-28 0:40 ` Peter Chen
2014-09-28 0:40 ` Peter Chen
2014-09-28 0:40 ` Peter Chen
2014-10-13 8:47 ` Peter Chen
2014-10-13 8:47 ` Peter Chen
2014-10-13 8:47 ` Peter Chen
2014-09-25 14:09 ` Felipe Balbi
2014-09-25 14:09 ` Felipe Balbi
2014-09-25 14:09 ` Felipe Balbi
2014-09-24 23:58 ` Sören Brinkmann
2014-09-24 23:58 ` Sören Brinkmann
2014-09-24 23:58 ` Sören Brinkmann
2014-09-29 14:57 ` Antoine Tenart
2014-09-29 14:57 ` Antoine Tenart
2014-09-29 14:57 ` Antoine Tenart
2014-09-25 1:16 ` Peter Chen
2014-09-25 1:16 ` Peter Chen
2014-09-25 1:16 ` Peter Chen
2014-09-25 7:11 ` Arnd Bergmann
2014-09-25 7:11 ` Arnd Bergmann
2014-09-25 7:11 ` Arnd Bergmann
2014-09-26 0:23 ` Peter Chen
2014-09-26 0:23 ` Peter Chen
2014-09-26 0:23 ` Peter Chen
2014-09-26 7:01 ` Arnd Bergmann
2014-09-26 7:01 ` Arnd Bergmann
2014-09-25 19:12 ` Arnd Bergmann
2014-09-25 19:12 ` Arnd Bergmann
2014-09-25 19:54 ` Antoine Tenart
2014-09-25 19:54 ` Antoine Tenart
2014-09-25 19:54 ` Antoine Tenart
2014-09-29 15:08 ` Antoine Tenart
2014-09-29 15:08 ` Antoine Tenart
2014-09-29 15:08 ` Antoine Tenart
2014-10-01 12:39 ` Antoine Tenart [this message]
2014-10-01 12:39 ` Antoine Tenart
2014-09-30 0:12 ` Peter Chen
2014-09-30 0:12 ` Peter Chen
2014-09-30 0:12 ` Peter Chen
2014-09-30 10:03 ` Arnd Bergmann
2014-09-30 10:03 ` Arnd Bergmann
2014-09-30 10:03 ` Arnd Bergmann
2014-09-30 12:39 ` Peter Chen
2014-09-30 12:39 ` Peter Chen
2014-09-30 12:39 ` Peter Chen
2014-09-30 13:43 ` Arnd Bergmann
2014-09-30 13:43 ` Arnd Bergmann
2014-09-30 13:43 ` Arnd Bergmann
2014-10-01 6:35 ` Peter Chen
2014-10-01 6:35 ` Peter Chen
2014-10-01 6:35 ` Peter Chen
2014-10-01 8:41 ` Arnd Bergmann
2014-10-01 8:41 ` Arnd Bergmann
2014-10-01 8:41 ` Arnd Bergmann
2014-10-01 12:25 ` Peter Chen
2014-10-01 12:25 ` Peter Chen
2014-10-01 12:25 ` Peter Chen
2014-10-01 23:40 ` Peter Chen
2014-10-01 23:40 ` Peter Chen
2014-10-01 23:40 ` Peter Chen
2014-09-23 10:28 ` [PATCH v6 08/12] Documentation: bindings: add doc for the USB2 ChipIdea USB driver Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` [PATCH v6 09/12] ARM: dts: berlin: add BG2Q nodes for USB support Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` [PATCH v6 10/12] ARM: dts: Berlin: enable USB on the BG2Q DMP Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` [PATCH v6 11/12] ARM: dts: berlin: add BG2CD nodes for USB support Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` [PATCH v6 12/12] ARM: dts: berlin: enable USB on the Google Chromecast Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
2014-09-23 10:28 ` Antoine Tenart
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=20141001123914.GA25307@kwain \
--to=antoine.tenart@free-electrons.com \
--cc=Peter.Chen@freescale.com \
--cc=alexandre.belloni@free-electrons.com \
--cc=arnd@arndb.de \
--cc=balbi@ti.com \
--cc=devicetree@vger.kernel.org \
--cc=jszhang@marvell.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=p.zabel@pengutronix.de \
--cc=sebastian.hesselbarth@gmail.com \
--cc=thomas.petazzoni@free-electrons.com \
--cc=zmxu@marvell.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 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.