Hi Pierre-Louis Thank you for your English lesson :) > > snd_soc_component_initialize() isn't initialize component->list, > > doesn't > > > but we should do it. > > This patch initialize it. > > initializes Thanks ! I will fix these, and [15/28], [23/28] too > > @@ -2641,6 +2641,10 @@ static int snd_soc_component_initialize(struct snd_soc_component *component, > > { > > struct snd_soc_dapm_context *dapm; > > + INIT_LIST_HEAD(&component->dai_list); > > + INIT_LIST_HEAD(&component->list); > > is this actually required or is this 'list' used for list management? Ahh, maybe be this is same comment for [16/28] from Ranjani. I will fix it and Subject. Thank you for your help !! Best regards --- Kuninori Morimoto _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx https://mailman.alsa-project.org/mailman/listinfo/alsa-devel