[PATCH 6/8] ASoC: soc-pcm: check DAI's activity more simply
Kuninori Morimoto
kuninori.morimoto.gx at renesas.com
Thu Feb 27 01:43:38 CET 2020
Hi Pierre-Louis
> >>> - if ((playback && playback_active == 1) ||
> >>> - (!playback && capture_active == 1))
> >>> + if (codec_dai->active == 1)
> >>
> >> nit-pick: we have two tests in soc-pcm.c
> >>
> >> if (codec_dai->active)
> >> if (codec_dai->active == 1)
> >>
> >> The two are functionality equivalent but it'd be good to choose one
> >> version - or possibly use 'active' as a boolean.
> >
> > In my understanding, dai->active can be 0/1/2.
>
> I see, I guess I missed this completely. Thanks Morimoto-san for the
> precision.
But, we want to use "if (codec_dai->active)" anyway.
Your review indicated my mistake.
Thank you for your help !!
Best regards
---
Kuninori Morimoto
More information about the Alsa-devel
mailing list