11 Sep
2019
11 Sep
'19
1:57 a.m.
Hi Sridharan
Thank you for your review.
-static void soc_rtd_free(struct snd_soc_pcm_runtime *rtd) -{ - if (rtd->dev_registered) { - /* we don't need to call kfree() for rtd->dev */
Morimoto-san,
I think it is useful to keep this comment when you move soc_rtd_free() to soc_free_pcm_runtime().
Yeah, indeed. In my mind, the comment /* soc_release_rtd_dev */ was almost for it. But, yes, Let's keep above comment too. will fix in v2
+ if (rtd->dev) + device_unregister(rtd->dev); /* soc_release_rtd_dev */