[PATCH v2 0/6] arm64: dts: qcom: sound cleanups

From: Johan Hovold
Date: Tue Jan 03 2023 - 05:31:55 EST


This series clean up the sc8280xp sound nodes somewhat, for example, by
making sure the nodes are disabled by default in the SoC dtsi.

Included are also two related cleanups for sm8450-hdk and sm8250.

Note that sound on the X13s is currently broken and depends on
out-of-tree hacks to even probe:

[ 10.719830] qcom-apm gprsvc:service:2:1: CMD timeout for [1001021] opcode
[ 10.762206] q6apm-dai 3000000.remoteproc:glink-edge:gpr:service@1:dais: Adding to iommu group 19
[ 10.857910] qcom-prm gprsvc:service:2:2: DSP returned error[100100f] 1
[ 10.858386] rx_macro: probe of 3200000.rxmacro failed with error -22
[ 10.975745] qcom-soundwire 3250000.soundwire-controller: Qualcomm Soundwire controller v1.6.0 Registered
[ 10.981406] wsa883x-codec sdw:0:0217:0202:00:1: WSA883X Version 1_1, Variant: WSA8835_V2
[ 10.986440] wsa883x-codec sdw:0:0217:0202:00:2: WSA883X Version 1_1, Variant: WSA8835_V2
[ 11.086238] qcom-soundwire 3330000.soundwire-controller: Qualcomm Soundwire controller v1.6.0 Registered
[ 11.087322] qcom-soundwire 3330000.soundwire-controller: qcom_swrm_irq_handler: SWR bus clsh detected
[ 14.342332] qcom-soundwire 3330000.soundwire-controller: swrm_wait_for_rd_fifo_avail err read underflow
[ 14.342464] soundwire sdw-master-1: trf on Slave 1 failed:-5 read addr 41 count 1
[ 14.342503] wcd9380-codec sdw:0:0217:010d:00:3: SDW_SCP_INTMASK1 write failed:-5
[ 14.342536] wcd9380-codec sdw:0:0217:010d:00:3: Slave 1 initialization failed: -5
[ 14.346734] qcom-soundwire 3330000.soundwire-controller: qcom_swrm_irq_handler: SWR bus clsh detected
[ 17.727488] wcd9380-codec sdw:0:0217:010d:00:3: Slave 1 state check1: UNATTACHED, status was 1
[ 17.839483] qcom-soundwire 3330000.soundwire-controller: qcom_swrm_irq_handler: SWR bus clsh detected
[ 17.896325] qcom-soundwire 3330000.soundwire-controller: swrm_wait_for_frame_gen_enabled: link status not disconnected
[ 17.896374] qcom-soundwire 3330000.soundwire-controller: link failed to connect
[ 20.962617] platform 3210000.soundwire-controller: deferred probe pending

The out-of-tree patches I had seen that looked related to speaker
protection had been merged for 6.2-rc1 so hopefully the above log spew
is the only downside of keeping these nodes enabled for now.

Johan


Changes in v2
- keep the sound nodes on x13s enabled despite the missing driver
support -- hopefully Srini and Krzysztof can get this sorted before
6.3 is released (Krzysztof)
- moves the x13 vamacro node (new patch)
- clean up the sm8450-hdk wcd938x node somewhat when moving it (Konrad)
- rebase on sm8250 patch which had already moved the wcd938x node
(Krzysztof)

Johan Hovold (6):
arm64: dts: qcom: sc8280xp: disable sound nodes
arm64: dts: qcom: sc8280xp-x13s: move vamacro node
arm64: dts: qcom: sc8280xp: clean up tx-macro node
arm64: dts: qcom: sc8280xp-x13s: move wcd938x codec node
arm64: dts: qcom: sm8450-hdk: move wcd938x codec node
arm64: dts: qcom: sm8250: clean up wcd938x codec node

.../qcom/sc8280xp-lenovo-thinkpad-x13s.dts | 78 ++++++++++++-------
arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 18 ++++-
arch/arm64/boot/dts/qcom/sm8250-mtp.dts | 16 ++--
arch/arm64/boot/dts/qcom/sm8450-hdk.dts | 53 +++++++------
4 files changed, 103 insertions(+), 62 deletions(-)

--
2.37.4