mirror of
https://github.com/torvalds/linux.git
synced 2024-12-31 23:31:29 +00:00
ASoC: soc-core: remove duplicated soc_is_dai_link_bound()
soc_is_dai_link_bound() check will be called both *before* soc_bind_dai_link() (A), and *under* soc_bind_dai_link() (B). These are very verbose code. Let's remove one of them. * static int soc_bind_dai_link(...) { ... (B) if (soc_is_dai_link_bound(...)) { ... return 0; } ... } static int snd_soc_instantiate_card(...) { ... for_each_card_links(...) { (A) if (soc_is_dai_link_bound(...)) continue; * ret = soc_bind_dai_link(...); if (ret) goto probe_end; } ... } Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com> Link: https://lore.kernel.org/r/87a79a3jns.wl-kuninori.morimoto.gx@renesas.com Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
parent
cd3c5ad7b2
commit
95b562e57f
@ -2065,9 +2065,6 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
|
||||
* Components with topology may bring new DAIs and DAI links.
|
||||
*/
|
||||
for_each_card_links(card, dai_link) {
|
||||
if (soc_is_dai_link_bound(card, dai_link))
|
||||
continue;
|
||||
|
||||
ret = soc_bind_dai_link(card, dai_link);
|
||||
if (ret)
|
||||
goto probe_end;
|
||||
|
Loading…
Reference in New Issue
Block a user