Re: [PATCH] ASoC: qcom: q6apm-lpass-dais: pass max number of channels to Audioreach

On 17/10/2023 18:14, Krzysztof Kozlowski wrote:

Using the params_channels() helper when setting hw_params, results in passing to Audioreach minimum number of channels valid for given hardware. This is not valid for any hardware which sets minimum channels to two and maximum to something bigger, like four channels.

Instead pass the maximum number of supported channels to allow playback of multi-channel formats.

Signed-off-by: Krzysztof Kozlowski krzysztof.kozlowski@linaro.org

sound/soc/qcom/qdsp6/q6apm-lpass-dais.c | 4 ++--

I am not sure if this is correct fix. I think sound machine drivers should just not override channels like:

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/soun...

Best regards, Krzysztof