From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
To: "Sridharan, Ranjani" <ranjani.sridharan@intel.com>
Cc: Linux-ALSA <alsa-devel@alsa-project.org>,
Mark Brown <broonie@kernel.org>,
Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Subject: Re: [alsa-devel] [PATCH v2 resend 03/18] ASoC: soc-core: tidyup soc_init_dai_link()
Date: 05 Nov 2019 14:35:44 +0900 [thread overview]
Message-ID: <87k18e3mxb.wl-kuninori.morimoto.gx@renesas.com> (raw)
In-Reply-To: <CAFQqKeWGojALvi9YyzqzmSm=Cyhw+SSMSgOM0A-t4Brvtmv6yA@mail.gmail.com>
Hi Sridharan
Thank you for your review
> > > +static int soc_dai_link_sanity_check(struct snd_soc_card *card,
> > > + struct snd_soc_dai_link *link)
> > > {
> > > int i;
> > > struct snd_soc_dai_link_component *codec, *platform;
> > > @@ -1043,11 +1043,15 @@ static int soc_bind_dai_link(struct
> > > snd_soc_card *card,
> > > struct snd_soc_pcm_runtime *rtd;
> > > struct snd_soc_dai_link_component *codec, *platform;
> > > struct snd_soc_component *component;
> > > - int i;
> > > + int i, ret;
> > >
> > > if (dai_link->ignore)
> > > return 0;
> > >
> > > + ret = soc_dai_link_sanity_check(card, dai_link);
> > > + if (ret < 0)
> > > + return ret;
> > Morimoto-san,
> >
> > Should this be done after checking if soc_is_dai_link_bound() maybe?
>
> Morimoto-san,
>
> I thought about this a bit more. With you changes in the series to move bind_dai_link() to snd_soc_add_dai(), is the check
> for soc_is_dai_link_bound() needed at all?
soc_is_dai_link_bound() is no longer needed.
It will be removed in my *next* patch-set.
Is this clear answer for you ?
Thank you for your help !!
Best regards
---
Kuninori Morimoto
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
next prev parent reply other threads:[~2019-11-05 5:36 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-30 1:23 [alsa-devel] [PATCH v2 resend 00/18] ASoC: soc-core cleanup - step 4 Kuninori Morimoto
2019-10-30 1:26 ` [alsa-devel] [PATCH v2 resend 01/18] ASoC: soc-core: remove unneeded snd_soc_tplg_component_remove() Kuninori Morimoto
2019-10-30 12:57 ` [alsa-devel] Applied "ASoC: soc-core: remove unneeded snd_soc_tplg_component_remove()" to the asoc tree Mark Brown
2019-11-05 2:55 ` [alsa-devel] [PATCH v2 resend 01/18] ASoC: soc-core: remove unneeded snd_soc_tplg_component_remove() Sridharan, Ranjani
2019-11-05 4:02 ` Kuninori Morimoto
2019-10-30 1:26 ` [alsa-devel] [PATCH v2 resend 02/18] ASoC: soc-core: move soc_init_dai_link() Kuninori Morimoto
2019-11-05 2:02 ` Pierre-Louis Bossart
2019-11-05 4:09 ` Kuninori Morimoto
2019-10-30 1:26 ` [alsa-devel] [PATCH v2 resend 03/18] ASoC: soc-core: tidyup soc_init_dai_link() Kuninori Morimoto
2019-11-05 3:02 ` Ranjani Sridharan
2019-11-05 4:19 ` Kuninori Morimoto
2019-11-05 5:22 ` Sridharan, Ranjani
2019-11-05 5:35 ` Kuninori Morimoto [this message]
2019-10-30 1:26 ` [alsa-devel] [PATCH v2 resend 04/18] ASoC: soc-core: remove duplicated soc_is_dai_link_bound() Kuninori Morimoto
2019-10-30 1:26 ` [alsa-devel] [PATCH v2 resend 05/18] ASoC: soc-core: call soc_bind_dai_link() under snd_soc_add_dai_link() Kuninori Morimoto
2019-10-30 1:26 ` [alsa-devel] [PATCH v2 resend 06/18] ASoC: soc-core: add soc_unbind_dai_link() Kuninori Morimoto
2019-10-30 1:26 ` [alsa-devel] [PATCH v2 resend 07/18] ASoC: soc-core: move snd_soc_lookup_component() Kuninori Morimoto
2019-11-05 2:22 ` Pierre-Louis Bossart
2019-11-05 4:10 ` Kuninori Morimoto
2019-10-30 1:26 ` [alsa-devel] [PATCH v2 resend 08/18] ASoC: soc-core: add snd_soc_del_component_unlocked() Kuninori Morimoto
2019-10-30 1:26 ` [alsa-devel] [PATCH v2 resend 09/18] ASoC: soc-core: remove snd_soc_component_add/del() Kuninori Morimoto
2019-10-30 1:26 ` [alsa-devel] [PATCH v2 resend 10/18] ASoC: soc-core: use snd_soc_lookup_component() at snd_soc_unregister_component() Kuninori Morimoto
2019-10-30 1:27 ` [alsa-devel] [PATCH v2 resend 11/18] ASoC: soc-core: move snd_soc_register_dai() Kuninori Morimoto
2019-10-30 1:27 ` [alsa-devel] [PATCH v2 resend 12/18] ASoC: soc-core: move snd_soc_unregister_dais() Kuninori Morimoto
2019-10-30 1:27 ` [alsa-devel] [PATCH v2 resend 13/18] ASoC: soc-core: add snd_soc_unregister_dai() Kuninori Morimoto
2019-10-30 1:27 ` [alsa-devel] [PATCH v2 resend 14/18] ASoC: soc-core: have legacy_dai_naming at snd_soc_register_dai() Kuninori Morimoto
2019-10-30 1:27 ` [alsa-devel] [PATCH v2 resend 15/18] ASoC: soc-core: don't call snd_soc_dapm_new_dai_widgets() " Kuninori Morimoto
2019-11-05 3:20 ` Ranjani Sridharan
2019-11-05 4:20 ` Kuninori Morimoto
2019-10-30 1:27 ` [alsa-devel] [PATCH v2 resend 16/18] ASoC: soc-core: call snd_soc_register_dai() from snd_soc_register_dais() Kuninori Morimoto
2019-10-30 1:27 ` [alsa-devel] [PATCH v2 resend 17/18] ASoC: soc-core: remove topology specific operation Kuninori Morimoto
2019-10-30 1:27 ` [alsa-devel] [PATCH v2 resend 18/18] ASoC: soc.h: dobj is used only when SND_SOC_TOPOLOGY Kuninori Morimoto
2019-10-30 11:34 ` [alsa-devel] [PATCH v2 resend 00/18] ASoC: soc-core cleanup - step 4 Mark Brown
2019-10-31 0:46 ` Kuninori Morimoto
2019-11-05 2:38 ` Pierre-Louis Bossart
2019-11-05 4:08 ` Kuninori Morimoto
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87k18e3mxb.wl-kuninori.morimoto.gx@renesas.com \
--to=kuninori.morimoto.gx@renesas.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=ranjani.sridharan@intel.com \
--cc=ranjani.sridharan@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).