On 11/18/19 7:10 PM, Kuninori Morimoto wrote:
Hi Pierre-Louis
These are already tested by Intel CI, and all issues were solved.
(https://github.com/thesofproject/linux/pull/1504)
Extra Tested-by / Reviewed-by are very welcome from Intel
if you don't mind I'd like to retest this new series, it's based on a
different tip and is not exactly the same as before.
Yes, of course.
Sorry I didn't mention about it.
- /* 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;
- }
+ if (card->snd_card)
+ snd_card_disconnect_sync(card->snd_card);
snd_soc_dapm_shutdown(card); <<< not tested yet.
/* remove and free each DAI */
soc_remove_link_dais(card);
+ soc_remove_link_components(card);
Yes.
It is from
2a6f0892bda954dc2688b002060093ee0fe38528
("ASoC: soc-core: call snd_soc_dapm_shutdown() at soc_cleanup_card_resources()")
No regression detected so from the Intel side we're good with this patchset.
Thanks Morimoto-san for this comprehensive analysis, really nice work!
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
