From mboxrd@z Thu Jan 1 00:00:00 1970 From: Liam Girdwood Subject: Re: [RFC][PATCH 21/26] alsa: ASoC: Add JZ4740 ASoC support Date: Thu, 03 Jun 2010 18:03:29 +0100 Message-ID: <1275584609.3118.26.camel@odin> References: <1275505397-16758-1-git-send-email-lars@metafoo.de> <1275505950-17334-5-git-send-email-lars@metafoo.de> <1275569309.3593.106.camel@odin> <4C07DD48.2050503@metafoo.de> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4C07DD48.2050503@metafoo.de> Sender: linux-kernel-owner@vger.kernel.org To: Lars-Peter Clausen Cc: Ralf Baechle , linux-mips@linux-mips.org, linux-kernel@vger.kernel.org, Mark Brown , alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org On Thu, 2010-06-03 at 18:50 +0200, Lars-Peter Clausen wrote: > >> + config = snd_soc_dai_get_dma_data(rtd->dai->cpu_dai, > substream); > >> + if (!prtd->dma) { > >> + const char *dma_channel_name; > >> + if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) > >> + dma_channel_name = "PCM Playback"; > >> + else > >> + dma_channel_name = "PCM Capture"; > >> + > >> + prtd->dma = jz4740_dma_request(substream, > dma_channel_name); > >> > > > > dma_channel_name variable is not required here. Just use the const > char > > * directly. > > > > > I actually had it like that before, but I think it is much more readable > in its current form. I disagree, having the char pointer here just adds an extra level of indirection and costs an extra two lines of code. Liam -- Freelance Developer, SlimLogic Ltd ASoC and Voltage Regulator Maintainer. http://www.slimlogic.co.uk