- 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.