From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Subject: Re: [PATCH 44/54 v2] ASoC: switch over to use snd_soc_register_component() on sh4 siu Date: Thu, 21 Mar 2013 11:00:30 -0600 Message-ID: <514B3CAE.9040400@wwwdotorg.org> References: <87txo96xiu.wl%kuninori.morimoto.gx@renesas.com> <8738vpf3qd.wl%kuninori.morimoto.gx@renesas.com> <87boadca4t.wl%kuninori.morimoto.gx@renesas.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from avon.wwwdotorg.org (avon.wwwdotorg.org [70.85.31.133]) by alsa0.perex.cz (Postfix) with ESMTP id 6942A265F8C for ; Thu, 21 Mar 2013 18:00:33 +0100 (CET) In-Reply-To: <87boadca4t.wl%kuninori.morimoto.gx@renesas.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Kuninori Morimoto Cc: Linux-ALSA , Lars-Peter Clausen , Mark Brown , Peter Ujfalusi , Simon , Liam Girdwood , Kuninori Morimoto List-Id: alsa-devel@alsa-project.org On 03/21/2013 04:36 AM, Kuninori Morimoto wrote: > diff --git a/sound/soc/sh/migor.c b/sound/soc/sh/migor.c > static struct snd_soc_dai_link migor_dai = { > .name = "wm8978", > .stream_name = "WM8978", > - .cpu_dai_name = "siu-i2s-dai", > + .cpu_dai_name = "siu-pcm-audio", I'm not sure what that change is for. The need for it should be described in the commit description. > diff --git a/sound/soc/sh/siu_dai.c b/sound/soc/sh/siu_dai.c > @@ -783,7 +787,8 @@ static int siu_probe(struct platform_device *pdev) > dev_set_drvdata(&pdev->dev, info); > > /* register using ARRAY version so we can keep dai name */ > - ret = snd_soc_register_dais(&pdev->dev, &siu_i2s_dai, 1); > + ret = snd_soc_register_component(&pdev->dev, &siu_i2s_component, > + &siu_i2s_dai, 1); That changes from snd_soc_register_dais() to snd_soc_register_dai(). Will that be a problem?