From: Russell King <rmk@arm.linux.org.uk>
To: Jaswinder Singh <jaswinder.singh@linaro.org>
Cc: "Koul, Vinod" <vinod.koul@intel.com>,
"Williams, Dan J" <dan.j.williams@intel.com>,
Linus Walleij <linus.walleij@linaro.org>,
linux-kernel@vger.kernel.org, linus.walleij@stericsson.com,
per.friden@stericsson.com, wei.zhang@freescale.com,
ebony.zhu@freescale.com, iws@ovro.caltech.edu,
s.hauer@pengutronix.de, maciej.sosnowski@intel.com,
saeed@marvell.com, shawn.guo@freescale.com, yur@emcraft.com,
agust@denx.de, iwamatsu.nobuhiro@renesas.com,
per.forlin@stericsson.com, jonas.aberg@stericsson.com,
anemo@mba.ocn.ne.jp
Subject: Re: [PATCHv2] DMAEngine: Let dmac drivers to set chan_id
Date: Thu, 28 Jul 2011 23:40:38 +0100 [thread overview]
Message-ID: <20110728224038.GD829@flint.arm.linux.org.uk> (raw)
In-Reply-To: <CAJe_ZhcggEW1wfF8EVy-qgfa194B+-qifongLOvPwVg5O9AcBg@mail.gmail.com>
On Thu, Jul 28, 2011 at 11:24:12PM +0530, Jaswinder Singh wrote:
> On 28 July 2011 02:07, Russell King <rmk@arm.linux.org.uk> wrote:
> >
> >> On a serious note, my proposal, and the reply, shows the possibility
> >> of having :-
> >> a) Client drivers that are truly platform agnostic -- no platform_data
> >> poking for
> >> channel selection
> >
> > I really doubt that's even possible. Take this setup:
>
> I don't want to suggest anything wrong just because I didn't understand
> your h/w.
> I hope you will be kind enough to help me better understand your setup,
> so that I have a fair chance to present my proposal.
>
> A simple 'yes' or a 'no'(with clarification) is all I ask.
>
> >
> > MMCI ---> DMAC
> >
> > where the DMAC has 32 request signals, and 8 channels.
> The DMAC is similar to PL330.
That coincidence is merely that.
> Only max 8 request-signals can be active at any time.
> Is my understanding right ?
Correct.
> > The MMCI is connected to two of them.
> I don't know anything about MMCI.
> So I assume it is just another third party MMC controller.
> It simply needs 2 dma channels(for RX, TX each) - be it from a DMAC
> that has a programmable RequestSignal->Peripheral map or a fixed map.
> Is my understanding right ?
Correct.
> > The DMAC can supply any of its physical channels for MMCI.
> The RequestSignal->Peripheral map is decided during board design
> and can not be changed later.
> Is my understanding right ?
No. See the board#3 case..
> > Board 1 has the MMCI connected to request signals #1 and #3.
> > Board 2 has the MMCI connected to request signals #8 and #22.
> Say,
> Board1
> MMCI_RX -> #1
> MMCI_TX -> #3
>
> Board2
> MMCI_RX -> #8
> MMCI_TX -> #22
>
> > Board 3 has the MMCI connected through an external FPGA mux, which can route the
> > MMCI requests to DMA request signals #1, #2 or #3.
> Say
> Board3
> MMCI_RX -> #{1,2,3}
> MMCI_TX -> #{1,2,3}
> And you can't change the route(mapping) after the dmac driver has
> been loaded.
No. You have to change it dynamically at run time according to the
DMA activity, because DMA request signals #1, #2 and #3 are shared
between 6 devices. To make matters worse, it's not six on any of
RQ#1 RQ#2 RQ#3, but some on a couple, some on another couple, and
some on all three.
BTW, we do support this with Linus W's code (which he's posted to
this thread.)
--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
maintainer of:
next prev parent reply other threads:[~2011-07-28 22:40 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-20 18:18 [PATCH] DMAEngine: Let dmac drivers set chan_id Jassi Brar
2011-07-21 4:01 ` [PATCHv2] DMAEngine: Let dmac drivers to " Jassi Brar
2011-07-22 15:27 ` Linus Walleij
2011-07-22 17:43 ` Jaswinder Singh
2011-07-22 22:23 ` Williams, Dan J
2011-07-23 3:56 ` Jaswinder Singh
2011-07-25 18:55 ` Williams, Dan J
2011-07-25 19:17 ` Jaswinder Singh
2011-07-25 20:08 ` Williams, Dan J
2011-07-26 14:30 ` Jaswinder Singh
2011-07-26 15:29 ` Williams, Dan J
2011-07-26 18:12 ` Jaswinder Singh
2011-07-27 4:21 ` Koul, Vinod
2011-07-27 7:17 ` Jaswinder Singh
2011-07-27 9:02 ` Koul, Vinod
2011-07-27 9:59 ` Mika Westerberg
2011-07-27 9:34 ` Koul, Vinod
2011-07-27 10:36 ` Mika Westerberg
2011-07-27 14:50 ` Jaswinder Singh
2011-07-27 16:36 ` Williams, Dan J
2011-07-27 17:14 ` Jaswinder Singh
2011-07-27 20:28 ` Russell King
2011-07-28 10:44 ` Jaswinder Singh
2011-07-28 22:27 ` Linus Walleij
2011-07-28 22:43 ` Russell King
2011-07-29 12:20 ` Linus Walleij
2011-07-29 11:54 ` Koul, Vinod
2011-07-28 22:35 ` Russell King
2011-07-29 14:11 ` Jaswinder Singh
2011-07-27 14:30 ` Jaswinder Singh
2011-07-27 20:37 ` Russell King
2011-07-28 10:56 ` Jaswinder Singh
2011-07-28 13:44 ` Russell King
2011-07-28 17:54 ` Jaswinder Singh
2011-07-28 18:14 ` Williams, Dan J
2011-07-28 18:25 ` Jaswinder Singh
2011-07-28 22:40 ` Russell King [this message]
2011-07-30 13:09 ` Jaswinder Singh
2011-07-30 14:22 ` Russell King
2011-07-30 15:00 ` Jaswinder Singh
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=20110728224038.GD829@flint.arm.linux.org.uk \
--to=rmk@arm.linux.org.uk \
--cc=agust@denx.de \
--cc=anemo@mba.ocn.ne.jp \
--cc=dan.j.williams@intel.com \
--cc=ebony.zhu@freescale.com \
--cc=iwamatsu.nobuhiro@renesas.com \
--cc=iws@ovro.caltech.edu \
--cc=jaswinder.singh@linaro.org \
--cc=jonas.aberg@stericsson.com \
--cc=linus.walleij@linaro.org \
--cc=linus.walleij@stericsson.com \
--cc=linux-kernel@vger.kernel.org \
--cc=maciej.sosnowski@intel.com \
--cc=per.forlin@stericsson.com \
--cc=per.friden@stericsson.com \
--cc=s.hauer@pengutronix.de \
--cc=saeed@marvell.com \
--cc=shawn.guo@freescale.com \
--cc=vinod.koul@intel.com \
--cc=wei.zhang@freescale.com \
--cc=yur@emcraft.com \
/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