From: Johan Hovold <johan+linaro@kernel.org>
To: Bjorn Andersson <andersson@kernel.org>
Cc: Andy Gross <agross@kernel.org>,
Konrad Dybcio <konrad.dybcio@linaro.org>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
Johan Hovold <johan+linaro@kernel.org>
Subject: [PATCH v2 0/6] arm64: dts: qcom: sound cleanups
Date: Tue, 3 Jan 2023 11:31:35 +0100 [thread overview]
Message-ID: <20230103103141.15807-1-johan+linaro@kernel.org> (raw)
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
next reply other threads:[~2023-01-03 10:31 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-03 10:31 Johan Hovold [this message]
2023-01-03 10:31 ` [PATCH v2 1/6] arm64: dts: qcom: sc8280xp: disable sound nodes Johan Hovold
2023-01-03 10:31 ` [PATCH v2 2/6] arm64: dts: qcom: sc8280xp-x13s: move vamacro node Johan Hovold
2023-01-03 13:35 ` Konrad Dybcio
2023-01-03 10:31 ` [PATCH v2 3/6] arm64: dts: qcom: sc8280xp: clean up tx-macro node Johan Hovold
2023-01-03 10:31 ` [PATCH v2 4/6] arm64: dts: qcom: sc8280xp-x13s: move wcd938x codec node Johan Hovold
2023-01-03 10:31 ` [PATCH v2 5/6] arm64: dts: qcom: sm8450-hdk: " Johan Hovold
2023-01-03 10:36 ` Johan Hovold
2023-01-03 13:36 ` Konrad Dybcio
2023-01-03 10:31 ` [PATCH v2 6/6] arm64: dts: qcom: sm8250: clean up " Johan Hovold
2023-01-03 13:37 ` Konrad Dybcio
2023-01-18 22:50 ` [PATCH v2 0/6] arm64: dts: qcom: sound cleanups Bjorn Andersson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230103103141.15807-1-johan+linaro@kernel.org \
--to=johan+linaro@kernel.org \
--cc=agross@kernel.org \
--cc=andersson@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=konrad.dybcio@linaro.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh+dt@kernel.org \
--cc=srinivas.kandagatla@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).