From: Magnus Damm <magnus.damm@gmail.com>
To: linux-sh@vger.kernel.org
Subject: Re: [PATCH v3 0/9] R-Car Gen2 DMA Controller driver
Date: Wed, 29 Oct 2014 02:36:03 +0000 [thread overview]
Message-ID: <CANqRtoTYi3nqxrqMLORCYUTDLPPDG_vaxReysJhocgAOGw5Bww@mail.gmail.com> (raw)
In-Reply-To: <1414345216-14486-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com>
Hi Laurent,
On Wed, Oct 29, 2014 at 6:02 AM, Laurent Pinchart
<laurent.pinchart@ideasonboard.com> wrote:
> Hi Morimoto-san,
>
> On Monday 27 October 2014 20:03:56 Kuninori Morimoto wrote:
>> Hi
>> Cc Simon
>>
>> > Laurent Pinchart (9):
>> > dmaengine: Add 16 bytes, 32 bytes and 64 bytes bus widths
>> > dmaengine: rcar-dmac: Remove duplicate sentence from DT bindings
>> > dmaengine: rcar-dmac: Add Renesas R-Car Gen2 DMA Controller (DMAC)
>> >
>> > driver
>> >
>> > ARM: shmobile: r8a7790: Rename mmcif node to mmc
>> > ARM: shmobile: r8a7791: Add MMCIF0 DT node
>> > ARM: shmobile: r8a7790: Reference DMA channels in MMCIF DT nodes
>> > ARM: shmobile: r8a7791: Reference DMA channels in MMCIF DT node
>> > ARM: shmobile: r8a7790: Reference DMA channels in SDHI DT nodes
>> > ARM: shmobile: r8a7791: Reference DMA channels in SDHI DT nodes
>> >
>> > .../devicetree/bindings/dma/renesas,rcar-dmac.txt | 3 -
>> > arch/arm/boot/dts/r8a7790.dtsi | 14 +-
>> > arch/arm/boot/dts/r8a7791.dtsi | 17 +
>> > drivers/dma/sh/Kconfig | 8 +
>> > drivers/dma/sh/Makefile | 1 +
>> > drivers/dma/sh/rcar-dmac.c | 1533 +++++++++++++++
>> > include/linux/dmaengine.h | 3 +
>> > 7 files changed, 1575 insertions(+), 4 deletions(-)
>> > create mode 100644 drivers/dma/sh/rcar-dmac.c
>>
>> On Lager board + SDHI
>>
>> Tested-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>
> Thank you.
>
>> Current SDHI driver + DMAC is working, but super slow,
>> especially, write case....
>
> *sigh* I'll test that.
If you have energy to deal with this then that is of course greatly appreciated.
From my point of view it is however easier to simply ignore SDHI for
now and remove those patches from this patch series. Once all the
dependencies are sorted out for the new DMAC driver and all code is
upstream then perhaps the SDHI driver is in a better shape and allows
for more easy DMA enablement.
Cheers,
/ magnus
next prev parent reply other threads:[~2014-10-29 2:36 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-26 17:40 [PATCH v3 0/9] R-Car Gen2 DMA Controller driver Laurent Pinchart
2014-10-27 0:45 ` Simon Horman
2014-10-28 3:03 ` Kuninori Morimoto
2014-10-28 21:02 ` Laurent Pinchart
2014-10-29 0:04 ` Kuninori Morimoto
2014-10-29 2:36 ` Magnus Damm [this message]
2014-10-29 7:24 ` Magnus Damm
2014-10-29 9:49 ` Laurent Pinchart
2014-10-29 9:49 ` Phil Edworthy
2014-10-29 9:59 ` Geert Uytterhoeven
2014-10-29 10:05 ` Laurent Pinchart
2014-10-29 11:15 ` Laurent Pinchart
2014-10-29 11:22 ` Laurent Pinchart
2014-10-30 0:16 ` Simon Horman
2014-10-30 13:41 ` Wolfram Sang
2014-11-04 7:07 ` Magnus Damm
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=CANqRtoTYi3nqxrqMLORCYUTDLPPDG_vaxReysJhocgAOGw5Bww@mail.gmail.com \
--to=magnus.damm@gmail.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;
as well as URLs for NNTP newsgroup(s).