[PATCH v3] s3cmci: port DMA code to dmaengine API

Arnd Bergmann arnd at arndb.de
Mon Jul 7 14:12:39 PDT 2014


On Monday 07 July 2014, Vasily Khoruzhick wrote:
> On Mon, Jun 30, 2014 at 10:03 PM, Vasily Khoruzhick <anarsoul at gmail.com> wrote:
> > On Mon, Jun 23, 2014 at 2:31 PM, Vasily Khoruzhick <anarsoul at gmail.com> wrote:
> >>
> >> On 1 June 2014 20:22:09 Vasily Khoruzhick wrote:
> >> > Utilise new s3c24xx-dma dmaengine driver for DMA ops.
> >> >
> >> > Signed-off-by: Vasily Khoruzhick <anarsoul at gmail.com>
> >> > ---
> >> > v2: - Don't store async dma descriptor, it's not used
> >> >     - Request DMA channel via dma_request_slave_channel_compat() and use
> >> >       hardcoded DMA channel.
> >> >     - Add dependency on S3C24XX_DMAC (s3cmci uses filter function from it)
> >> > v3: - Use 'depends' instead of 'select' in Kconfig
> >> >     - Drop useless controller registers reads from dma completion callback
> >>
> >> Ping?
> >
> > Ping?
> 
> Could anyone tell me through which tree this patch should be merged?
> I'm pretty sure that I've included everyone from get_maintainer.pl
> output
> except LKML, but no response from maintainer so far.

I think you did everything right, Ulf or Chris should really pick up this
patch. The one thing you could improve is the changeset text which you
wrote as "Utilise new s3c24xx-dma dmaengine driver for DMA ops." What we
really need here is an explanation of why we want this, e.g. what problems
are solved by it etc. I definitely really want this patch as part of 3.17
so we can enabled multiplatform for the platforms that use this driver.

You should also list the

Acked-by: Arnd Bergmann <arnd at arndb.de>

line under your "Signed-off-by" line.

	Arnd



More information about the linux-arm-kernel mailing list