Hi Jon
Thank you for reporting.
I am seeing an audio regression on -next and bisect is pointing to this commit. I am seeing the following crash on boot during probe deferral of the soundcard ...
It seems timing bug. I have a plan to post below patch if my current posting patch are accepted, but it seems it is necessary immediately. I believe your issue will be solved by this patch, but can you please test it ? I will formally post it with your tested-by if it was OK.
# It will be more cleanuped in the future, # but it needs more other cleanup patches...
-------------------- Subject: [PATCH] ASoC: soc-core: care card_probed at soc_cleanup_card_resources()
soc_cleanup_card_resources() will call card->remove(), but it should be called if card->probe() or card->late_probe() are called. snd_soc_bind_card() might be error before calling card->probe() / card->late_probe(). In that time, card->remove() will be called. This patch adds card_probed parameter to judge it.
Signed-off-by: Kuninori Morimoto kuninori.morimoto.gx@renesas.com --- sound/soc/soc-core.c | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index 55014e7..b078227 100644 --- a/sound/soc/soc-core.c +++ b/sound/soc/soc-core.c @@ -1980,7 +1980,8 @@ static void __soc_setup_card_name(char *name, int len, } }
-static void soc_cleanup_card_resources(struct snd_soc_card *card) +static void soc_cleanup_card_resources(struct snd_soc_card *card, + int card_probed) { struct snd_soc_dai_link *link, *_link;
@@ -2007,7 +2008,7 @@ static void soc_cleanup_card_resources(struct snd_soc_card *card) soc_cleanup_card_debugfs(card);
/* remove the card */ - if (card->remove) + if (card_probed && card->remove) card->remove(card); }
@@ -2015,7 +2016,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) { struct snd_soc_pcm_runtime *rtd; struct snd_soc_dai_link *dai_link; - int ret, i; + int ret, i, card_probed = 0;
mutex_lock(&client_mutex); mutex_lock_nested(&card->mutex, SND_SOC_CARD_CLASS_INIT); @@ -2067,6 +2068,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) ret = card->probe(card); if (ret < 0) goto probe_end; + card_probed = 1; }
/* probe all components used by DAI links on this card */ @@ -2129,6 +2131,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) goto probe_end; } } + card_probed = 1;
snd_soc_dapm_new_widgets(card);
@@ -2145,7 +2148,7 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
probe_end: if (ret < 0) - soc_cleanup_card_resources(card); + soc_cleanup_card_resources(card, card_probed);
mutex_unlock(&card->mutex); mutex_unlock(&client_mutex); @@ -2439,11 +2442,13 @@ EXPORT_SYMBOL_GPL(snd_soc_register_card); static void snd_soc_unbind_card(struct snd_soc_card *card, bool unregister) { if (card->instantiated) { + int card_probed = 1; + card->instantiated = false; snd_soc_dapm_shutdown(card); snd_soc_flush_all_delayed_work(card);
- soc_cleanup_card_resources(card); + soc_cleanup_card_resources(card, card_probed); if (!unregister) list_add(&card->list, &unbind_card_list); } else {