mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Russell King - ARM Linux <linux@armlinux.org.uk>
To: Vinod Koul <vinod.koul@intel.com>
Cc: Mason <slash.tmp@free.fr>, Lars-Peter Clausen <lars@metafoo.de>,
	Dave Jiang <dave.jiang@intel.com>, Arnd Bergmann <arnd@arndb.de>,
	Mark Brown <broonie@linaro.org>,
	Linus Walleij <linus.walleij@linaro.org>,
	Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
	LKML <linux-kernel@vger.kernel.org>,
	Mans Rullgard <mans@mansr.com>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	dmaengine@vger.kernel.org,
	Dan Williams <dan.j.williams@intel.com>,
	Jon Mason <jon.mason@intel.com>, Lee Jones <lee.jones@linaro.org>,
	Maxime Ripard <maxime.ripard@free-electrons.com>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>
Subject: Re: Tearing down DMA transfer setup after DMA client has finished
Date: Fri, 25 Nov 2016 12:45:29 +0000	[thread overview]
Message-ID: <20161125124528.GJ14217@n2100.armlinux.org.uk> (raw)
In-Reply-To: <20161125045549.GC2698@localhost>

On Fri, Nov 25, 2016 at 10:25:49AM +0530, Vinod Koul wrote:
> Looking at thread and discussion now, first thinking would be to ensure
> the transaction is completed properly and then isr fired. You may need
> to talk to your HW designers to find a way for that. It is quite common
> that DMA controllers will fire and complete whereas the transaction is
> still in flight.
> 
> If that is not doable, then since you claim this is custom part which
> other vendors wont use (hope we are wrong down the line), then we can
> have a custom api,
> 
> foo_sbox_configure(bool enable, ...);
> 
> This can be invoked from NFC driver when required for configuration and
> teardown. For very specific cases where people need some specific
> configuration we do allow custom APIs.
> 
> Only problem with that would be it wont be a generic solution and you
> seem to be fine with that.

Isn't this just the same problem as PL08x or any other system which
has multiple requests from devices, but only a limited number of
hardware channels - so you have to route the request signals to the
appropriate hardware channels according to the requests queued up?

If so, no new "custom" APIs are required, it's already able to be
solved within the DMA engine drivers...

(We also have more complex situations already supported, such as
PL08x with a FPGA routing on three of its request signals.)

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

  parent reply	other threads:[~2016-11-25 12:46 UTC|newest]

Thread overview: 82+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-23 10:25 Mason
2016-11-23 12:13 ` Måns Rullgård
2016-11-23 12:41   ` Mason
2016-11-23 17:21     ` Måns Rullgård
2016-11-24 10:53       ` Mason
2016-11-24 14:17         ` Måns Rullgård
2016-11-24 15:20           ` Mason
2016-11-24 16:37             ` Måns Rullgård
2016-11-25  4:55 ` Vinod Koul
2016-11-25 11:57   ` Måns Rullgård
2016-11-25 14:05     ` Mason
2016-11-25 14:12       ` Måns Rullgård
2016-11-25 14:28         ` Mason
2016-11-25 14:42           ` Måns Rullgård
2016-11-25 12:45   ` Russell King - ARM Linux [this message]
2016-11-25 13:07     ` Måns Rullgård
2016-11-25 13:34       ` Russell King - ARM Linux
2016-11-25 13:50         ` Måns Rullgård
2016-11-25 13:58           ` Russell King - ARM Linux
2016-11-25 14:03             ` Måns Rullgård
2016-11-25 14:17               ` Russell King - ARM Linux
2016-11-25 14:40                 ` Måns Rullgård
2016-11-25 14:56                   ` Russell King - ARM Linux
2016-11-25 15:08                     ` Måns Rullgård
2016-11-25 15:02                 ` Mason
2016-11-25 15:12                   ` Måns Rullgård
2016-11-25 15:21                     ` Mason
2016-11-25 15:28                       ` Måns Rullgård
2016-11-25 12:46   ` Mason
2016-11-25 13:11     ` Måns Rullgård
2016-11-25 14:21       ` Mason
2016-11-25 14:37         ` Måns Rullgård
2016-11-25 15:35           ` Mason
2016-11-29 18:25     ` Mason
2016-12-06  5:12       ` Vinod Koul
2016-12-06 12:42         ` Mason
2016-12-06 13:14           ` Måns Rullgård
2016-12-06 15:24             ` Mason
2016-12-06 15:34               ` Måns Rullgård
2016-12-06 22:55                 ` Mason
2016-12-07 16:43             ` Vinod Koul
2016-12-07 16:45               ` Måns Rullgård
2016-12-08 10:39                 ` Vinod Koul
2016-12-08 10:54                   ` Mason
2016-12-08 11:18                     ` Geert Uytterhoeven
2016-12-08 11:47                       ` Måns Rullgård
2016-12-08 12:03                         ` Geert Uytterhoeven
2016-12-08 12:17                           ` Mason
2016-12-08 12:21                           ` Måns Rullgård
2016-12-08 16:37                     ` Vinod Koul
2016-12-08 16:48                       ` Måns Rullgård
2016-12-09  6:59                         ` Vinod Koul
2016-12-09 10:25                           ` Sebastian Frias
2016-12-09 11:34                             ` Måns Rullgård
2016-12-09 11:35                             ` 1Måns Rullgård
2016-12-09 17:17                             ` Vinod Koul
2016-12-09 17:28                               ` Måns Rullgård
2016-12-09 17:53                                 ` Vinod Koul
2016-12-09 17:34                               ` Mason
2016-12-09 17:56                                 ` Vinod Koul
2016-12-09 18:17                                   ` Vinod Koul
2016-12-09 18:23                                   ` Mason
2016-12-12  5:01                                     ` Vinod Koul
2016-12-15 11:17                                     ` Mark Brown
2016-12-08 11:44                   ` Måns Rullgård
2016-12-08 11:59                     ` Geert Uytterhoeven
2016-12-08 12:20                       ` Måns Rullgård
2016-12-08 12:31                         ` Geert Uytterhoeven
2016-12-08 12:41                         ` Mason
2016-12-08 12:44                           ` Måns Rullgård
2016-12-08 13:29                             ` Mason
2016-12-08 13:39                               ` Måns Rullgård
2016-12-08 15:50                         ` Vinod Koul
2016-12-08 16:36                           ` Måns Rullgård
2016-12-08 15:40                     ` Vinod Koul
2016-12-08 15:43                       ` Mason
2016-12-08 16:21                         ` Vinod Koul
2016-12-08 16:46                       ` Måns Rullgård
2016-12-07 16:41           ` Vinod Koul
2016-12-07 16:44             ` Måns Rullgård
2016-12-08 10:37               ` Vinod Koul
2016-12-08 11:44                 ` Måns Rullgård

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=20161125124528.GJ14217@n2100.armlinux.org.uk \
    --to=linux@armlinux.org.uk \
    --cc=arnd@arndb.de \
    --cc=b.zolnierkie@samsung.com \
    --cc=broonie@linaro.org \
    --cc=dan.j.williams@intel.com \
    --cc=dave.jiang@intel.com \
    --cc=dmaengine@vger.kernel.org \
    --cc=jon.mason@intel.com \
    --cc=lars@metafoo.de \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=lee.jones@linaro.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mans@mansr.com \
    --cc=maxime.ripard@free-electrons.com \
    --cc=slash.tmp@free.fr \
    --cc=vinod.koul@intel.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

all inboxes | Powered by JetHome®