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

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

[Le Qi]

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 <le.qi@oss.qualcomm.com>
---
This patch depends on:

ASoC: qcom: sc8280xp: enhance machine driver for board-specific config

Link: https://lore.kernel.org/r/20260727151230.408122-4-mohammad.rafi.shaik@oss.qualcomm.com/

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.

 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 = {
-- 
2.34.1

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

[Mohammad Rafi Shaik]

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 <le.qi@oss.qualcomm.com>
> ---
> This patch depends on:
> 
> ASoC: qcom: sc8280xp: enhance machine driver for board-specific config
> 
> Link: https://lore.kernel.org/r/20260727151230.408122-4-mohammad.rafi.shaik@oss.qualcomm.com/
> 
> 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 = {

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

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 411 bytes --]

On Fri, Aug 07, 2026 at 09:24:50AM +0800, 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.

This doesn't apply against current code, please check and resend.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]