Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754734Ab2JJKAG (ORCPT ); Wed, 10 Oct 2012 06:00:06 -0400 Received: from arroyo.ext.ti.com ([192.94.94.40]:47070 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752716Ab2JJKAD convert rfc822-to-8bit (ORCPT ); Wed, 10 Oct 2012 06:00:03 -0400 From: "Kumar, Anil" To: Daniel Mack CC: "Porter, Matt" , Mark Brown , "Hebbar, Gururaja" , "Nori, Sekhar" , Linux Kernel Mailing List , Alsa Devel List , Linux DaVinci Kernel List , Ben Gardiner Subject: RE: [PATCH 0/2] Convert davinci ASoC to genalloc SRAM Thread-Topic: [PATCH 0/2] Convert davinci ASoC to genalloc SRAM Thread-Index: AQHNpqffLG9GQkGCZEGqausiKS4GCZeyGaAg//+n7YCAAIm4AA== Date: Wed, 10 Oct 2012 09:59:48 +0000 Message-ID: References: <1349345745-10584-1-git-send-email-mporter@ti.com> <50751E0F.3010906@gmail.com> In-Reply-To: <50751E0F.3010906@gmail.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [172.24.170.142] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3109 Lines: 78 On Wed, Oct 10, 2012 at 12:34:47, Daniel Mack wrote: > On 10.10.2012 08:54, Kumar, Anil wrote: > > On Thu, Oct 04, 2012 at 15:45:43, Porter, Matt wrote: > >> This series converts davinci ASoC to use genalloc and enables > >> that support on DA850. It applies on top of the uio_pruss/genalloc > >> series [1] which allows DaVinci to provide a gen_pool via pdata > >> for driver use. > >> > >> I've tested this on the AM180x EVM. Note that prior to this, > >> the SRAM paths in the driver were completely unused. I've only > >> enabled ping-pong buffering on the platform I can test as it's > >> best to allow those with DM644x and similar platforms to set the > >> playback/capture sram size to something that's known to work > >> for them. > >> > >> [1] http://www.spinics.net/lists/arm-kernel/msg198854.html > > > > Tested on DA850 EVM after applying patches from [1] above. > > Tested on Linux-torvalds 3.6-rc6 master branch. > > Tested ASoC module (aplay & arecord for various frequencies) > > > > Acked-by: Kumar, Anil > > As you're on it, could you also quickly merge this branch > > git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git > topic/davinci > > And see if my recent patches for more McASP flexibility cause any > trouble on your hardware? > > > Daniel > > I have merged it with 3.6-rc6 and I have got error at compile time. I have fixed this locally with the following patch [1]. Kindly confirm if the patch is correct? With that Tested on DA850 EVM Tested on Linux-torvalds 3.6-rc6 master branch merged + [1] Tested ASoC module (aplay & arecord for various frequencies) Tested-By: Kumar, Anil [1] git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/davinci. [2] patch diff --git a/include/linux/dmaengine.h b/include/linux/dmaengine.h index 09da4e5..9c02a45 100644 --- a/include/linux/dmaengine.h +++ b/include/linux/dmaengine.h @@ -591,7 +591,7 @@ struct dma_device { struct dma_async_tx_descriptor *(*device_prep_dma_cyclic)( struct dma_chan *chan, dma_addr_t buf_addr, size_t buf_len, size_t period_len, enum dma_transfer_direction direction, - unsigned long flags, void *context); + void *context); struct dma_async_tx_descriptor *(*device_prep_interleaved_dma)( struct dma_chan *chan, struct dma_interleaved_template *xt, unsigned long flags); @@ -656,7 +656,7 @@ static inline struct dma_async_tx_descriptor *dmaengine_prep_dma_cyclic( size_t period_len, enum dma_transfer_direction dir) { return chan->device->device_prep_dma_cyclic(chan, buf_addr, buf_len, - period_len, dir, flags, NULL); + period_len, dir, NULL); } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/