Re: [PATCH] ASoC: qcom: sc8280xp: configure codec sysclk for QCS615

Mohammad Rafi Shaik <[email protected]>
Newsgroups org.kernel.vger.linux-sound,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-kernel
Message-ID <[email protected]>

On 8/7/2026 6:54 AM, Le Qi wrote:
> Continuous high-amplitude noise could occur in the DA7213 microphone
> capture path after a Bluetooth out-of-range/reset event followed by
> reconnection. The noise was present in both the raw ALSA capture and
> PipeWire input, confirming that it originated before Bluetooth
> encoding.
> 
> The codec already obtains and enables MCLK through its DT clock and
> bias-level handling. However, the machine driver did not explicitly
> configure the codec sysclk during hw_params(). Enable
> codec_sysclk_set for QCS615 so that the DA7213 clock source and rate
> are configured before the codec power-up sequence.
> 
> Verified on QCS615 Talos with repeated Bluetooth disconnect and
> reconnect cycles. The noise was no longer reproducible.
> 
> Signed-off-by: Le Qi <[email protected]>
> ---
> This patch depends on:
> 
> ASoC: qcom: sc8280xp: enhance machine driver for board-specific config
> 
> Link: https://lore.kernel.org/r/[email protected]/
> 
> The prerequisite patch introduces qcs615_priv_data and the
> codec_sysclk_set field used by this change. It has not yet been merged
> into linux-next.


Drop this dependency flag, as the change has already been applied.

Thanks & Regards,
Rafi.

> 
>   sound/soc/qcom/sc8280xp.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/sound/soc/qcom/sc8280xp.c b/sound/soc/qcom/sc8280xp.c
> index cf272d0612fe..8d949e4dec5b 100644
> --- a/sound/soc/qcom/sc8280xp.c
> +++ b/sound/soc/qcom/sc8280xp.c
> @@ -306,6 +306,7 @@ static const struct snd_soc_common qcs615_priv_data = {
>   	.driver_name = "qcs615",
>   	.dapm_widgets = sc8280xp_dapm_widgets,
>   	.num_dapm_widgets = ARRAY_SIZE(sc8280xp_dapm_widgets),
> +	.codec_sysclk_set = true,
>   };
>   
>   static const struct snd_soc_common qcm6490_priv_data = {
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.