On Mon, 2020-05-11 at 14:56 +0900, Kuninori Morimoto wrote: > From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> > --- > sound/soc/soc-compress.c | 4 ++-- > sound/soc/soc-core.c | 4 ++-- > sound/soc/soc-dapm.c | 8 ++++---- > sound/soc/soc-pcm.c | 14 +++++++------- > 4 files changed, 15 insertions(+), 15 deletions(-) > > diff --git a/sound/soc/soc-compress.c b/sound/soc/soc-compress.c > index def3ae78b4a7..92d70e75a5a4 100644 > --- a/sound/soc/soc-compress.c > +++ b/sound/soc/soc-compress.c > @@ -231,10 +231,10 @@ static int soc_compr_free(struct > snd_compr_stream *cstream) > > snd_soc_dai_digital_mute(codec_dai, 1, cstream->direction); > > - if (!cpu_dai->active) > + if (!snd_soc_dai_activity(cpu_dai)) I have a feeling this is probably incorrect. snd_soc_dai_activity() checks for stream_active count which is different from snd_soc_dai's active member, isnt it? Thanks, Ranjani