Re: [PATCH v1] ASoC: soc-pcm: Fix mute and unmute control for non-dynamic DAI links

[Mohammad Rafi Shaik]

On 10/8/2025 7:16 PM, Srinivas Kandagatla wrote:
> On 10/7/25 3:33 AM, Mohammad Rafi Shaik wrote:
> > In setups where the same codec DAI is reused across multiple DAI
> > links, mute controls via `snd_soc_dai_digital_mute()` is skipped for
>
> Please explain the problem.
In Qualcomm audioreach setup, if platform dai not specified in DT, then
cpu dai using as platform and initialize as static dai-link and created
pcm device and link-dynamic == false by default.

In existing setup if dynamic==false, it's skipping trigger snd-ops and 
the codec is always on mute state.

> > non-dynamic links. The trigger operations are not invoked when
> > `dai_link->dynamic == 0`, and mute controls is currently conditioned
>
> I dont think any of the Qualcomm upstream platforms use this flag.

Yes, we are using dynamic flag and it's set to true in Qualcomm platforms.

Please check: 
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/tree/sound/soc/qcom/qdsp6/topology.c#n1093

> > only on `snd_soc_dai_mute_is_ctrled_at_trigger()`. This patch ensures
> > that mute and unmute is applied explicitly for non-dynamic links.
> How is this resolving the issue, mute on these codecs happens at trigger
> level instead of prepare.
yes agree, but if link->dynamic==false the trigger ops not getting 
called, which leading to codec always on mute state.

Thanks & Regards,
Rafi.

> --srini
> > Fixes: f0220575e65a ("ASoC: soc-dai: add flag to mute and unmute stream during trigger")
> > Cc: stable@xxxxxxxxxxxxxxx
> > Signed-off-by: Mohammad Rafi Shaik <mohammad.rafi.shaik@xxxxxxxxxxxxxxxx>
> > ---
> >   sound/soc/soc-pcm.c | 4 ++--
> >   1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/sound/soc/soc-pcm.c b/sound/soc/soc-pcm.c
> > index 2c21fd528afd..4ed829b49bc2 100644
> > --- a/sound/soc/soc-pcm.c
> > +++ b/sound/soc/soc-pcm.c
> > @@ -949,7 +949,7 @@ static int __soc_pcm_prepare(struct snd_soc_pcm_runtime *rtd,
> >   			SND_SOC_DAPM_STREAM_START);
> >   
> >   	for_each_rtd_dais(rtd, i, dai) {
> > -		if (!snd_soc_dai_mute_is_ctrled_at_trigger(dai))
> > +		if (!snd_soc_dai_mute_is_ctrled_at_trigger(dai) || !rtd->dai_link->dynamic)
> >   			snd_soc_dai_digital_mute(dai, 0, substream->stream);
> >   	}
> >   
> > @@ -1007,7 +1007,7 @@ static int soc_pcm_hw_clean(struct snd_soc_pcm_runtime *rtd,
> >   			soc_pcm_set_dai_params(dai, NULL);
> >   
> >   		if (snd_soc_dai_stream_active(dai, substream->stream) == 1) {
> > -			if (!snd_soc_dai_mute_is_ctrled_at_trigger(dai))
> > +			if (!snd_soc_dai_mute_is_ctrled_at_trigger(dai) || !rtd->dai_link->dynamic)
> >   				snd_soc_dai_digital_mute(dai, 1, substream->stream);
> >   		}
> >   	}