From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH v3] s3cmci: port DMA code to dmaengine API Date: Mon, 7 Jul 2014 23:12:39 +0200 Message-ID: <201407072312.40687.arnd@arndb.de> References: <1401643329-6460-1-git-send-email-anarsoul@gmail.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset="utf-8" Content-Transfer-Encoding: 7bit Return-path: Received: from mout.kundenserver.de ([212.227.126.130]:56474 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751424AbaGGVNf (ORCPT ); Mon, 7 Jul 2014 17:13:35 -0400 In-Reply-To: Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Vasily Khoruzhick Cc: Ulf Hansson , arm-linux , linux-mmc , Chris Ball , Jaehoon Chung , Mark Brown , Ben Dooks On Monday 07 July 2014, Vasily Khoruzhick wrote: > On Mon, Jun 30, 2014 at 10:03 PM, Vasily Khoruzhick wrote: > > On Mon, Jun 23, 2014 at 2:31 PM, Vasily Khoruzhick 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 > >> > --- > >> > 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 line under your "Signed-off-by" line. Arnd