From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lu Guanqun Subject: Re: [PATCH 05/19] ASoC: mrst_machine: enable user to select different output Date: Thu, 5 May 2011 08:34:07 +0800 Message-ID: <20110505003407.GC8633@qtel.sh.intel.com> References: <20110504133756.32443.6282.stgit@localhost> <20110504134519.32443.90599.stgit@localhost> <20110504162231.GF10912@opensource.wolfsonmicro.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by alsa0.perex.cz (Postfix) with ESMTP id 4C46E10391D for ; Thu, 5 May 2011 02:35:24 +0200 (CEST) Content-Disposition: inline In-Reply-To: <20110504162231.GF10912@opensource.wolfsonmicro.com> 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: Mark Brown Cc: Takashi Iwai , "Koul, Vinod" , ALSA , Liam Girdwood , "Wang, Xingchao" List-Id: alsa-devel@alsa-project.org On Thu, May 05, 2011 at 12:22:32AM +0800, Mark Brown wrote: > 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. OK. Will change that. -- guanqun