From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Brown Subject: Re: [PATCH 05/19] ASoC: mrst_machine: enable user to select different output Date: Wed, 4 May 2011 17:22:32 +0100 Message-ID: <20110504162231.GF10912@opensource.wolfsonmicro.com> References: <20110504133756.32443.6282.stgit@localhost> <20110504134519.32443.90599.stgit@localhost> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from opensource2.wolfsonmicro.com (opensource.wolfsonmicro.com [80.75.67.52]) by alsa0.perex.cz (Postfix) with ESMTP id C19C81037EA for ; Wed, 4 May 2011 18:23:02 +0200 (CEST) Content-Disposition: inline In-Reply-To: <20110504134519.32443.90599.stgit@localhost> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@alsa-project.org Errors-To: alsa-devel-bounces@alsa-project.org To: Lu Guanqun Cc: Takashi Iwai , Koul Vinod , ALSA , Liam Girdwood , Wang Xingchao List-Id: alsa-devel@alsa-project.org On Wed, May 04, 2011 at 09:45:19PM +0800, Lu Guanqun wrote: > + ret = snd_soc_add_controls(codec, > + mrst_snd_controls, > + ARRAY_SIZE(mrst_snd_controls)); > + if (ret) > + return ret; > + It'd be better to use the data based init stuff to add the controls.