mbox

[GIT,PULL] Qualcomm EBI2 bindings and bus driver

Message ID CACRpkdYvowN2htUuN46JNVdfaKNw3AEbBfzAeScHQcLU5o3ZUg@mail.gmail.com
State New
Headers show

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git

Message

Linus Walleij Sept. 8, 2016, 1:34 p.m. UTC
Hi ARM SoC folks,

please pull this new EBI2 bus driver and its bindings into an
apropriate branch in the ARM SoC tree. The binding now has
Rob Herrings ACK and we have hopefully finally figured out how
this should be done.

Yours,
Linus Walleij

The following changes since commit 29b4817d4018df78086157ea3a55c1d9424a7cfc:

  Linux 4.8-rc1 (2016-08-07 18:18:00 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git
tags/qcom-ebi2-arm-soc

for you to fetch changes up to 335a127548081322bd2b294d715418648912f20c:

  bus: qcom: add EBI2 driver (2016-09-08 15:27:05 +0200)

----------------------------------------------------------------
Qualcomm EBI2 bindings and bus driver.

----------------------------------------------------------------
Linus Walleij (2):
      bus: qcom: add EBI2 device tree bindings
      bus: qcom: add EBI2 driver

 .../devicetree/bindings/bus/qcom,ebi2.txt          | 138 +++++++
 drivers/bus/Kconfig                                |   7 +
 drivers/bus/Makefile                               |   1 +
 drivers/bus/qcom-ebi2.c                            | 408 +++++++++++++++++++++
 4 files changed, 554 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/bus/qcom,ebi2.txt
 create mode 100644 drivers/bus/qcom-ebi2.c

Comments

Arnd Bergmann Sept. 19, 2016, 10:12 p.m. UTC | #1
On Thursday, September 8, 2016 3:34:42 PM CEST Linus Walleij wrote:
> please pull this new EBI2 bus driver and its bindings into an
> apropriate branch in the ARM SoC tree. The binding now has
> Rob Herrings ACK and we have hopefully finally figured out how
> this should be done.
> 

I just noticed that you sent the pull request directly to arm@kernel.org
rather than the Qualcomm maintainers (which are only on Cc).

Usually, I'd expect to see this forwarded from Andy, is there a
reason to do it differently here? If so, can I have an Ack
from him?

	Arnd
Andy Gross Sept. 19, 2016, 10:36 p.m. UTC | #2
On Tue, Sep 20, 2016 at 12:12:33AM +0200, Arnd Bergmann wrote:
> On Thursday, September 8, 2016 3:34:42 PM CEST Linus Walleij wrote:
> > please pull this new EBI2 bus driver and its bindings into an
> > apropriate branch in the ARM SoC tree. The binding now has
> > Rob Herrings ACK and we have hopefully finally figured out how
> > this should be done.
> > 
> 
> I just noticed that you sent the pull request directly to arm@kernel.org
> rather than the Qualcomm maintainers (which are only on Cc).
> 
> Usually, I'd expect to see this forwarded from Andy, is there a
> reason to do it differently here? If so, can I have an Ack
> from him?

When I looked at this last, I thought there were comments.  Sorry I missed it.
If you want to pick it up Arnd, please do, otherwise I can send it along in the
next pull request.

Acked-by: Andy Gross <andy.gross@linaro.org>
Linus Walleij Sept. 20, 2016, 9:49 p.m. UTC | #3
On Tue, Sep 20, 2016 at 12:12 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Thursday, September 8, 2016 3:34:42 PM CEST Linus Walleij wrote:
>> please pull this new EBI2 bus driver and its bindings into an
>> apropriate branch in the ARM SoC tree. The binding now has
>> Rob Herrings ACK and we have hopefully finally figured out how
>> this should be done.
>>
>
> I just noticed that you sent the pull request directly to arm@kernel.org
> rather than the Qualcomm maintainers (which are only on Cc).
>
> Usually, I'd expect to see this forwarded from Andy, is there a
> reason to do it differently here? If so, can I have an Ack
> from him?

Sorry I sent it like this beacuse in an earlier revision of the patch
I asked how it was to be handled and got no answers, so I just assumed
new drivers was a plain separate thing. (I understand Andy has to merge
the DTS changes.)

I saw Andy gave his ACK though, any comments should be addressed
in this version as well.

Yours,
Linus Walleij
Linus Walleij Sept. 20, 2016, 9:50 p.m. UTC | #4
On Tue, Sep 20, 2016 at 12:36 AM, Andy Gross <andy.gross@linaro.org> wrote:
> On Tue, Sep 20, 2016 at 12:12:33AM +0200, Arnd Bergmann wrote:
>> On Thursday, September 8, 2016 3:34:42 PM CEST Linus Walleij wrote:
>> > please pull this new EBI2 bus driver and its bindings into an
>> > apropriate branch in the ARM SoC tree. The binding now has
>> > Rob Herrings ACK and we have hopefully finally figured out how
>> > this should be done.
>> >
>>
>> I just noticed that you sent the pull request directly to arm@kernel.org
>> rather than the Qualcomm maintainers (which are only on Cc).
>>
>> Usually, I'd expect to see this forwarded from Andy, is there a
>> reason to do it differently here? If so, can I have an Ack
>> from him?
>
> When I looked at this last, I thought there were comments.  Sorry I missed it.
> If you want to pick it up Arnd, please do, otherwise I can send it along in the
> next pull request.
>
> Acked-by: Andy Gross <andy.gross@linaro.org>

Thanks, I hope Arnd can pull this as-is.

Will you apply the DTS changes Andy?

Yours,
Linus Walleij
Andy Gross Sept. 20, 2016, 10:09 p.m. UTC | #5
On Tue, Sep 20, 2016 at 11:50:14PM +0200, Linus Walleij wrote:
> On Tue, Sep 20, 2016 at 12:36 AM, Andy Gross <andy.gross@linaro.org> wrote:
> > On Tue, Sep 20, 2016 at 12:12:33AM +0200, Arnd Bergmann wrote:
> >> On Thursday, September 8, 2016 3:34:42 PM CEST Linus Walleij wrote:
> >> > please pull this new EBI2 bus driver and its bindings into an
> >> > apropriate branch in the ARM SoC tree. The binding now has
> >> > Rob Herrings ACK and we have hopefully finally figured out how
> >> > this should be done.
> >> >
> >>
> >> I just noticed that you sent the pull request directly to arm@kernel.org
> >> rather than the Qualcomm maintainers (which are only on Cc).
> >>
> >> Usually, I'd expect to see this forwarded from Andy, is there a
> >> reason to do it differently here? If so, can I have an Ack
> >> from him?
> >
> > When I looked at this last, I thought there were comments.  Sorry I missed it.
> > If you want to pick it up Arnd, please do, otherwise I can send it along in the
> > next pull request.
> >
> > Acked-by: Andy Gross <andy.gross@linaro.org>
> 
> Thanks, I hope Arnd can pull this as-is.
> 
> Will you apply the DTS changes Andy?

Sure I'll put it in a supplemental pull.


Andy
Arnd Bergmann Sept. 21, 2016, 8:42 p.m. UTC | #6
On Tuesday, September 20, 2016 11:50:14 PM CEST Linus Walleij wrote:
> On Tue, Sep 20, 2016 at 12:36 AM, Andy Gross <andy.gross@linaro.org> wrote:
> > On Tue, Sep 20, 2016 at 12:12:33AM +0200, Arnd Bergmann wrote:
> >> On Thursday, September 8, 2016 3:34:42 PM CEST Linus Walleij wrote:
> >> > please pull this new EBI2 bus driver and its bindings into an
> >> > apropriate branch in the ARM SoC tree. The binding now has
> >> > Rob Herrings ACK and we have hopefully finally figured out how
> >> > this should be done.
> >> >
> >>
> >> I just noticed that you sent the pull request directly to arm@kernel.org
> >> rather than the Qualcomm maintainers (which are only on Cc).
> >>
> >> Usually, I'd expect to see this forwarded from Andy, is there a
> >> reason to do it differently here? If so, can I have an Ack
> >> from him?
> >
> > When I looked at this last, I thought there were comments.  Sorry I missed it.
> > If you want to pick it up Arnd, please do, otherwise I can send it along in the
> > next pull request.
> >
> > Acked-by: Andy Gross <andy.gross@linaro.org>
> 
> Thanks, I hope Arnd can pull this as-is.
> 

Pulled into next/drivers, with Andy's Ack added to the merge commit, thanks!

	Arnd