From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: linux-sh@vger.kernel.org
Subject: Re: [PATCH 6/7] ARM: shmobile: r8a7790: Add DMAC devices to DT
Date: Tue, 15 Jul 2014 22:52:42 +0000 [thread overview]
Message-ID: <4554201.y5rj3hyRxJ@avalon> (raw)
In-Reply-To: <1405455522-20676-7-git-send-email-laurent.pinchart+renesas@ideasonboard.com>
Hi Iwamatsu-san,
Thank you for the review.
On Wednesday 16 July 2014 07:49:15 Nobuhiro Iwamatsu wrote:
> 2014-07-16 5:18 GMT+09:00 Laurent Pinchart
>
> <laurent.pinchart+renesas@ideasonboard.com>:
> > Instantiate the two system DMA controllers in the r8a7790 device tree.
> >
> > Signed-off-by: Laurent Pinchart
> > <laurent.pinchart+renesas@ideasonboard.com>
> > ---
> >
> > arch/arm/boot/dts/r8a7790.dtsi | 59 +++++++++++++++++++++++++++++++++++++
> > 1 file changed, 59 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/r8a7790.dtsi
> > b/arch/arm/boot/dts/r8a7790.dtsi index 220662f..0f15f7c 100644
> > --- a/arch/arm/boot/dts/r8a7790.dtsi
> > +++ b/arch/arm/boot/dts/r8a7790.dtsi
> > @@ -217,6 +217,65 @@
> > <0 3 IRQ_TYPE_LEVEL_HIGH>;
> > };
> >
> > + dmac0: dma-controller@e6700000 {
> > + compatible = "renesas,rcar-dmac";
> > + reg = <0 0xe6700000 0 0x20000>;
> > + interrupts = <0 197 IRQ_TYPE_LEVEL_HIGH
> > + 0 200 IRQ_TYPE_LEVEL_HIGH
> > + 0 201 IRQ_TYPE_LEVEL_HIGH
> > + 0 202 IRQ_TYPE_LEVEL_HIGH
> > + 0 203 IRQ_TYPE_LEVEL_HIGH
> > + 0 204 IRQ_TYPE_LEVEL_HIGH
> > + 0 205 IRQ_TYPE_LEVEL_HIGH
> > + 0 206 IRQ_TYPE_LEVEL_HIGH
> > + 0 207 IRQ_TYPE_LEVEL_HIGH
> > + 0 208 IRQ_TYPE_LEVEL_HIGH
> > + 0 209 IRQ_TYPE_LEVEL_HIGH
> > + 0 210 IRQ_TYPE_LEVEL_HIGH
> > + 0 211 IRQ_TYPE_LEVEL_HIGH
> > + 0 212 IRQ_TYPE_LEVEL_HIGH
> > + 0 213 IRQ_TYPE_LEVEL_HIGH
> > + 0 214 IRQ_TYPE_LEVEL_HIGH>;
> > + interrupt-names = "error",
> > + "ch0", "ch1", "ch2", "ch3",
> > + "ch4", "ch5", "ch6", "ch7",
> > + "ch8", "ch9", "ch10", "ch11",
> > + "ch12", "ch13", "ch14", "ch15";
> > + clocks = <&mstp2_clks R8A7790_CLK_SYS_DMAC0>;
> > + clock-names = "fck";
> > + #dma-cells = <1>;
> > + dma-channels = <15>;
> > + };
>
> I think that dma-channels should be set to 16.
The r8a7790 and r8a7791 datasheets document 30 channels, split across two DMA
controllers with 15 channels each.
> > +
> > + dmac1: dma-controller@e6720000 {
> > + compatible = "renesas,rcar-dmac";
> > + reg = <0 0xe6720000 0 0x20000>;
> > + interrupts = <0 220 IRQ_TYPE_LEVEL_HIGH
> > + 0 216 IRQ_TYPE_LEVEL_HIGH
> > + 0 217 IRQ_TYPE_LEVEL_HIGH
> > + 0 218 IRQ_TYPE_LEVEL_HIGH
> > + 0 219 IRQ_TYPE_LEVEL_HIGH
> > + 0 308 IRQ_TYPE_LEVEL_HIGH
> > + 0 309 IRQ_TYPE_LEVEL_HIGH
> > + 0 310 IRQ_TYPE_LEVEL_HIGH
> > + 0 311 IRQ_TYPE_LEVEL_HIGH
> > + 0 312 IRQ_TYPE_LEVEL_HIGH
> > + 0 313 IRQ_TYPE_LEVEL_HIGH
> > + 0 314 IRQ_TYPE_LEVEL_HIGH
> > + 0 315 IRQ_TYPE_LEVEL_HIGH
> > + 0 316 IRQ_TYPE_LEVEL_HIGH
> > + 0 317 IRQ_TYPE_LEVEL_HIGH
> > + 0 318 IRQ_TYPE_LEVEL_HIGH>;
> > + interrupt-names = "error",
> > + "ch0", "ch1", "ch2", "ch3",
> > + "ch4", "ch5", "ch6", "ch7",
> > + "ch8", "ch9", "ch10", "ch11",
> > + "ch12", "ch13", "ch14", "ch15";
> > + clocks = <&mstp2_clks R8A7790_CLK_SYS_DMAC1>;
> > + clock-names = "fck";
> > + #dma-cells = <1>;
> > + dma-channels = <15>;
>
> likewise.
>
> > + };
> >
> > i2c0: i2c@e6508000 {
> > #address-cells = <1>;
> > #size-cells = <0>;
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2014-07-15 22:52 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-15 20:18 [PATCH 6/7] ARM: shmobile: r8a7790: Add DMAC devices to DT Laurent Pinchart
2014-07-15 22:49 ` Nobuhiro Iwamatsu
2014-07-15 22:52 ` Laurent Pinchart [this message]
2014-07-15 23:06 ` Nobuhiro Iwamatsu
2014-07-15 23:12 ` Laurent Pinchart
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=4554201.y5rj3hyRxJ@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=linux-sh@vger.kernel.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