Hi Ranjani > > @@ -843,11 +843,7 @@ static int soc_pcm_components_hw_free(struct > > snd_pcm_substream *substream, > > if (component == last) > > break; > > > > - if (!component->driver->ops || > > - !component->driver->ops->hw_free) > > - continue; > > - > > - component->driver->ops->hw_free(substream); > > + snd_soc_component_hw_free(component, substream); > Morimoto-san, > > Just wondering why we dont care about the return value here. I see that > the original code ignores the return value too but is it something that > should be addressed? Yes, indeed. Thank you for your feedback. I will fix up it at v2 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
