From: vinod.koul@intel.com (Koul, Vinod)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4] dmaengine: mxs-dma: add dma support for i.MX23/28
Date: Sat, 26 Feb 2011 21:27:35 +0530 [thread overview]
Message-ID: <1298735855.6974.21.camel@vkoul-udesk3> (raw)
In-Reply-To: <1298619259-14344-1-git-send-email-shawn.guo@freescale.com>
On Fri, 2011-02-25 at 15:34 +0800, Shawn Guo wrote:
> This patch adds dma support for Freescale MXS-based SoC i.MX23/28,
> including apbh-dma and apbx-dma.
>
> * apbh-dma and apbx-dma are supported in the driver as two mxs-dma
> instances.
>
> * apbh-dma is different between mx23 and mx28, hardware version
> register is used to differentiate.
>
> * mxs-dma supports pio function besides data transfer. The driver
> uses dma_data_direction DMA_NONE to identify the pio mode, and
> steals sgl and sg_len to get pio words and numbers from clients.
>
> * mxs dmaengine has some very specific features, like sense function
> and the special NAND support (nand_lock, nand_wait4ready). These
> are too specific to implemented in generic dmaengine driver.
>
> * The driver refers to imx-sdma and only a single descriptor is
> statically assigned to each channel.
>
> Signed-off-by: Shawn Guo <shawn.guo@freescale.com>
Please fix the checkpatch errors and warnings, at least it should have
zero errors.
ERROR: code indent should use tabs where possible
#430: FILE: drivers/dma/mxs-dma.c:250:
+ if (mxs_chan->desc.callback)$
WARNING: please, no spaces at the start of a line
#430: FILE: drivers/dma/mxs-dma.c:250:
+ if (mxs_chan->desc.callback)$
ERROR: code indent should use tabs where possible
#431: FILE: drivers/dma/mxs-dma.c:251:
+ mxs_chan->desc.callback(mxs_chan->desc.callback_param);
WARNING: please, no spaces at the start of a line
#431: FILE: drivers/dma/mxs-dma.c:251:
+ mxs_chan->desc.callback(mxs_chan->desc.callback_param);
ERROR: do not initialise statics to 0 or NULL
#564: FILE: drivers/dma/mxs-dma.c:384:
+ static int idx = 0;
~Vinod
prev parent reply other threads:[~2011-02-26 15:57 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-25 7:34 [PATCH v4] dmaengine: mxs-dma: add dma support for i.MX23/28 Shawn Guo
2011-02-25 10:55 ` Wolfram Sang
2011-02-25 11:11 ` Shawn Guo
2011-02-26 15:57 ` Koul, Vinod [this message]
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=1298735855.6974.21.camel@vkoul-udesk3 \
--to=vinod.koul@intel.com \
--cc=linux-arm-kernel@lists.infradead.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