Hi Takashi-san
@@ -1965,12 +1968,6 @@ static void soc_cleanup_card_resources(struct snd_soc_card *card) { struct snd_soc_dai_link *link, *_link;
- /* free the ALSA card at first; this syncs with pending operations */
- if (card->snd_card) {
snd_card_free(card->snd_card);card->snd_card = NULL;- }
- /* remove and free each DAI */ soc_remove_link_dais(card); soc_remove_link_components(card);
@@ -1988,6 +1985,11 @@ static void soc_cleanup_card_resources(struct snd_soc_card *card) /* remove the card */ if (card->remove) card->remove(card);
- if (card->snd_card) {
snd_card_free(card->snd_card);card->snd_card = NULL;- }
This will likely break unbind again; when unbind is performed in a busy state, the code may release still-in-use resources. The rcar driver seems to have its own disconnect_sync() call so it'd work even with this change, but others wouldn't.
At least you need to call snd_card_disconnect_sync() at the first place, then release the rest at the second place.
Ahh, I didn't notice about busy state and async process. Thank you for pointing it.
Thank you for your help !! Best regards --- Kuninori Morimoto