* Re: [PATCH] ARM: dts: add AM33XX EDMA support [not found] ` <5218F519.3040409@cogentembedded.com> @ 2013-08-24 18:33 ` Joel Fernandes 2013-08-24 19:33 ` Sergei Shtylyov 0 siblings, 1 reply; 4+ messages in thread From: Joel Fernandes @ 2013-08-24 18:33 UTC (permalink / raw) To: Sergei Shtylyov Cc: Sebastian Andrzej Siewior, Benoit Cousson, tony, devicetree, Matt Porter, linux-kernel, santosh.shilimkar, linux-omap, linux-arm-kernel Updating CC with Matt's current email address. On 08/24/2013 01:02 PM, Sergei Shtylyov wrote: > Hello. > > On 08/23/2013 11:06 PM, Sebastian Andrzej Siewior wrote: > >> From: Matt Porter <mdp@ti.com> > >> Adds AM33XX EDMA support to the am33xx.dtsi as documented in >> Documentation/devicetree/bindings/dma/ti-edma.txt > >> Joel: Drop DT entries that are non-hardware-description for now as discussed >> in [1] > >> [1] https://patchwork.kernel.org/patch/2226761/ > >> Signed-off-by: Matt Porter <mporter@ti.com> >> Signed-off-by: Joel A Fernandes <joelagnel@ti.com> >> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> >> --- >> Could someone please pick this up? > >> arch/arm/boot/dts/am33xx.dtsi | 12 ++++++++++++ >> 1 file changed, 12 insertions(+) > >> diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi >> index 38b446b..784f774 100644 >> --- a/arch/arm/boot/dts/am33xx.dtsi >> +++ b/arch/arm/boot/dts/am33xx.dtsi >> @@ -96,6 +96,18 @@ >> reg = <0x48200000 0x1000>; >> }; >> >> + edma: edma@49000000 { > > The node should be named "dma-controller", not "edma",according to ePAPR > section 2.2.2: > > http://www.power.org/resources/downloads/Power_ePAPR_APPROVED_v1.0.pdf So you mean something like the following? edma: dma-controller@49000000 { ... } Thanks, -Joel ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ARM: dts: add AM33XX EDMA support 2013-08-24 18:33 ` [PATCH] ARM: dts: add AM33XX EDMA support Joel Fernandes @ 2013-08-24 19:33 ` Sergei Shtylyov 0 siblings, 0 replies; 4+ messages in thread From: Sergei Shtylyov @ 2013-08-24 19:33 UTC (permalink / raw) To: Joel Fernandes Cc: Sebastian Andrzej Siewior, Benoit Cousson, tony, devicetree, Matt Porter, linux-kernel, santosh.shilimkar, linux-omap, linux-arm-kernel Hello. On 08/24/2013 10:33 PM, Joel Fernandes wrote: > Updating CC with Matt's current email address. >>> From: Matt Porter <mdp@ti.com> >>> Adds AM33XX EDMA support to the am33xx.dtsi as documented in >>> Documentation/devicetree/bindings/dma/ti-edma.txt >>> Joel: Drop DT entries that are non-hardware-description for now as discussed >>> in [1] >>> [1] https://patchwork.kernel.org/patch/2226761/ >>> Signed-off-by: Matt Porter <mporter@ti.com> >>> Signed-off-by: Joel A Fernandes <joelagnel@ti.com> >>> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> >>> --- >>> Could someone please pick this up? >>> arch/arm/boot/dts/am33xx.dtsi | 12 ++++++++++++ >>> 1 file changed, 12 insertions(+) >>> diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi >>> index 38b446b..784f774 100644 >>> --- a/arch/arm/boot/dts/am33xx.dtsi >>> +++ b/arch/arm/boot/dts/am33xx.dtsi >>> @@ -96,6 +96,18 @@ >>> reg = <0x48200000 0x1000>; >>> }; >>> >>> + edma: edma@49000000 { >> The node should be named "dma-controller", not "edma",according to ePAPR >> section 2.2.2: >> http://www.power.org/resources/downloads/Power_ePAPR_APPROVED_v1.0.pdf > So you mean something like the following? > edma: dma-controller@49000000 { > ... > } Yes, exactly. > Thanks, > -Joel WBR, Sergei ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ARM: dts: add AM33XX EDMA support [not found] <1377284808-32412-1-git-send-email-bigeasy@linutronix.de> [not found] ` <5218F519.3040409@cogentembedded.com> @ 2013-08-26 8:33 ` Benoit Cousson 2013-08-26 16:40 ` Joel Fernandes 1 sibling, 1 reply; 4+ messages in thread From: Benoit Cousson @ 2013-08-26 8:33 UTC (permalink / raw) To: Sebastian Andrzej Siewior Cc: tony, Joel Fernandes, santosh.shilimkar, linux-omap, devicetree, linux-arm-kernel, linux-kernel, Matt Porter, Matt Porter, Joel A Fernandes Hi Sebastian, Is this patch different from that one: http://www.mail-archive.com/linux-omap@vger.kernel.org/msg92176.html Lokesh just pointed me this patch because it was missing for the SHAM/AES series from Mark Greer. Bottom-line, I've just applied the original one along with a second one that was adding EDMA in SPI. Regards, Benoit On 23/08/2013 21:06, Sebastian Andrzej Siewior wrote: > From: Matt Porter <mdp@ti.com> > > Adds AM33XX EDMA support to the am33xx.dtsi as documented in > Documentation/devicetree/bindings/dma/ti-edma.txt > > Joel: Drop DT entries that are non-hardware-description for now as discussed in [1] > > [1] https://patchwork.kernel.org/patch/2226761/ > > Signed-off-by: Matt Porter <mporter@ti.com> > Signed-off-by: Joel A Fernandes <joelagnel@ti.com> > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> > --- > Could someone please pick this up? > > arch/arm/boot/dts/am33xx.dtsi | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi > index 38b446b..784f774 100644 > --- a/arch/arm/boot/dts/am33xx.dtsi > +++ b/arch/arm/boot/dts/am33xx.dtsi > @@ -96,6 +96,18 @@ > reg = <0x48200000 0x1000>; > }; > > + edma: edma@49000000 { > + compatible = "ti,edma3"; > + ti,hwmods = "tpcc", "tptc0", "tptc1", "tptc2"; > + reg = <0x49000000 0x10000>, > + <0x44e10f90 0x10>; > + interrupts = <12 13 14>; > + #dma-cells = <1>; > + dma-channels = <64>; > + ti,edma-regions = <4>; > + ti,edma-slots = <256>; > + }; > + > gpio0: gpio@44e07000 { > compatible = "ti,omap4-gpio"; > ti,hwmods = "gpio1"; > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ARM: dts: add AM33XX EDMA support 2013-08-26 8:33 ` Benoit Cousson @ 2013-08-26 16:40 ` Joel Fernandes 0 siblings, 0 replies; 4+ messages in thread From: Joel Fernandes @ 2013-08-26 16:40 UTC (permalink / raw) To: Benoit Cousson Cc: Sebastian Andrzej Siewior, tony, santosh.shilimkar, linux-omap, devicetree, linux-arm-kernel, linux-kernel, Matt Porter Hi Benoit, On 08/26/2013 03:33 AM, Benoit Cousson wrote: > Hi Sebastian, > > Is this patch different from that one: > http://www.mail-archive.com/linux-omap@vger.kernel.org/msg92176.html > > Lokesh just pointed me this patch because it was missing for the SHAM/AES series > from Mark Greer. > > Bottom-line, I've just applied the original one along with a second one that was > adding EDMA in SPI. Sebastian just submitted another version of the patch [1]. This is the latest version of the EDMA DT patch. If its ok, could you use this patch? You could ignore all earlier versions. [1] http://www.spinics.net/lists/arm-kernel/msg269712.html Thanks, -Joel > Regards, > Benoit > > > On 23/08/2013 21:06, Sebastian Andrzej Siewior wrote: >> From: Matt Porter <mdp@ti.com> >> >> Adds AM33XX EDMA support to the am33xx.dtsi as documented in >> Documentation/devicetree/bindings/dma/ti-edma.txt >> >> Joel: Drop DT entries that are non-hardware-description for now as discussed >> in [1] >> >> [1] https://patchwork.kernel.org/patch/2226761/ >> >> Signed-off-by: Matt Porter <mporter@ti.com> >> Signed-off-by: Joel A Fernandes <joelagnel@ti.com> >> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> >> --- >> Could someone please pick this up? >> >> arch/arm/boot/dts/am33xx.dtsi | 12 ++++++++++++ >> 1 file changed, 12 insertions(+) >> >> diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi >> index 38b446b..784f774 100644 >> --- a/arch/arm/boot/dts/am33xx.dtsi >> +++ b/arch/arm/boot/dts/am33xx.dtsi >> @@ -96,6 +96,18 @@ >> reg = <0x48200000 0x1000>; >> }; >> >> + edma: edma@49000000 { >> + compatible = "ti,edma3"; >> + ti,hwmods = "tpcc", "tptc0", "tptc1", "tptc2"; >> + reg = <0x49000000 0x10000>, >> + <0x44e10f90 0x10>; >> + interrupts = <12 13 14>; >> + #dma-cells = <1>; >> + dma-channels = <64>; >> + ti,edma-regions = <4>; >> + ti,edma-slots = <256>; >> + }; >> + >> gpio0: gpio@44e07000 { >> compatible = "ti,omap4-gpio"; >> ti,hwmods = "gpio1"; >> > ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-08-26 16:40 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1377284808-32412-1-git-send-email-bigeasy@linutronix.de>
[not found] ` <5218F519.3040409@cogentembedded.com>
2013-08-24 18:33 ` [PATCH] ARM: dts: add AM33XX EDMA support Joel Fernandes
2013-08-24 19:33 ` Sergei Shtylyov
2013-08-26 8:33 ` Benoit Cousson
2013-08-26 16:40 ` Joel Fernandes
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).