From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752407Ab3KSTt4 (ORCPT ); Tue, 19 Nov 2013 14:49:56 -0500 Received: from smtp-out-036.synserver.de ([212.40.185.36]:1053 "EHLO smtp-out-164.synserver.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752058Ab3KSTtz (ORCPT ); Tue, 19 Nov 2013 14:49:55 -0500 X-SynServer-TrustedSrc: 1 X-SynServer-AuthUser: lars@metafoo.de X-SynServer-PPID: 30819 Message-ID: <528BBE90.4020802@metafoo.de> Date: Tue, 19 Nov 2013 20:40:00 +0100 From: Lars-Peter Clausen User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20131103 Icedove/17.0.10 MIME-Version: 1.0 To: Lee Jones CC: Mark Brown , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linus.walleij@linaro.org, alsa-devel@alsa-project.org Subject: Re: [PATCH 08/10] ASoC: ux500_pcm: Differentiate between pdata and DT initialisation References: <1384859269-19801-1-git-send-email-lee.jones@linaro.org> <1384859269-19801-9-git-send-email-lee.jones@linaro.org> <20131119184829.GF2674@sirena.org.uk> <20131119193341.GC19953@lee--X1> In-Reply-To: <20131119193341.GC19953@lee--X1> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/19/2013 08:33 PM, Lee Jones wrote: > On Tue, 19 Nov 2013, Mark Brown wrote: > >> On Tue, Nov 19, 2013 at 11:07:47AM +0000, Lee Jones wrote: >> >>> require slightly different flags to inform the core that we 'are' >>> booting with DT. >> >> Is there some situation when we would want to say we're booting from DT >> when we aren't? Just wondering about the quotes. > > The quotes do 'not' mean anything special. :) > >>> +static const struct snd_dmaengine_pcm_config ux500_dmaengine_of_pcm_config = { >>> + .pcm_hardware = &ux500_pcm_hw, >>> + .prealloc_buffer_size = 128 * 1024, >> >> You shouldn't need to set this explicitly, the generic code should be >> able to pick a number for you - if you do need this number please >> explain why the number was chosen in the comments (or fix the core to >> guess better). At the minute the core just makes up a number too but at >> least then it's a consistent random number between platforms. >> >> Can you also get away without the pcm_hardware - the core should also >> have support for discovering this by querying the DMA controller? > > Despite the '+'s, I'm not actually adding these parameters, I'm > duplicating the pdata version and removing the stuff I 'know' that's > not required. I don't know what happens when/if these two parameters > are removed. I can add this to my TODO when I rip out platform data > support, which will happen when this stuff lands. > I think the patch is fine for now. Once non-DT support has been removed for ux500 we should be able to remove the whole ux500_pcm.c file (Assuming that the ux500 DMA engine driver gains dma_slave_caps support). - Lars