Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
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

      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