[PATCH v4] dmaengine: mxs-dma: add dma support for i.MX23/28
Koul, Vinod
vinod.koul at intel.com
Sat Feb 26 10:57:35 EST 2011
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 at 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
More information about the linux-arm-kernel
mailing list