Re: [PATCH v12 00/12] Add soundcard support for sc7280 based platforms.

From: Srinivasa Rao Mandadapu
Date: Tue Jun 07 2022 - 12:38:58 EST



On 6/7/2022 3:02 AM, Matthias Kaehlcke wrote:
Thanks for your time Matthias!!!
On Fri, May 20, 2022 at 05:48:32PM +0530, Srinivasa Rao Mandadapu wrote:
This patch set is to add bolero digital macros, WCD and maxim codecs nodes
for audio on sc7280 based platforms.

This patch set depends on:
-- https://patchwork.kernel.org/project/linux-arm-msm/list/?series=638776
-- https://patchwork.kernel.org/project/linux-arm-msm/list/?series=634597
-- https://patchwork.kernel.org/project/linux-clk/list/?series=637999
-- https://patchwork.kernel.org/project/linux-arm-msm/list/?series=638002
Another dependency (at least in terms of functionality) is:

ASoC: qcom: soundwire: Add support for controlling audio CGCR from HLOS
https://patchwork.kernel.org/patch/12853622/
This is landed today.

And then there is this:

arm64: dts: qcom: sc7280: Add lpasscore & lpassaudio clock controllers
https://patchwork.kernel.org/project/linux-arm-msm/patch/20220523100058.26241-1-quic_tdas@xxxxxxxxxxx/

A previous version (v3) of that patch already landed (9499240d15f2
"arm64: dts: qcom: sc7280: Add lpasscore & lpassaudio clock controllers"),
it is not clear to me why it is still evolving as if that weren't the
case.

I too have same doubt. the changes should be incremental and new patch. May be Taniya considered

the status in patchwork.kernel org, where still it's showing new.


From the newer version of the patch at least marking the 'lpasscc' node
as disabled is needed.

yes, agree. And the node name changed to lpasscore to lpass_core.

Included Taniya in the mail chain for further discussion.