Commit 53e947a0 authored by Kuninori Morimoto's avatar Kuninori Morimoto Committed by Mark Brown

ASoC: soc-core: merge card resources cleanup method

We need to cleanup card resources when snd_soc_instantiate_card() was
failed, or when snd_soc_unbind_card() was called.
But they are cleanuping card resources on each way.
Same code in many places makes code un-understandable.

This patch reuses soc_cleanup_card_resources() for cleanuping code
resource. Then, it makes avoiding cleanup order.
It will be called from snd_soc_instantiate_card() and
snd_soc_unbind_card().

Then, original soc_cleanup_card_resources() included
snd_soc_flush_all_delayed_work(), but it is now separated.
Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 65462e44
...@@ -2008,6 +2008,29 @@ static void soc_check_tplg_fes(struct snd_soc_card *card) ...@@ -2008,6 +2008,29 @@ static void soc_check_tplg_fes(struct snd_soc_card *card)
} }
} }
static int soc_cleanup_card_resources(struct snd_soc_card *card)
{
/* free the ALSA card at first; this syncs with pending operations */
if (card->snd_card)
snd_card_free(card->snd_card);
/* remove and free each DAI */
soc_remove_dai_links(card);
soc_remove_pcm_runtimes(card);
/* remove auxiliary devices */
soc_remove_aux_devices(card);
snd_soc_dapm_free(&card->dapm);
soc_cleanup_card_debugfs(card);
/* remove the card */
if (card->remove)
card->remove(card);
return 0;
}
static int snd_soc_instantiate_card(struct snd_soc_card *card) static int snd_soc_instantiate_card(struct snd_soc_card *card)
{ {
struct snd_soc_pcm_runtime *rtd; struct snd_soc_pcm_runtime *rtd;
...@@ -2017,6 +2040,11 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) ...@@ -2017,6 +2040,11 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
mutex_lock(&client_mutex); mutex_lock(&client_mutex);
mutex_lock_nested(&card->mutex, SND_SOC_CARD_CLASS_INIT); mutex_lock_nested(&card->mutex, SND_SOC_CARD_CLASS_INIT);
card->dapm.bias_level = SND_SOC_BIAS_OFF;
card->dapm.dev = card->dev;
card->dapm.card = card;
list_add(&card->dapm.list, &card->dapm_list);
/* check whether any platform is ignore machine FE and using topology */ /* check whether any platform is ignore machine FE and using topology */
soc_check_tplg_fes(card); soc_check_tplg_fes(card);
...@@ -2024,14 +2052,14 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) ...@@ -2024,14 +2052,14 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
for_each_card_prelinks(card, i, dai_link) { for_each_card_prelinks(card, i, dai_link) {
ret = soc_bind_dai_link(card, dai_link); ret = soc_bind_dai_link(card, dai_link);
if (ret != 0) if (ret != 0)
goto base_error; goto probe_end;
} }
/* bind aux_devs too */ /* bind aux_devs too */
for (i = 0; i < card->num_aux_devs; i++) { for (i = 0; i < card->num_aux_devs; i++) {
ret = soc_bind_aux_dev(card, i); ret = soc_bind_aux_dev(card, i);
if (ret != 0) if (ret != 0)
goto base_error; goto probe_end;
} }
/* add predefined DAI links to the list */ /* add predefined DAI links to the list */
...@@ -2045,16 +2073,11 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) ...@@ -2045,16 +2073,11 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
dev_err(card->dev, dev_err(card->dev,
"ASoC: can't create sound card for card %s: %d\n", "ASoC: can't create sound card for card %s: %d\n",
card->name, ret); card->name, ret);
goto base_error; goto probe_end;
} }
soc_init_card_debugfs(card); soc_init_card_debugfs(card);
card->dapm.bias_level = SND_SOC_BIAS_OFF;
card->dapm.dev = card->dev;
card->dapm.card = card;
list_add(&card->dapm.list, &card->dapm_list);
#ifdef CONFIG_DEBUG_FS #ifdef CONFIG_DEBUG_FS
snd_soc_dapm_debugfs_init(&card->dapm, card->debugfs_card_root); snd_soc_dapm_debugfs_init(&card->dapm, card->debugfs_card_root);
#endif #endif
...@@ -2076,7 +2099,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) ...@@ -2076,7 +2099,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
if (card->probe) { if (card->probe) {
ret = card->probe(card); ret = card->probe(card);
if (ret < 0) if (ret < 0)
goto card_probe_error; goto probe_end;
} }
/* probe all components used by DAI links on this card */ /* probe all components used by DAI links on this card */
...@@ -2087,7 +2110,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) ...@@ -2087,7 +2110,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
dev_err(card->dev, dev_err(card->dev,
"ASoC: failed to instantiate card %d\n", "ASoC: failed to instantiate card %d\n",
ret); ret);
goto probe_dai_err; goto probe_end;
} }
} }
} }
...@@ -2095,7 +2118,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) ...@@ -2095,7 +2118,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
/* probe auxiliary components */ /* probe auxiliary components */
ret = soc_probe_aux_devices(card); ret = soc_probe_aux_devices(card);
if (ret < 0) if (ret < 0)
goto probe_dai_err; goto probe_end;
/* /*
* Find new DAI links added during probing components and bind them. * Find new DAI links added during probing components and bind them.
...@@ -2107,10 +2130,10 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) ...@@ -2107,10 +2130,10 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
ret = soc_init_dai_link(card, dai_link); ret = soc_init_dai_link(card, dai_link);
if (ret) if (ret)
goto probe_dai_err; goto probe_end;
ret = soc_bind_dai_link(card, dai_link); ret = soc_bind_dai_link(card, dai_link);
if (ret) if (ret)
goto probe_dai_err; goto probe_end;
} }
/* probe all DAI links on this card */ /* probe all DAI links on this card */
...@@ -2121,7 +2144,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) ...@@ -2121,7 +2144,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
dev_err(card->dev, dev_err(card->dev,
"ASoC: failed to instantiate card %d\n", "ASoC: failed to instantiate card %d\n",
ret); ret);
goto probe_dai_err; goto probe_end;
} }
} }
} }
...@@ -2168,7 +2191,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) ...@@ -2168,7 +2191,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
if (ret < 0) { if (ret < 0) {
dev_err(card->dev, "ASoC: %s late_probe() failed: %d\n", dev_err(card->dev, "ASoC: %s late_probe() failed: %d\n",
card->name, ret); card->name, ret);
goto probe_aux_dev_err; goto probe_end;
} }
} }
...@@ -2178,33 +2201,17 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) ...@@ -2178,33 +2201,17 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
if (ret < 0) { if (ret < 0) {
dev_err(card->dev, "ASoC: failed to register soundcard %d\n", dev_err(card->dev, "ASoC: failed to register soundcard %d\n",
ret); ret);
goto probe_aux_dev_err; goto probe_end;
} }
card->instantiated = 1; card->instantiated = 1;
dapm_mark_endpoints_dirty(card); dapm_mark_endpoints_dirty(card);
snd_soc_dapm_sync(&card->dapm); snd_soc_dapm_sync(&card->dapm);
mutex_unlock(&card->mutex);
mutex_unlock(&client_mutex);
return 0;
probe_aux_dev_err:
soc_remove_aux_devices(card);
probe_dai_err:
soc_remove_dai_links(card);
card_probe_error:
if (card->remove)
card->remove(card);
snd_soc_dapm_free(&card->dapm); probe_end:
soc_cleanup_card_debugfs(card); if (ret < 0)
snd_card_free(card->snd_card); soc_cleanup_card_resources(card);
base_error:
soc_remove_pcm_runtimes(card);
mutex_unlock(&card->mutex); mutex_unlock(&card->mutex);
mutex_unlock(&client_mutex); mutex_unlock(&client_mutex);
...@@ -2233,31 +2240,6 @@ static int soc_probe(struct platform_device *pdev) ...@@ -2233,31 +2240,6 @@ static int soc_probe(struct platform_device *pdev)
return snd_soc_register_card(card); return snd_soc_register_card(card);
} }
static int soc_cleanup_card_resources(struct snd_soc_card *card)
{
/* make sure any delayed work runs */
snd_soc_flush_all_delayed_work(card);
/* free the ALSA card at first; this syncs with pending operations */
snd_card_free(card->snd_card);
/* remove and free each DAI */
soc_remove_dai_links(card);
soc_remove_pcm_runtimes(card);
/* remove auxiliary devices */
soc_remove_aux_devices(card);
snd_soc_dapm_free(&card->dapm);
soc_cleanup_card_debugfs(card);
/* remove the card */
if (card->remove)
card->remove(card);
return 0;
}
/* removes a socdev */ /* removes a socdev */
static int soc_remove(struct platform_device *pdev) static int soc_remove(struct platform_device *pdev)
{ {
...@@ -2823,6 +2805,7 @@ static void snd_soc_unbind_card(struct snd_soc_card *card, bool unregister) ...@@ -2823,6 +2805,7 @@ static void snd_soc_unbind_card(struct snd_soc_card *card, bool unregister)
if (card->instantiated) { if (card->instantiated) {
card->instantiated = false; card->instantiated = false;
snd_soc_dapm_shutdown(card); snd_soc_dapm_shutdown(card);
snd_soc_flush_all_delayed_work(card);
soc_cleanup_card_resources(card); soc_cleanup_card_resources(card);
if (!unregister) if (!unregister)
list_add(&card->list, &unbind_card_list); list_add(&card->list, &unbind_card_list);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment