From: Vinod Koul <vinod.koul@intel.com>
To: Joel Fernandes <joelf@ti.com>
Cc: Matt Porter <matt@ohporter.com>, Mark Brown <broonie@linaro.org>,
Russell King <linux@arm.linux.org.uk>, Dan Williams <djbw@fb.com>,
Jyri Sarha <jsarha@ti.com>, Lars Peter-Clausen <lars@metafoo.de>,
Linux OMAP List <linux-omap@vger.kernel.org>,
Linux ARM Kernel List <linux-arm-kernel@lists.infradead.org>,
Linux DaVinci Kernel List
<davinci-linux-open-source@linux.davincidsp.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
alsa-devel@alsa-project.org
Subject: Re: [PATCH 1/3] dma: edma: Split out PaRAM set calculations into its own function
Date: Mon, 21 Oct 2013 12:56:12 +0530 [thread overview]
Message-ID: <20131021072612.GN14013@intel.com> (raw)
In-Reply-To: <1379977515-3794-2-git-send-email-joelf@ti.com>
On Mon, Sep 23, 2013 at 06:05:13PM -0500, Joel Fernandes wrote:
> PaRAM set calculation is abstracted into its own function to
> enable better reuse for other DMA cases such as cyclic. We adapt
> the Slave SG case to use the new function.
>
> This provides a much cleaner abstraction to the internals of the
> PaRAM set. However, any PaRAM attributes that are not common to
> all DMA types must be set separately such as setting of interrupts.
> This function takes care of the most-common attributes.
>
> Also added comments clarifying A-sync case calculations.
Applied, thanks
--
~Vinod
WARNING: multiple messages have this Message-ID (diff)
From: vinod.koul@intel.com (Vinod Koul)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/3] dma: edma: Split out PaRAM set calculations into its own function
Date: Mon, 21 Oct 2013 12:56:12 +0530 [thread overview]
Message-ID: <20131021072612.GN14013@intel.com> (raw)
In-Reply-To: <1379977515-3794-2-git-send-email-joelf@ti.com>
On Mon, Sep 23, 2013 at 06:05:13PM -0500, Joel Fernandes wrote:
> PaRAM set calculation is abstracted into its own function to
> enable better reuse for other DMA cases such as cyclic. We adapt
> the Slave SG case to use the new function.
>
> This provides a much cleaner abstraction to the internals of the
> PaRAM set. However, any PaRAM attributes that are not common to
> all DMA types must be set separately such as setting of interrupts.
> This function takes care of the most-common attributes.
>
> Also added comments clarifying A-sync case calculations.
Applied, thanks
--
~Vinod
next prev parent reply other threads:[~2013-10-21 7:26 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-23 23:05 [PATCH 0/3] dma: edma: Add cyclic DMA support Joel Fernandes
2013-09-23 23:05 ` Joel Fernandes
2013-09-23 23:05 ` Joel Fernandes
[not found] ` <1379977515-3794-1-git-send-email-joelf-l0cyMroinI0@public.gmane.org>
2013-09-23 23:05 ` [PATCH 1/3] dma: edma: Split out PaRAM set calculations into its own function Joel Fernandes
2013-09-23 23:05 ` Joel Fernandes
2013-09-23 23:05 ` Joel Fernandes
2013-10-21 7:26 ` Vinod Koul [this message]
2013-10-21 7:26 ` Vinod Koul
2013-09-23 23:05 ` [PATCH 2/3] dma: edma: Add support for Cyclic DMA Joel Fernandes
2013-09-23 23:05 ` Joel Fernandes
2013-09-23 23:05 ` Joel Fernandes
2013-10-21 6:53 ` Vinod Koul
2013-10-21 6:53 ` Vinod Koul
[not found] ` <20131021065319.GM14013-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2013-10-22 15:30 ` Joel Fernandes
2013-10-22 15:30 ` Joel Fernandes
2013-10-22 15:30 ` Joel Fernandes
2013-10-24 16:38 ` Vinod Koul
2013-10-24 16:38 ` Vinod Koul
[not found] ` <20131024163829.GA21230-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2013-10-24 17:57 ` Joel Fernandes
2013-10-24 17:57 ` Joel Fernandes
2013-10-24 17:57 ` Joel Fernandes
2013-10-31 14:10 ` Vinod Koul
2013-10-31 14:10 ` Vinod Koul
[not found] ` <20131031141012.GJ18788-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2013-10-31 16:03 ` Joel Fernandes
2013-10-31 16:03 ` Joel Fernandes
2013-10-31 16:03 ` Joel Fernandes
2013-09-23 23:05 ` [PATCH 3/3] dma: edma: Increase maximum SG limit to 20 Joel Fernandes
2013-09-23 23:05 ` Joel Fernandes
2013-09-23 23:05 ` Joel Fernandes
2013-10-21 7:26 ` Vinod Koul
2013-10-21 7:26 ` Vinod Koul
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=20131021072612.GN14013@intel.com \
--to=vinod.koul@intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@linaro.org \
--cc=davinci-linux-open-source@linux.davincidsp.com \
--cc=djbw@fb.com \
--cc=joelf@ti.com \
--cc=jsarha@ti.com \
--cc=lars@metafoo.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=matt@ohporter.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.