On 25/09/2026 10:09, Srinivas Kandagatla wrote:
On 9/24/26 2:24 PM, David Heidelberg via B4 Relay wrote:
From: David Heidelberg <[email protected]>
qcom_slim_ngd_xfer_msg() silently returns success for every core
reconfiguration message, so the generic channel removal done by
slim_stream_disable() never reaches the ADSP SLIMbus master. The
master therefore keeps the channels of a closed stream. When the next
stream defines the same channel numbers again, the WCD9340 capture
completes without any error but records only zeros; removing the
channels explicitly when the stream is disabled makes it work again.
Implement the disable_stream controller op with the sequence the
downstream NGD driver sends for channel removal, CHAN_CTRL(REMOVE)
listing the stream's channels followed by RECONFIG_NOW. Like
downstream, treat -EREMOTEIO from the removal as success: it means the
controller is being restarted and the channels are gone anyway.
Based on work done by Casey Tunturi.
Assisted-by: Claude:claude-fable-5-1
Signed-off-by: David Heidelberg <[email protected]>
---
there is an identical patch on the list,
https://lore.kernel.org/linux-arm-msm/[email protected]/T/#m3b50aa43a6493f8d3b607b1607b37bf14b199f69
...and I see you just merged resent v3 today :)
I'll drop this patch from the series then.
Thank you
David
