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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Monday 07 July 2014, Vasily Khoruzhick wrote:
> On Mon, Jun 30, 2014 at 10:03 PM, Vasily Khoruzhick <anarsoul@xxxxxxxxx> wrote:
> > On Mon, Jun 23, 2014 at 2:31 PM, Vasily Khoruzhick <anarsoul@xxxxxxxxx> 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@xxxxxxxxx>
> >> > ---
> >> > 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@xxxxxxxx>

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

	Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux