[alsa-devel] [PATCH 23/25] ASoC: soc-component: move snd_soc_component_seq_notifier()

Kuninori Morimoto kuninori.morimoto.gx at renesas.com
Thu Jul 25 03:54:00 CEST 2019


Hi Charles

> > @@ -2736,8 +2728,7 @@ static int snd_soc_component_initialize(struct snd_soc_component *component,
> >  	dapm->bias_level = SND_SOC_BIAS_OFF;
> >  	dapm->idle_bias_off = !driver->idle_bias_on;
> >  	dapm->suspend_bias_off = driver->suspend_bias_off;
> > -	if (driver->seq_notifier)
> > -		dapm->seq_notifier = snd_soc_component_seq_notifier;
> > +	dapm->seq_notifier = snd_soc_component_seq_notifier;
> 
> If we are updating this to always set a seq_notifier would it be
> worth removing the NULL checks in soc-dapm.c? At the moment I
> guess it will end up checking twice, DAPM checks if dapm->seq_notifier
> is NULL but it never will be, then the helper checks if
> driver->seq_notifier is NULL.

Yes, indeed.
I will double-check it, and will fix it at v2

Thank you for your help !!
Best regards
---
Kuninori Morimoto


More information about the Alsa-devel mailing list