public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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:

  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