From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chanwoo Choi Subject: Re: [PATCH] ASoC: multi-component: SAMSUNG: Fix compile error when registering a DAI Date: Thu, 22 Jul 2010 17:05:02 +0900 Message-ID: <4C47FBAE.5040106@samsung.com> References: <4C47D3A3.9000102@samsung.com> <4C47F723.4090506@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mailout4.samsung.com (mailout4.samsung.com [203.254.224.34]) by alsa0.perex.cz (Postfix) with ESMTP id D5CA6103989 for ; Thu, 22 Jul 2010 11:12:17 +0200 (CEST) Received: from epmmp2 (mailout4.samsung.com [203.254.224.34]) by mailout4.samsung.com (Sun Java(tm) System Messaging Server 7u3-15.01 64bit (built Feb 12 2010)) with ESMTP id <0L5Y0086794EB750@mailout4.samsung.com> for alsa-devel@alsa-project.org; Thu, 22 Jul 2010 17:05:02 +0900 (KST) Received: from TNRNDGASPAPP1.tn.corp.samsungelectronics.net ([165.213.149.150]) by mmp2.samsung.com (iPlanet Messaging Server 5.2 Patch 2 (built Jul 14 2004)) with ESMTPA id <0L5Y00IOO94EY3@mmp2.samsung.com> for alsa-devel@alsa-project.org; Thu, 22 Jul 2010 17:05:02 +0900 (KST) In-reply-to: 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: Seungwhan Youn Cc: Joonyoung Shim , alsa-devel@alsa-project.org, Mark Brown , Kyungmin Park , Liam Girdwood List-Id: alsa-devel@alsa-project.org Seungwhan Youn wrote: > On Thu, Jul 22, 2010 at 4:45 PM, Chanwoo Choi wrote: >> Seungwhan Youn wrote: >>> Hi, >>> >>> On Thu, Jul 22, 2010 at 2:14 PM, Chanwoo Choi wrote: >>>> This patch modify some code which register a DAI with ASoC core >>>> according to modified registration API of DAI. >>>> >>>> Signed-off-by: Chanwoo Choi >>>> Signed-off-by: Kyungmin Park >>>> --- >>>> sound/soc/s3c24xx/aquila_wm8994.c | 3 +-- >>>> sound/soc/s3c24xx/s3c-i2s-v2.c | 2 +- >>>> 2 files changed, 2 insertions(+), 3 deletions(-) >>>> >>>> diff --git a/sound/soc/s3c24xx/aquila_wm8994.c b/sound/soc/s3c24xx/aquila_wm8994.c >>>> index 69c41eb..007ed3a 100644 >>>> --- a/sound/soc/s3c24xx/aquila_wm8994.c >>>> +++ b/sound/soc/s3c24xx/aquila_wm8994.c >>>> @@ -268,8 +268,7 @@ static int __init aquila_init(void) >>>> return -ENOMEM; >>>> >>>> /* register voice DAI here */ >>>> - ret = snd_soc_register_dai(&aquila_snd_device->dev, >>>> - 0, &voice_dai); >>>> + ret = snd_soc_register_dai(&aquila_snd_device->dev, &voice_dai); >>>> if (ret) >>>> return ret; >>>> >>>> diff --git a/sound/soc/s3c24xx/s3c-i2s-v2.c b/sound/soc/s3c24xx/s3c-i2s-v2.c >>>> index f4fbc0e..b3866d5 100644 >>>> --- a/sound/soc/s3c24xx/s3c-i2s-v2.c >>>> +++ b/sound/soc/s3c24xx/s3c-i2s-v2.c >>>> @@ -750,7 +750,7 @@ int s3c_i2sv2_register_dai(struct device *dev, int id, >>>> drv->suspend = s3c2412_i2s_suspend; >>>> drv->resume = s3c2412_i2s_resume; >>>> >>>> - return snd_soc_register_dai(dev, id, drv); >>>> + return snd_soc_register_dai(dev, drv); >>> How about s3c64xx-i2s? I think, if this patch fixes all errors in >> Sorry, I confused your comment. >> What do you mean "How about s3c64xx-i2s" ? > > Actually 'sound/soc/s3c24xx/s3c64xx-i2s.c' has same error. > > So, If this patch's purpose is to fix compile error, my opinion is > that this patch fix all the same errors in 'sound/soc/s3c24xx/'. The 'sound/soc/s3c24xx/s3c64xx-i2s.c' haven't directly called 'snd_soc_register_dai()' function. It is call 's3c_i2sv2_register_dai()' in 'sound/soc/s3c24xx/s3c-i2s-v2.c' to register a DAI. > >>> 'sound/soc/s3c24xx', >>> It'll be more nice than this. >>