From: Nicolas Ferre <nicolas.ferre-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
To: Jean-Christophe PLAGNIOL-VILLARD
<plagnioj-sclMFOaUSTBWk0Htik3J/w@public.gmane.org>,
Vinod Koul <vinod.koul-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH 0/3] at_hdmac: dt dma bindings update
Date: Fri, 31 May 2013 10:04:37 +0200 [thread overview]
Message-ID: <51A85995.1050909@atmel.com> (raw)
In-Reply-To: <20130530201321.GE23653-RQcB7r2h9QmfDR2tN2SG5Ni2O/JbrIOy@public.gmane.org>
On 30/05/2013 22:13, Jean-Christophe PLAGNIOL-VILLARD :
> On 22:56 Thu 30 May , Vinod Koul wrote:
>> On Thu, May 30, 2013 at 06:08:20PM +0200, ludovic.desroches-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org wrote:
>>> From: Ludovic Desroches <ludovic.desroches-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
>>>
>>> Hi,
>>>
>>> This set of patches update the dt dma binding for at_hdmac since we need one
>>> more parameter. In order to keep backward compatibility, an existing cell will
>>> be used to add this parameter. Since the content of that cell will become a
>>> magic value due to the concatenation of the two parameters, it is time to
>>> use macros.
>> which tree is this targetted to be applied?
>
> arm-soc as this depends on alreadied applied cleanup I've
... already pushed (it must be the end of your sentence, Jean-Christophe...)
Vinod, Arnd,
- patch 1 shall be applied before any of the two others of the series
- patch 2 is pure DT modification, but relies on Jean-Christophe's
already pushed cleanup branch in arm-soc
- patch 3 is the pure dmaengine one, but relies on the 1st patch
So, maybe the preferred way to include this material upstream is to go
through arm-soc as suggested by Jean-Christophe.
Bye,
>> Arnd, can you review this series and ack if this needs to be applied thru
>> dmaengine?
>
> I prefer the invert
>
> Best Regards,
> J.
>>
>> --
>> ~Vinod
>>>
>>> Regards
>>>
>>> Ludovic Desroches (3):
>>> ARM: at91: dt: add header to define at_hdmac configuration
>>> ARM: at91: dt: switch DMA DT bindings to pre-processor
>>> at_hdmac: add FIFO configuration parameter to DMA DT binding
>>>
>>> .../devicetree/bindings/dma/atmel-dma.txt | 7 ++++--
>>> arch/arm/boot/dts/at91sam9g45.dtsi | 5 ++--
>>> arch/arm/boot/dts/at91sam9n12.dtsi | 11 +++++----
>>> arch/arm/boot/dts/at91sam9x5.dtsi | 17 +++++++-------
>>> arch/arm/boot/dts/sama5d3.dtsi | 19 +++++++--------
>>> drivers/dma/at_hdmac.c | 25 ++++++++++++++++----
>>> include/dt-bindings/dma/at91.h | 27 ++++++++++++++++++++++
>>> 7 files changed, 81 insertions(+), 30 deletions(-)
>>> create mode 100644 include/dt-bindings/dma/at91.h
>>>
>>> --
>>> 1.7.11.3
>>>
>>
>> --
>
--
Nicolas Ferre
prev parent reply other threads:[~2013-05-31 8:04 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-30 16:08 [PATCH 0/3] at_hdmac: dt dma bindings update ludovic.desroches
2013-05-30 16:08 ` [PATCH 1/3] ARM: at91: dt: add header to define at_hdmac configuration ludovic.desroches
[not found] ` <1369930103-11963-2-git-send-email-ludovic.desroches-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
2013-05-31 21:30 ` Arnd Bergmann
[not found] ` <201305312330.29364.arnd-r2nGTMty4D4@public.gmane.org>
2013-06-03 7:32 ` Ludovic Desroches
2013-06-03 7:32 ` Ludovic Desroches
[not found] ` <20130603073226.GG11384@ludovic.desroches@atmel.com>
[not found] ` <20130603073226.GG11384-tOglhBvtKaELqBSMlTbpHvU/3Eu2kcEP@public.gmane.org@atmel.com>
2013-06-04 13:17 ` Nicolas Ferre
2013-05-30 16:08 ` [PATCH 2/3] ARM: at91: dt: switch DMA DT bindings to pre-processor ludovic.desroches
2013-05-30 16:08 ` [PATCH 3/3] at_hdmac: add FIFO configuration parameter to DMA DT binding ludovic.desroches
[not found] ` <1369930103-11963-4-git-send-email-ludovic.desroches-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
2013-05-30 16:32 ` Nicolas Ferre
2013-05-30 16:39 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-31 9:16 ` Ludovic Desroches
[not found] ` <20130530163936.GC23653-RQcB7r2h9QmfDR2tN2SG5Ni2O/JbrIOy@public.gmane.org>
2013-05-31 9:16 ` Ludovic Desroches
[not found] ` <20130531091627.GD11384@ludovic.desroches@atmel.com>
[not found] ` <20130531091627.GD11384-tOglhBvtKaELqBSMlTbpHvU/3Eu2kcEP@public.gmane.org@atmel.com>
2013-05-31 9:31 ` Nicolas Ferre
[not found] ` <51A86DF5.4010808-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
2013-05-31 15:11 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-31 23:11 ` Olof Johansson
[not found] ` <CAOesGMhQetZfyzMpCax63xoRqtRW2ZRXt04MCbgCT-CcvUq43A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-06-03 7:39 ` Ludovic Desroches
2013-06-03 7:39 ` Ludovic Desroches
[not found] ` <1369930103-11963-1-git-send-email-ludovic.desroches-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
2013-05-30 16:37 ` [PATCH 0/3] at_hdmac: dt dma bindings update Jean-Christophe PLAGNIOL-VILLARD
2013-05-30 17:26 ` Vinod Koul
[not found] ` <20130530172649.GC3767-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2013-05-30 20:13 ` Jean-Christophe PLAGNIOL-VILLARD
[not found] ` <20130530201321.GE23653-RQcB7r2h9QmfDR2tN2SG5Ni2O/JbrIOy@public.gmane.org>
2013-05-31 8:04 ` Nicolas Ferre [this message]
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=51A85995.1050909@atmel.com \
--to=nicolas.ferre-aife0yeh4naavxtiumwx3w@public.gmane.org \
--cc=arnd-r2nGTMty4D4@public.gmane.org \
--cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=plagnioj-sclMFOaUSTBWk0Htik3J/w@public.gmane.org \
--cc=vinod.koul-ral2JQCrhuEAvxtiuMwx3w@public.gmane.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 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).