[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <d9971ca4-1911-4204-b175-1ceeaae7c238@kernel.org>
Date: Wed, 8 Oct 2025 14:46:38 +0100
From: Srinivas Kandagatla <srini@...nel.org>
To: Mohammad Rafi Shaik <mohammad.rafi.shaik@....qualcomm.com>,
Liam Girdwood <lgirdwood@...il.com>, Mark Brown <broonie@...nel.org>,
Jaroslav Kysela <perex@...ex.cz>, Takashi Iwai <tiwai@...e.com>,
Srinivas Kandagatla <srini@...nel.org>
Cc: linux-sound@...r.kernel.org, linux-kernel@...r.kernel.org,
kernel@....qualcomm.com, prasad.kumpatla@....qualcomm.com,
ajay.nandam@....qualcomm.com, stable@...r.kernel.org
Subject: Re: [PATCH v1] ASoC: soc-pcm: Fix mute and unmute control for
non-dynamic DAI links
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.
> 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.
> 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.
--srini
>
> Fixes: f0220575e65a ("ASoC: soc-dai: add flag to mute and unmute stream during trigger")
> Cc: stable@...r.kernel.org
> Signed-off-by: Mohammad Rafi Shaik <mohammad.rafi.shaik@....qualcomm.com>
> ---
> 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);
> }
> }
Powered by blists - more mailing lists