* [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards
@ 2025-04-29 9:24 Prasad Kumpatla
2025-04-29 9:24 ` [PATCH v2 1/7] arm64: dts: qcom: qcs6490-audioreach: Add gpr node Prasad Kumpatla
` (8 more replies)
0 siblings, 9 replies; 26+ messages in thread
From: Prasad Kumpatla @ 2025-04-29 9:24 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley
Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel,
kernel, Prasad Kumpatla
Audio support is now enabled on the qcs6490-RB3Gen2 and qcm6490-idp boards.
The updates include adding the necessary audio device tree support and the required
dependencies.
Both the qcs6490-RB3Gen2 and qcm6490-idp boards are derived from the same SoC
platform. Therefore, the audio support changes are included in a single patch
set for consistency and ease of maintenance.
Changes in [v2]:
- Created dtsi file to handle common audio nodes to support Audioreach.
- Addressed the review comments.
- Link to V1 : https://lore.kernel.org/linux-arm-msm/20250317054151.6095-2-quic_pkumpatl@quicinc.com/
Mohammad Rafi Shaik (7):
arm64: dts: qcom: qcs6490-audioreach: Add gpr node
arm64: dts: qcom: sc7280: Add WSA SoundWire and LPASS support
arm64: dts: qcom: qcs6490-audioreach: Modify LPASS macros clock
settings for audioreach
arm64: dts: qcom: qcs6490-rb3gen2: Add WSA8830 speakers amplifier
arm64: dts: qcom: qcs6490-rb3gen2: Add sound card
arm64: dts: qcom: qcm6490-idp: Add WSA8830 speakers and WCD9370
headset codec
arm64: dts: qcom: qcm6490-idp: Add sound card
arch/arm64/boot/dts/qcom/qcm6490-idp.dts | 250 ++++++++++++++++++
.../boot/dts/qcom/qcs6490-audioreach.dtsi | 99 +++++++
arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 119 +++++++++
arch/arm64/boot/dts/qcom/sc7280.dtsi | 70 ++++-
4 files changed, 537 insertions(+), 1 deletion(-)
create mode 100644 arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi
base-commit: 33035b665157558254b3c21c3f049fd728e72368
--
2.17.1
^ permalink raw reply [flat|nested] 26+ messages in thread* [PATCH v2 1/7] arm64: dts: qcom: qcs6490-audioreach: Add gpr node 2025-04-29 9:24 [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards Prasad Kumpatla @ 2025-04-29 9:24 ` Prasad Kumpatla 2025-04-29 10:48 ` Konrad Dybcio 2025-04-29 9:24 ` [PATCH v2 2/7] arm64: dts: qcom: sc7280: Add WSA SoundWire and LPASS support Prasad Kumpatla ` (7 subsequent siblings) 8 siblings, 1 reply; 26+ messages in thread From: Prasad Kumpatla @ 2025-04-29 9:24 UTC (permalink / raw) To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik, Prasad Kumpatla From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Add GPR(Generic Pack router) node along with APM(Audio Process Manager) and PRM(Proxy resource Manager) audio services. Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> --- .../boot/dts/qcom/qcs6490-audioreach.dtsi | 51 +++++++++++++++++++ arch/arm64/boot/dts/qcom/sc7280.dtsi | 2 +- 2 files changed, 52 insertions(+), 1 deletion(-) create mode 100644 arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi diff --git a/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi b/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi new file mode 100644 index 000000000000..b11b9eea64c1 --- /dev/null +++ b/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi @@ -0,0 +1,51 @@ +// SPDX-License-Identifier: GPL-2.0-only +/* + * qcs6490 device tree source for Audioreach Solution. + * This file will handle the common audio device tree nodes. + * + * Copyright (c) 2025 Qualcomm Innovation Center, Inc. All rights reserved. + */ + +#include <dt-bindings/clock/qcom,lpass-sc7280.h> +#include <dt-bindings/soc/qcom,gpr.h> +#include <dt-bindings/sound/qcom,q6afe.h> +#include <dt-bindings/sound/qcom,q6dsp-lpass-ports.h> + +&remoteproc_adsp_glink { + /delete-node/ apr; + + gpr { + compatible = "qcom,gpr"; + qcom,glink-channels = "adsp_apps"; + qcom,domain = <GPR_DOMAIN_ID_ADSP>; + qcom,intents = <512 20>; + #address-cells = <1>; + #size-cells = <0>; + + q6apm: service@1 { + compatible = "qcom,q6apm"; + reg = <GPR_APM_MODULE_IID>; + #sound-dai-cells = <0>; + + q6apmdai: dais { + compatible = "qcom,q6apm-dais"; + iommus = <&apps_smmu 0x1801 0x0>; + }; + + q6apmbedai: bedais { + compatible = "qcom,q6apm-lpass-dais"; + #sound-dai-cells = <1>; + }; + }; + + q6prm: service@2 { + compatible = "qcom,q6prm"; + reg = <GPR_PRM_MODULE_IID>; + + q6prmcc: clock-controller { + compatible = "qcom,q6prm-lpass-clocks"; + #clock-cells = <2>; + }; + }; + }; +}; diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi index 8e86d75cc6b4..3453740c0d14 100644 --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi @@ -3778,7 +3778,7 @@ status = "disabled"; - glink-edge { + remoteproc_adsp_glink: glink-edge { interrupts-extended = <&ipcc IPCC_CLIENT_LPASS IPCC_MPROC_SIGNAL_GLINK_QMP IRQ_TYPE_EDGE_RISING>; -- 2.17.1 ^ permalink raw reply related [flat|nested] 26+ messages in thread
* Re: [PATCH v2 1/7] arm64: dts: qcom: qcs6490-audioreach: Add gpr node 2025-04-29 9:24 ` [PATCH v2 1/7] arm64: dts: qcom: qcs6490-audioreach: Add gpr node Prasad Kumpatla @ 2025-04-29 10:48 ` Konrad Dybcio 2025-05-08 16:16 ` Prasad Kumpatla 0 siblings, 1 reply; 26+ messages in thread From: Konrad Dybcio @ 2025-04-29 10:48 UTC (permalink / raw) To: Prasad Kumpatla, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 4/29/25 11:24 AM, Prasad Kumpatla wrote: > From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > > Add GPR(Generic Pack router) node along with > APM(Audio Process Manager) and PRM(Proxy resource > Manager) audio services. > > Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > --- [...] > +&remoteproc_adsp_glink { > + /delete-node/ apr; > + > + gpr { > + compatible = "qcom,gpr"; > + qcom,glink-channels = "adsp_apps"; > + qcom,domain = <GPR_DOMAIN_ID_ADSP>; > + qcom,intents = <512 20>; > + #address-cells = <1>; > + #size-cells = <0>; > + > + q6apm: service@1 { > + compatible = "qcom,q6apm"; > + reg = <GPR_APM_MODULE_IID>; > + #sound-dai-cells = <0>; No `qcom,protection-domain` under either of the services? Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v2 1/7] arm64: dts: qcom: qcs6490-audioreach: Add gpr node 2025-04-29 10:48 ` Konrad Dybcio @ 2025-05-08 16:16 ` Prasad Kumpatla 0 siblings, 0 replies; 26+ messages in thread From: Prasad Kumpatla @ 2025-05-08 16:16 UTC (permalink / raw) To: Konrad Dybcio, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 4/29/2025 4:18 PM, Konrad Dybcio wrote: > On 4/29/25 11:24 AM, Prasad Kumpatla wrote: >> From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >> >> Add GPR(Generic Pack router) node along with >> APM(Audio Process Manager) and PRM(Proxy resource >> Manager) audio services. >> >> Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >> --- > > [...] > >> +&remoteproc_adsp_glink { >> + /delete-node/ apr; >> + >> + gpr { >> + compatible = "qcom,gpr"; >> + qcom,glink-channels = "adsp_apps"; >> + qcom,domain = <GPR_DOMAIN_ID_ADSP>; >> + qcom,intents = <512 20>; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + q6apm: service@1 { >> + compatible = "qcom,q6apm"; >> + reg = <GPR_APM_MODULE_IID>; >> + #sound-dai-cells = <0>; > > No `qcom,protection-domain` under either of the services? With `qcom,protection-domain` facing some boot issue. Will check and update in next versions. Thanks, Prasad> > Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH v2 2/7] arm64: dts: qcom: sc7280: Add WSA SoundWire and LPASS support 2025-04-29 9:24 [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards Prasad Kumpatla 2025-04-29 9:24 ` [PATCH v2 1/7] arm64: dts: qcom: qcs6490-audioreach: Add gpr node Prasad Kumpatla @ 2025-04-29 9:24 ` Prasad Kumpatla 2025-04-29 10:51 ` Konrad Dybcio 2025-04-29 9:24 ` [PATCH v2 3/7] arm64: dts: qcom: qcs6490-audioreach: Modify LPASS macros clock settings for audioreach Prasad Kumpatla ` (6 subsequent siblings) 8 siblings, 1 reply; 26+ messages in thread From: Prasad Kumpatla @ 2025-04-29 9:24 UTC (permalink / raw) To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik, Prasad Kumpatla From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Add WSA LPASS macro Codec along with SoundWire controller. Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> --- arch/arm64/boot/dts/qcom/sc7280.dtsi | 68 ++++++++++++++++++++++++++++ 1 file changed, 68 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi index 3453740c0d14..d4aacb97a18c 100644 --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi @@ -2601,6 +2601,64 @@ status = "disabled"; }; + lpass_wsa_macro: codec@3240000 { + compatible = "qcom,sc7280-lpass-wsa-macro"; + reg = <0x0 0x03240000 0x0 0x1000>; + + clocks = <&lpass_aon LPASS_AON_CC_TX_MCLK_CLK>, + <&lpass_aon LPASS_AON_CC_TX_MCLK_2X_CLK>, + <&lpass_va_macro>; + clock-names = "mclk", "npl", "fsgen"; + + pinctrl-names = "default"; + pinctrl-0 = <&lpass_wsa_swr_clk>, <&lpass_wsa_swr_data>; + + power-domains = <&lpass_hm LPASS_CORE_CC_LPASS_CORE_HM_GDSC>, + <&lpass_aon LPASS_AON_CC_LPASS_AUDIO_HM_GDSC>; + power-domain-names = "macro", "dcodec"; + + #clock-cells = <0>; + clock-output-names = "mclk"; + #sound-dai-cells = <1>; + + status = "disabled"; + }; + + swr2: soundwire@3250000 { + compatible = "qcom,soundwire-v1.6.0"; + reg = <0x0 0x03250000 0x0 0x2000>; + + interrupts = <GIC_SPI 170 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&lpass_wsa_macro>; + clock-names = "iface"; + + resets = <&lpass_audiocc LPASS_AUDIO_SWR_WSA_CGCR>; + reset-names = "swr_audio_cgcr"; + + qcom,din-ports = <2>; + qcom,dout-ports = <6>; + + qcom,ports-sinterval-low = /bits/ 8 <0x07 0x1f 0x3f 0x07 + 0x1f 0x3f 0x0f 0x0f>; + qcom,ports-offset1 = /bits/ 8 <0x01 0x02 0x0c 0x06 0x12 0x0d 0x07 0x0a>; + qcom,ports-offset2 = /bits/ 8 <0xff 0x00 0x1f 0xff 0x00 0x1f 0x00 0x00>; + qcom,ports-hstart = /bits/ 8 <0xff 0xff 0xff 0xff 0xff 0xff 0xff 0xff>; + qcom,ports-hstop = /bits/ 8 <0xff 0xff 0xff 0xff 0xff 0xff 0xff 0xff>; + qcom,ports-word-length = /bits/ 8 <0xff 0xff 0xff 0xff 0xff 0xff 0xff 0xff>; + qcom,ports-block-pack-mode = /bits/ 8 <0xff 0xff 0x01 0xff 0xff 0x01 + 0xff 0xff>; + qcom,ports-block-group-count = /bits/ 8 <0xff 0xff 0xff 0xff 0xff 0xff + 0xff 0xff>; + qcom,ports-lane-control = /bits/ 8 <0xff 0xff 0xff 0xff 0xff 0xff + 0xff 0xff>; + + #address-cells = <2>; + #size-cells = <0>; + #sound-dai-cells = <1>; + + status = "disabled"; + }; + lpass_audiocc: clock-controller@3300000 { compatible = "qcom,sc7280-lpassaudiocc"; reg = <0 0x03300000 0 0x30000>, @@ -2807,6 +2865,16 @@ pins = "gpio1", "gpio2", "gpio14"; function = "swr_tx_data"; }; + + lpass_wsa_swr_clk: wsa-swr-clk-state { + pins = "gpio10"; + function = "wsa_swr_clk"; + }; + + lpass_wsa_swr_data: wsa-swr-data-state { + pins = "gpio11"; + function = "wsa_swr_data"; + }; }; gpu: gpu@3d00000 { -- 2.17.1 ^ permalink raw reply related [flat|nested] 26+ messages in thread
* Re: [PATCH v2 2/7] arm64: dts: qcom: sc7280: Add WSA SoundWire and LPASS support 2025-04-29 9:24 ` [PATCH v2 2/7] arm64: dts: qcom: sc7280: Add WSA SoundWire and LPASS support Prasad Kumpatla @ 2025-04-29 10:51 ` Konrad Dybcio 2025-05-08 16:31 ` Prasad Kumpatla 0 siblings, 1 reply; 26+ messages in thread From: Konrad Dybcio @ 2025-04-29 10:51 UTC (permalink / raw) To: Prasad Kumpatla, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 4/29/25 11:24 AM, Prasad Kumpatla wrote: > From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > > Add WSA LPASS macro Codec along with SoundWire controller. > > Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > --- > arch/arm64/boot/dts/qcom/sc7280.dtsi | 68 ++++++++++++++++++++++++++++ > 1 file changed, 68 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi > index 3453740c0d14..d4aacb97a18c 100644 > --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi > +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi > @@ -2601,6 +2601,64 @@ > status = "disabled"; > }; > > + lpass_wsa_macro: codec@3240000 { > + compatible = "qcom,sc7280-lpass-wsa-macro"; > + reg = <0x0 0x03240000 0x0 0x1000>; > + > + clocks = <&lpass_aon LPASS_AON_CC_TX_MCLK_CLK>, > + <&lpass_aon LPASS_AON_CC_TX_MCLK_2X_CLK>, > + <&lpass_va_macro>; > + clock-names = "mclk", "npl", "fsgen"; Please make this a vertical list > + > + pinctrl-names = "default"; > + pinctrl-0 = <&lpass_wsa_swr_clk>, <&lpass_wsa_swr_data>; property-n property-names across all changes, please [...] > + swr2: soundwire@3250000 { > + compatible = "qcom,soundwire-v1.6.0"; > + reg = <0x0 0x03250000 0x0 0x2000>; > + > + interrupts = <GIC_SPI 170 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&lpass_wsa_macro>; > + clock-names = "iface"; > + > + resets = <&lpass_audiocc LPASS_AUDIO_SWR_WSA_CGCR>; > + reset-names = "swr_audio_cgcr"; > + > + qcom,din-ports = <2>; The computer tells me it should be 3 Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v2 2/7] arm64: dts: qcom: sc7280: Add WSA SoundWire and LPASS support 2025-04-29 10:51 ` Konrad Dybcio @ 2025-05-08 16:31 ` Prasad Kumpatla 2025-05-08 17:47 ` Konrad Dybcio 0 siblings, 1 reply; 26+ messages in thread From: Prasad Kumpatla @ 2025-05-08 16:31 UTC (permalink / raw) To: Konrad Dybcio, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 4/29/2025 4:21 PM, Konrad Dybcio wrote: > On 4/29/25 11:24 AM, Prasad Kumpatla wrote: >> From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >> >> Add WSA LPASS macro Codec along with SoundWire controller. >> >> Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >> --- >> arch/arm64/boot/dts/qcom/sc7280.dtsi | 68 ++++++++++++++++++++++++++++ >> 1 file changed, 68 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi >> index 3453740c0d14..d4aacb97a18c 100644 >> --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi >> +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi >> @@ -2601,6 +2601,64 @@ >> status = "disabled"; >> }; >> >> + lpass_wsa_macro: codec@3240000 { >> + compatible = "qcom,sc7280-lpass-wsa-macro"; >> + reg = <0x0 0x03240000 0x0 0x1000>; >> + >> + clocks = <&lpass_aon LPASS_AON_CC_TX_MCLK_CLK>, >> + <&lpass_aon LPASS_AON_CC_TX_MCLK_2X_CLK>, >> + <&lpass_va_macro>; >> + clock-names = "mclk", "npl", "fsgen"; > > Please make this a vertical list Ack> >> + >> + pinctrl-names = "default"; >> + pinctrl-0 = <&lpass_wsa_swr_clk>, <&lpass_wsa_swr_data>; > > property-n > property-names > > across all changes, please > > [...] Ack> >> + swr2: soundwire@3250000 { >> + compatible = "qcom,soundwire-v1.6.0"; >> + reg = <0x0 0x03250000 0x0 0x2000>; >> + >> + interrupts = <GIC_SPI 170 IRQ_TYPE_LEVEL_HIGH>; >> + clocks = <&lpass_wsa_macro>; >> + clock-names = "iface"; >> + >> + resets = <&lpass_audiocc LPASS_AUDIO_SWR_WSA_CGCR>; >> + reset-names = "swr_audio_cgcr"; >> + >> + qcom,din-ports = <2>; > > The computer tells me it should be 3 For swr2 - soundwire version-V1.6.0 contains din-ports as 2 only. Please refer below link https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/tree/arch/arm64/boot/dts/qcom/sc8280xp.dtsi#n2931 Thanks, Prasad> > > Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v2 2/7] arm64: dts: qcom: sc7280: Add WSA SoundWire and LPASS support 2025-05-08 16:31 ` Prasad Kumpatla @ 2025-05-08 17:47 ` Konrad Dybcio 0 siblings, 0 replies; 26+ messages in thread From: Konrad Dybcio @ 2025-05-08 17:47 UTC (permalink / raw) To: Prasad Kumpatla, Konrad Dybcio, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 5/8/25 6:31 PM, Prasad Kumpatla wrote: > > > On 4/29/2025 4:21 PM, Konrad Dybcio wrote: >> On 4/29/25 11:24 AM, Prasad Kumpatla wrote: >>> From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >>> >>> Add WSA LPASS macro Codec along with SoundWire controller. >>> >>> Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >>> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >>> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >>> --- [...] >>> + swr2: soundwire@3250000 { >>> + compatible = "qcom,soundwire-v1.6.0"; >>> + reg = <0x0 0x03250000 0x0 0x2000>; >>> + >>> + interrupts = <GIC_SPI 170 IRQ_TYPE_LEVEL_HIGH>; >>> + clocks = <&lpass_wsa_macro>; >>> + clock-names = "iface"; >>> + >>> + resets = <&lpass_audiocc LPASS_AUDIO_SWR_WSA_CGCR>; >>> + reset-names = "swr_audio_cgcr"; >>> + >>> + qcom,din-ports = <2>; >> >> The computer tells me it should be 3 > For swr2 - soundwire version-V1.6.0 contains din-ports as 2 only. Please refer below link > https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/tree/arch/arm64/boot/dts/qcom/sc8280xp.dtsi#n2931 I'm referring to what I've seen in the internal data as parameters, please doublecheck Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH v2 3/7] arm64: dts: qcom: qcs6490-audioreach: Modify LPASS macros clock settings for audioreach 2025-04-29 9:24 [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards Prasad Kumpatla 2025-04-29 9:24 ` [PATCH v2 1/7] arm64: dts: qcom: qcs6490-audioreach: Add gpr node Prasad Kumpatla 2025-04-29 9:24 ` [PATCH v2 2/7] arm64: dts: qcom: sc7280: Add WSA SoundWire and LPASS support Prasad Kumpatla @ 2025-04-29 9:24 ` Prasad Kumpatla 2025-04-29 10:54 ` Konrad Dybcio 2025-04-29 9:24 ` [PATCH v2 4/7] arm64: dts: qcom: qcs6490-rb3gen2: Add WSA8830 speakers amplifier Prasad Kumpatla ` (5 subsequent siblings) 8 siblings, 1 reply; 26+ messages in thread From: Prasad Kumpatla @ 2025-04-29 9:24 UTC (permalink / raw) To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik, Prasad Kumpatla From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Modify and enable WSA, VA, RX and TX lpass macros and lpass_tlmm clock settings. For audioreach solution mclk, npl and fsgen clocks are enabled through the q6prm clock driver. Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> --- .../boot/dts/qcom/qcs6490-audioreach.dtsi | 48 +++++++++++++++++++ 1 file changed, 48 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi b/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi index b11b9eea64c1..f3859d805ea7 100644 --- a/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi +++ b/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi @@ -11,6 +11,54 @@ #include <dt-bindings/sound/qcom,q6afe.h> #include <dt-bindings/sound/qcom,q6dsp-lpass-ports.h> +&lpass_rx_macro { + /delete-property/ power-domains; + /delete-property/ power-domain-names; + clocks = <&q6prmcc LPASS_CLK_ID_TX_CORE_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&q6prmcc LPASS_CLK_ID_TX_CORE_NPL_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&q6prmcc LPASS_HW_MACRO_VOTE LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&q6prmcc LPASS_HW_DCODEC_VOTE LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&lpass_va_macro>; + clock-names = "mclk", "npl", "macro", "dcodec", "fsgen"; +}; + +&lpass_tlmm { + clocks = <&q6prmcc LPASS_HW_MACRO_VOTE LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&q6prmcc LPASS_HW_DCODEC_VOTE LPASS_CLK_ATTRIBUTE_COUPLE_NO>; + clock-names = "core", "audio"; +}; + +&lpass_tx_macro { + /delete-property/ power-domains; + /delete-property/ power-domain-names; + clocks = <&q6prmcc LPASS_CLK_ID_TX_CORE_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&q6prmcc LPASS_CLK_ID_TX_CORE_NPL_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&q6prmcc LPASS_HW_MACRO_VOTE LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&q6prmcc LPASS_HW_DCODEC_VOTE LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&lpass_va_macro>; + clock-names = "mclk", "npl", "macro", "dcodec", "fsgen"; +}; + +&lpass_va_macro { + /delete-property/ power-domains; + /delete-property/ power-domain-names; + clocks = <&q6prmcc LPASS_CLK_ID_VA_CORE_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&q6prmcc LPASS_HW_MACRO_VOTE LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&q6prmcc LPASS_HW_DCODEC_VOTE LPASS_CLK_ATTRIBUTE_COUPLE_NO>; + clock-names = "mclk", "macro", "dcodec"; +}; + +&lpass_wsa_macro { + /delete-property/ power-domains; + /delete-property/ power-domain-names; + clocks = <&q6prmcc LPASS_CLK_ID_TX_CORE_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&q6prmcc LPASS_CLK_ID_TX_CORE_NPL_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&q6prmcc LPASS_HW_MACRO_VOTE LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&q6prmcc LPASS_HW_DCODEC_VOTE LPASS_CLK_ATTRIBUTE_COUPLE_NO>, + <&lpass_va_macro>; + clock-names = "mclk", "npl", "macro", "dcodec", "fsgen"; +}; + &remoteproc_adsp_glink { /delete-node/ apr; -- 2.17.1 ^ permalink raw reply related [flat|nested] 26+ messages in thread
* Re: [PATCH v2 3/7] arm64: dts: qcom: qcs6490-audioreach: Modify LPASS macros clock settings for audioreach 2025-04-29 9:24 ` [PATCH v2 3/7] arm64: dts: qcom: qcs6490-audioreach: Modify LPASS macros clock settings for audioreach Prasad Kumpatla @ 2025-04-29 10:54 ` Konrad Dybcio 2025-05-08 16:38 ` Prasad Kumpatla 0 siblings, 1 reply; 26+ messages in thread From: Konrad Dybcio @ 2025-04-29 10:54 UTC (permalink / raw) To: Prasad Kumpatla, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 4/29/25 11:24 AM, Prasad Kumpatla wrote: > From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > > Modify and enable WSA, VA, RX and TX lpass macros and lpass_tlmm clock > settings. For audioreach solution mclk, npl and fsgen clocks are enabled > through the q6prm clock driver. > > Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > --- > .../boot/dts/qcom/qcs6490-audioreach.dtsi | 48 +++++++++++++++++++ > 1 file changed, 48 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi b/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi > index b11b9eea64c1..f3859d805ea7 100644 > --- a/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi > +++ b/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi > @@ -11,6 +11,54 @@ > #include <dt-bindings/sound/qcom,q6afe.h> > #include <dt-bindings/sound/qcom,q6dsp-lpass-ports.h> > > +&lpass_rx_macro { > + /delete-property/ power-domains; > + /delete-property/ power-domain-names; > + clocks = <&q6prmcc LPASS_CLK_ID_TX_CORE_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, > + <&q6prmcc LPASS_CLK_ID_TX_CORE_NPL_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, TX -> RX? [...] > +&lpass_wsa_macro { > + /delete-property/ power-domains; > + /delete-property/ power-domain-names; > + clocks = <&q6prmcc LPASS_CLK_ID_TX_CORE_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, > + <&q6prmcc LPASS_CLK_ID_TX_CORE_NPL_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, TX -> WSA? Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v2 3/7] arm64: dts: qcom: qcs6490-audioreach: Modify LPASS macros clock settings for audioreach 2025-04-29 10:54 ` Konrad Dybcio @ 2025-05-08 16:38 ` Prasad Kumpatla 2025-05-08 16:45 ` Konrad Dybcio 0 siblings, 1 reply; 26+ messages in thread From: Prasad Kumpatla @ 2025-05-08 16:38 UTC (permalink / raw) To: Konrad Dybcio, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 4/29/2025 4:24 PM, Konrad Dybcio wrote: > On 4/29/25 11:24 AM, Prasad Kumpatla wrote: >> From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >> >> Modify and enable WSA, VA, RX and TX lpass macros and lpass_tlmm clock >> settings. For audioreach solution mclk, npl and fsgen clocks are enabled >> through the q6prm clock driver. >> >> Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >> --- >> .../boot/dts/qcom/qcs6490-audioreach.dtsi | 48 +++++++++++++++++++ >> 1 file changed, 48 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi b/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi >> index b11b9eea64c1..f3859d805ea7 100644 >> --- a/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi >> +++ b/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi >> @@ -11,6 +11,54 @@ >> #include <dt-bindings/sound/qcom,q6afe.h> >> #include <dt-bindings/sound/qcom,q6dsp-lpass-ports.h> >> >> +&lpass_rx_macro { >> + /delete-property/ power-domains; >> + /delete-property/ power-domain-names; >> + clocks = <&q6prmcc LPASS_CLK_ID_TX_CORE_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, >> + <&q6prmcc LPASS_CLK_ID_TX_CORE_NPL_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, > > TX -> RX? > > [...] > >> +&lpass_wsa_macro { >> + /delete-property/ power-domains; >> + /delete-property/ power-domain-names; >> + clocks = <&q6prmcc LPASS_CLK_ID_TX_CORE_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, >> + <&q6prmcc LPASS_CLK_ID_TX_CORE_NPL_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, > > TX -> WSA? For qcs6490 RX drives clk from TX CORE which is mandated from DSP side, Unlike sm8450 dedicated core clocks. Core TX clk is used for both RX and WSA as per DSP recommendations. Thanks, Prasad> > Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v2 3/7] arm64: dts: qcom: qcs6490-audioreach: Modify LPASS macros clock settings for audioreach 2025-05-08 16:38 ` Prasad Kumpatla @ 2025-05-08 16:45 ` Konrad Dybcio 0 siblings, 0 replies; 26+ messages in thread From: Konrad Dybcio @ 2025-05-08 16:45 UTC (permalink / raw) To: Prasad Kumpatla, Konrad Dybcio, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 5/8/25 6:38 PM, Prasad Kumpatla wrote: > > > On 4/29/2025 4:24 PM, Konrad Dybcio wrote: >> On 4/29/25 11:24 AM, Prasad Kumpatla wrote: >>> From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >>> >>> Modify and enable WSA, VA, RX and TX lpass macros and lpass_tlmm clock >>> settings. For audioreach solution mclk, npl and fsgen clocks are enabled >>> through the q6prm clock driver. >>> >>> Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >>> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >>> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >>> --- >>> .../boot/dts/qcom/qcs6490-audioreach.dtsi | 48 +++++++++++++++++++ >>> 1 file changed, 48 insertions(+) >>> >>> diff --git a/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi b/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi >>> index b11b9eea64c1..f3859d805ea7 100644 >>> --- a/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi >>> +++ b/arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi >>> @@ -11,6 +11,54 @@ >>> #include <dt-bindings/sound/qcom,q6afe.h> >>> #include <dt-bindings/sound/qcom,q6dsp-lpass-ports.h> >>> +&lpass_rx_macro { >>> + /delete-property/ power-domains; >>> + /delete-property/ power-domain-names; >>> + clocks = <&q6prmcc LPASS_CLK_ID_TX_CORE_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, >>> + <&q6prmcc LPASS_CLK_ID_TX_CORE_NPL_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, >> >> TX -> RX? >> >> [...] >> >>> +&lpass_wsa_macro { >>> + /delete-property/ power-domains; >>> + /delete-property/ power-domain-names; >>> + clocks = <&q6prmcc LPASS_CLK_ID_TX_CORE_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, >>> + <&q6prmcc LPASS_CLK_ID_TX_CORE_NPL_MCLK LPASS_CLK_ATTRIBUTE_COUPLE_NO>, >> >> TX -> WSA? > > For qcs6490 RX drives clk from TX CORE which is mandated from DSP side, Unlike sm8450 dedicated core clocks. Core TX clk is used for both RX and WSA as per DSP recommendations. Please leave a comment as it's not obvious Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH v2 4/7] arm64: dts: qcom: qcs6490-rb3gen2: Add WSA8830 speakers amplifier 2025-04-29 9:24 [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards Prasad Kumpatla ` (2 preceding siblings ...) 2025-04-29 9:24 ` [PATCH v2 3/7] arm64: dts: qcom: qcs6490-audioreach: Modify LPASS macros clock settings for audioreach Prasad Kumpatla @ 2025-04-29 9:24 ` Prasad Kumpatla 2025-04-29 10:57 ` Konrad Dybcio 2025-04-29 9:24 ` [PATCH v2 5/7] arm64: dts: qcom: qcs6490-rb3gen2: Add sound card Prasad Kumpatla ` (4 subsequent siblings) 8 siblings, 1 reply; 26+ messages in thread From: Prasad Kumpatla @ 2025-04-29 9:24 UTC (permalink / raw) To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik, Prasad Kumpatla From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Add nodes for WSA8830 speakers amplifier on qcs6490-rb3gen2 board. Enable lpass_wsa and lpass_va macros along with pinctrl settings for audio. Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> --- arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 70 ++++++++++++++++++++ 1 file changed, 70 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts index 5fbcd48f2e2d..43cbdbe43afd 100644 --- a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts +++ b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts @@ -19,6 +19,7 @@ #include "pm7325.dtsi" #include "pm8350c.dtsi" #include "pmk8350.dtsi" +#include "qcs6490-audioreach.dtsi" /delete-node/ &ipa_fw_mem; /delete-node/ &rmtfs_mem; @@ -765,6 +766,44 @@ }; }; +&lpass_dmic01_clk { + drive-strength = <8>; + bias-disable; +}; + +&lpass_dmic01_data { + bias-pull-down; +}; + +&lpass_dmic23_clk { + drive-strength = <8>; + bias-disable; +}; + +&lpass_dmic23_data { + bias-pull-down; +}; + +&lpass_va_macro { + status = "okay"; +}; + +&lpass_wsa_macro { + status = "okay"; +}; + +&lpass_wsa_swr_clk { + drive-strength = <2>; + slew-rate = <1>; + bias-disable; +}; + +&lpass_wsa_swr_data { + drive-strength = <2>; + slew-rate = <1>; + bias-bus-hold; +}; + &mdss { status = "okay"; }; @@ -1039,6 +1078,31 @@ status = "okay"; }; +&swr2 { + status = "okay"; + left_spkr: speaker@0,1 { + compatible = "sdw10217020200"; + reg = <0 1>; + powerdown-gpios = <&tlmm 158 GPIO_ACTIVE_LOW>; + #sound-dai-cells = <0>; + sound-name-prefix = "SpkrLeft"; + #thermal-sensor-cells = <0>; + vdd-supply = <&vreg_l18b_1p8>; + qcom,port-mapping = <1 2 3 7>; + }; + + right_spkr: speaker@0,2 { + compatible = "sdw10217020200"; + reg = <0 2>; + powerdown-gpios = <&tlmm 158 GPIO_ACTIVE_LOW>; + #sound-dai-cells = <0>; + sound-name-prefix = "SpkrRight"; + #thermal-sensor-cells = <0>; + vdd-supply = <&vreg_l18b_1p8>; + qcom,port-mapping = <4 5 6 8>; + }; +}; + &tlmm { gpio-reserved-ranges = <32 2>, /* ADSP */ <48 4>; /* NFC */ @@ -1095,6 +1159,12 @@ */ bias-pull-up; }; + + sw_ctrl: sw-ctrl-state { + pins = "gpio86"; + function = "gpio"; + bias-pull-down; + }; }; &uart5 { -- 2.17.1 ^ permalink raw reply related [flat|nested] 26+ messages in thread
* Re: [PATCH v2 4/7] arm64: dts: qcom: qcs6490-rb3gen2: Add WSA8830 speakers amplifier 2025-04-29 9:24 ` [PATCH v2 4/7] arm64: dts: qcom: qcs6490-rb3gen2: Add WSA8830 speakers amplifier Prasad Kumpatla @ 2025-04-29 10:57 ` Konrad Dybcio 2025-05-08 16:43 ` Prasad Kumpatla 0 siblings, 1 reply; 26+ messages in thread From: Konrad Dybcio @ 2025-04-29 10:57 UTC (permalink / raw) To: Prasad Kumpatla, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 4/29/25 11:24 AM, Prasad Kumpatla wrote: > From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > > Add nodes for WSA8830 speakers amplifier on qcs6490-rb3gen2 board. > > Enable lpass_wsa and lpass_va macros along with pinctrl settings > for audio. > > Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > --- > arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 70 ++++++++++++++++++++ > 1 file changed, 70 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts > index 5fbcd48f2e2d..43cbdbe43afd 100644 > --- a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts > +++ b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts > @@ -19,6 +19,7 @@ > #include "pm7325.dtsi" > #include "pm8350c.dtsi" > #include "pmk8350.dtsi" > +#include "qcs6490-audioreach.dtsi" > > /delete-node/ &ipa_fw_mem; > /delete-node/ &rmtfs_mem; > @@ -765,6 +766,44 @@ > }; > }; > > +&lpass_dmic01_clk { > + drive-strength = <8>; > + bias-disable; > +}; > + > +&lpass_dmic01_data { > + bias-pull-down; > +}; > + > +&lpass_dmic23_clk { > + drive-strength = <8>; > + bias-disable; > +}; > + > +&lpass_dmic23_data { > + bias-pull-down; > +}; > + > +&lpass_va_macro { > + status = "okay"; > +}; > + > +&lpass_wsa_macro { > + status = "okay"; > +}; > + > +&lpass_wsa_swr_clk { > + drive-strength = <2>; > + slew-rate = <1>; > + bias-disable; > +}; > + > +&lpass_wsa_swr_data { > + drive-strength = <2>; > + slew-rate = <1>; > + bias-bus-hold; > +}; These properties should likely be moved to sc7280.dtsi > + > &mdss { > status = "okay"; > }; > @@ -1039,6 +1078,31 @@ > status = "okay"; > }; > > +&swr2 { > + status = "okay"; Please keep a newline between the properties and first subnode [...] > &tlmm { > gpio-reserved-ranges = <32 2>, /* ADSP */ > <48 4>; /* NFC */ > @@ -1095,6 +1159,12 @@ > */ > bias-pull-up; > }; > + > + sw_ctrl: sw-ctrl-state { Not only non-descript, this node is not used Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v2 4/7] arm64: dts: qcom: qcs6490-rb3gen2: Add WSA8830 speakers amplifier 2025-04-29 10:57 ` Konrad Dybcio @ 2025-05-08 16:43 ` Prasad Kumpatla 0 siblings, 0 replies; 26+ messages in thread From: Prasad Kumpatla @ 2025-05-08 16:43 UTC (permalink / raw) To: Konrad Dybcio, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 4/29/2025 4:27 PM, Konrad Dybcio wrote: > On 4/29/25 11:24 AM, Prasad Kumpatla wrote: >> From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >> >> Add nodes for WSA8830 speakers amplifier on qcs6490-rb3gen2 board. >> >> Enable lpass_wsa and lpass_va macros along with pinctrl settings >> for audio. >> >> Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >> --- >> arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 70 ++++++++++++++++++++ >> 1 file changed, 70 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts >> index 5fbcd48f2e2d..43cbdbe43afd 100644 >> --- a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts >> +++ b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts >> @@ -19,6 +19,7 @@ >> #include "pm7325.dtsi" >> #include "pm8350c.dtsi" >> #include "pmk8350.dtsi" >> +#include "qcs6490-audioreach.dtsi" >> >> /delete-node/ &ipa_fw_mem; >> /delete-node/ &rmtfs_mem; >> @@ -765,6 +766,44 @@ >> }; >> }; >> >> +&lpass_dmic01_clk { >> + drive-strength = <8>; >> + bias-disable; >> +}; >> + >> +&lpass_dmic01_data { >> + bias-pull-down; >> +}; >> + >> +&lpass_dmic23_clk { >> + drive-strength = <8>; >> + bias-disable; >> +}; >> + >> +&lpass_dmic23_data { >> + bias-pull-down; >> +}; >> + >> +&lpass_va_macro { >> + status = "okay"; >> +}; >> + >> +&lpass_wsa_macro { >> + status = "okay"; >> +}; >> + >> +&lpass_wsa_swr_clk { >> + drive-strength = <2>; >> + slew-rate = <1>; >> + bias-disable; >> +}; >> + >> +&lpass_wsa_swr_data { >> + drive-strength = <2>; >> + slew-rate = <1>; >> + bias-bus-hold; >> +}; > > These properties should likely be moved to sc7280.dtsi Ack> >> + >> &mdss { >> status = "okay"; >> }; >> @@ -1039,6 +1078,31 @@ >> status = "okay"; >> }; >> >> +&swr2 { >> + status = "okay"; > > Please keep a newline between the properties and first subnode Ack> > [...] > >> &tlmm { >> gpio-reserved-ranges = <32 2>, /* ADSP */ >> <48 4>; /* NFC */ >> @@ -1095,6 +1159,12 @@ >> */ >> bias-pull-up; >> }; >> + >> + sw_ctrl: sw-ctrl-state { > > Not only non-descript, this node is not used Will check and update Thanks, Prasad> > Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH v2 5/7] arm64: dts: qcom: qcs6490-rb3gen2: Add sound card 2025-04-29 9:24 [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards Prasad Kumpatla ` (3 preceding siblings ...) 2025-04-29 9:24 ` [PATCH v2 4/7] arm64: dts: qcom: qcs6490-rb3gen2: Add WSA8830 speakers amplifier Prasad Kumpatla @ 2025-04-29 9:24 ` Prasad Kumpatla 2025-04-29 10:58 ` Konrad Dybcio 2025-04-29 9:24 ` [PATCH v2 6/7] arm64: dts: qcom: qcm6490-idp: Add WSA8830 speakers and WCD9370 headset codec Prasad Kumpatla ` (3 subsequent siblings) 8 siblings, 1 reply; 26+ messages in thread From: Prasad Kumpatla @ 2025-04-29 9:24 UTC (permalink / raw) To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik, Prasad Kumpatla From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Add the sound card node with tested playback over WSA8835 speakers and digital on-board mics. Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> --- arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 49 ++++++++++++++++++++ 1 file changed, 49 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts index 43cbdbe43afd..9cfe648e29bb 100644 --- a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts +++ b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts @@ -1078,6 +1078,55 @@ status = "okay"; }; +&sound { + compatible = "qcom,qcs6490-rb3gen2-sndcard"; + model = "qcs6490-rb3gen2-snd-card"; + + audio-routing = + "SpkrLeft IN", "WSA_SPK1 OUT", + "SpkrRight IN", "WSA_SPK2 OUT", + "VA DMIC0", "vdd-micb", + "VA DMIC1", "vdd-micb", + "VA DMIC2", "vdd-micb", + "VA DMIC3", "vdd-micb"; + + #address-cells = <1>; + #size-cells = <0>; + + wsa-dai-link { + link-name = "WSA Playback"; + + codec { + sound-dai = <&left_spkr>, <&right_spkr>, <&swr2 0>, + <&lpass_wsa_macro 0>; + }; + + cpu { + sound-dai = <&q6apmbedai WSA_CODEC_DMA_RX_0>; + }; + + platform { + sound-dai = <&q6apm>; + }; + }; + + va-dai-link { + link-name = "VA Capture"; + + codec { + sound-dai = <&lpass_va_macro 0>; + }; + + cpu { + sound-dai = <&q6apmbedai VA_CODEC_DMA_TX_0>; + }; + + platform { + sound-dai = <&q6apm>; + }; + }; +}; + &swr2 { status = "okay"; left_spkr: speaker@0,1 { -- 2.17.1 ^ permalink raw reply related [flat|nested] 26+ messages in thread
* Re: [PATCH v2 5/7] arm64: dts: qcom: qcs6490-rb3gen2: Add sound card 2025-04-29 9:24 ` [PATCH v2 5/7] arm64: dts: qcom: qcs6490-rb3gen2: Add sound card Prasad Kumpatla @ 2025-04-29 10:58 ` Konrad Dybcio 2025-05-08 16:44 ` Prasad Kumpatla 0 siblings, 1 reply; 26+ messages in thread From: Konrad Dybcio @ 2025-04-29 10:58 UTC (permalink / raw) To: Prasad Kumpatla, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 4/29/25 11:24 AM, Prasad Kumpatla wrote: > From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > > Add the sound card node with tested playback over WSA8835 speakers > and digital on-board mics. > > Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > --- > arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 49 ++++++++++++++++++++ > 1 file changed, 49 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts > index 43cbdbe43afd..9cfe648e29bb 100644 > --- a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts > +++ b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts > @@ -1078,6 +1078,55 @@ > status = "okay"; > }; > > +&sound { > + compatible = "qcom,qcs6490-rb3gen2-sndcard"; > + model = "qcs6490-rb3gen2-snd-card"; > + > + audio-routing = > + "SpkrLeft IN", "WSA_SPK1 OUT", Please unbreak the line and align the entries [...] > + "SpkrRight IN", "WSA_SPK2 OUT", > + "VA DMIC0", "vdd-micb", > + "VA DMIC1", "vdd-micb", > + "VA DMIC2", "vdd-micb", > + "VA DMIC3", "vdd-micb"; > + > + #address-cells = <1>; > + #size-cells = <0>; > + > + wsa-dai-link { > + link-name = "WSA Playback"; > + > + codec { > + sound-dai = <&left_spkr>, <&right_spkr>, <&swr2 0>, > + <&lpass_wsa_macro 0>; Please align the <s Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v2 5/7] arm64: dts: qcom: qcs6490-rb3gen2: Add sound card 2025-04-29 10:58 ` Konrad Dybcio @ 2025-05-08 16:44 ` Prasad Kumpatla 0 siblings, 0 replies; 26+ messages in thread From: Prasad Kumpatla @ 2025-05-08 16:44 UTC (permalink / raw) To: Konrad Dybcio, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 4/29/2025 4:28 PM, Konrad Dybcio wrote: > On 4/29/25 11:24 AM, Prasad Kumpatla wrote: >> From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >> >> Add the sound card node with tested playback over WSA8835 speakers >> and digital on-board mics. >> >> Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >> --- >> arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 49 ++++++++++++++++++++ >> 1 file changed, 49 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts >> index 43cbdbe43afd..9cfe648e29bb 100644 >> --- a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts >> +++ b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts >> @@ -1078,6 +1078,55 @@ >> status = "okay"; >> }; >> >> +&sound { >> + compatible = "qcom,qcs6490-rb3gen2-sndcard"; >> + model = "qcs6490-rb3gen2-snd-card"; >> + >> + audio-routing = >> + "SpkrLeft IN", "WSA_SPK1 OUT", > > Please unbreak the line and align the entries > > [...] Ack> >> + "SpkrRight IN", "WSA_SPK2 OUT", >> + "VA DMIC0", "vdd-micb", >> + "VA DMIC1", "vdd-micb", >> + "VA DMIC2", "vdd-micb", >> + "VA DMIC3", "vdd-micb"; >> + >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + wsa-dai-link { >> + link-name = "WSA Playback"; >> + >> + codec { >> + sound-dai = <&left_spkr>, <&right_spkr>, <&swr2 0>, >> + <&lpass_wsa_macro 0>; > > Please align the <s Ack> > Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH v2 6/7] arm64: dts: qcom: qcm6490-idp: Add WSA8830 speakers and WCD9370 headset codec 2025-04-29 9:24 [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards Prasad Kumpatla ` (4 preceding siblings ...) 2025-04-29 9:24 ` [PATCH v2 5/7] arm64: dts: qcom: qcs6490-rb3gen2: Add sound card Prasad Kumpatla @ 2025-04-29 9:24 ` Prasad Kumpatla 2025-04-29 11:01 ` Konrad Dybcio 2025-04-29 9:24 ` [PATCH v2 7/7] arm64: dts: qcom: qcm6490-idp: Add sound card Prasad Kumpatla ` (2 subsequent siblings) 8 siblings, 1 reply; 26+ messages in thread From: Prasad Kumpatla @ 2025-04-29 9:24 UTC (permalink / raw) To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik, Prasad Kumpatla From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Add nodes for WSA8830 speakers and WCD9370 headset codec on qcm6490-idp board. Enable lpass macros along with audio support pin controls. Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> --- arch/arm64/boot/dts/qcom/qcm6490-idp.dts | 162 +++++++++++++++++++++++ 1 file changed, 162 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts index 7a155ef6492e..1a59080cbfaf 100644 --- a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts +++ b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts @@ -18,6 +18,7 @@ #include "pm7325.dtsi" #include "pm8350c.dtsi" #include "pmk8350.dtsi" +#include "qcs6490-audioreach.dtsi" /delete-node/ &ipa_fw_mem; /delete-node/ &rmtfs_mem; @@ -169,6 +170,30 @@ regulator-min-microvolt = <3700000>; regulator-max-microvolt = <3700000>; }; + + wcd9370: audio-codec-0 { + compatible = "qcom,wcd9370-codec"; + + pinctrl-0 = <&wcd_reset_n>; + pinctrl-1 = <&wcd_reset_n_sleep>; + pinctrl-names = "default", "sleep"; + + reset-gpios = <&tlmm 83 GPIO_ACTIVE_HIGH>; + + vdd-buck-supply = <&vreg_l17b_1p7>; + vdd-rxtx-supply = <&vreg_l18b_1p8>; + vdd-px-supply = <&vreg_l18b_1p8>; + vdd-mic-bias-supply = <&vreg_bob_3p296>; + + qcom,micbias1-microvolt = <1800000>; + qcom,micbias2-microvolt = <1800000>; + qcom,micbias3-microvolt = <1800000>; + + qcom,rx-device = <&wcd937x_rx>; + qcom,tx-device = <&wcd937x_tx>; + + #sound-dai-cells = <1>; + }; }; &apps_rsc { @@ -536,6 +561,76 @@ firmware-name = "qcom/qcm6490/a660_zap.mbn"; }; +&lpass_dmic01_clk { + drive-strength = <8>; + bias-disable; +}; + +&lpass_dmic01_data { + bias-pull-down; +}; + +&lpass_dmic23_clk { + drive-strength = <8>; + bias-disable; +}; + +&lpass_dmic23_data { + bias-pull-down; +}; + +&lpass_rx_macro { + status = "okay"; +}; + +&lpass_rx_swr_clk { + drive-strength = <2>; + slew-rate = <1>; + bias-disable; +}; + +&lpass_rx_swr_data { + drive-strength = <2>; + slew-rate = <1>; + bias-bus-hold; +}; + +&lpass_tx_macro { + status = "okay"; +}; + +&lpass_tx_swr_clk { + drive-strength = <2>; + slew-rate = <1>; + bias-disable; +}; + +&lpass_tx_swr_data { + drive-strength = <2>; + slew-rate = <1>; + bias-bus-hold; +}; + +&lpass_va_macro { + status = "okay"; +}; + +&lpass_wsa_macro { + status = "okay"; +}; + +&lpass_wsa_swr_clk { + drive-strength = <2>; + slew-rate = <1>; + bias-disable; +}; + +&lpass_wsa_swr_data { + drive-strength = <2>; + slew-rate = <1>; + bias-bus-hold; +}; + &mdss { status = "okay"; }; @@ -716,6 +811,54 @@ cd-gpios = <&tlmm 91 GPIO_ACTIVE_LOW>; }; +&swr0 { + status = "okay"; + + wcd937x_rx: codec@0,4 { + compatible = "sdw20217010a00"; + reg = <0 4>; + qcom,rx-port-mapping = <1 2 3 4 5>; + qcom,rx-channel-mapping = /bits/ 8 <1 2 1 1 2 1 1 2>; + }; +}; + +&swr1 { + status = "okay"; + + wcd937x_tx: codec@0,3 { + compatible = "sdw20217010a00"; + reg = <0 3>; + qcom,tx-port-mapping = <1 1 2 3>; + qcom,tx-channel-mapping = /bits/ 8 <1 2 1 1 2 3 3 4 1 2 3 4>; + }; +}; + +&swr2 { + status = "okay"; + + left_spkr: speaker@0,1 { + compatible = "sdw10217020200"; + reg = <0 1>; + powerdown-gpios = <&tlmm 63 GPIO_ACTIVE_LOW>; + #sound-dai-cells = <0>; + sound-name-prefix = "SpkrLeft"; + #thermal-sensor-cells = <0>; + vdd-supply = <&vreg_l18b_1p8>; + qcom,port-mapping = <1 2 3 7>; + }; + + right_spkr: speaker@0,2 { + compatible = "sdw10217020200"; + reg = <0 2>; + powerdown-gpios = <&tlmm 62 GPIO_ACTIVE_LOW>; + #sound-dai-cells = <0>; + sound-name-prefix = "SpkrRight"; + #thermal-sensor-cells = <0>; + vdd-supply = <&vreg_l18b_1p8>; + qcom,port-mapping = <4 5 6 8>; + }; +}; + &tlmm { gpio-reserved-ranges = <32 2>, /* ADSP */ <48 4>; /* NFC */ @@ -725,6 +868,25 @@ function = "gpio"; bias-pull-up; }; + + sw_ctrl: sw-ctrl-state { + pins = "gpio86"; + function = "gpio"; + bias-pull-down; + }; + + wcd_reset_n: wcd-reset-n-state { + pins = "gpio83"; + function = "gpio"; + drive-strength = <8>; + }; + + wcd_reset_n_sleep: wcd-reset-n-sleep-state { + pins = "gpio83"; + function = "gpio"; + drive-strength = <8>; + bias-disable; + }; }; &uart5 { -- 2.17.1 ^ permalink raw reply related [flat|nested] 26+ messages in thread
* Re: [PATCH v2 6/7] arm64: dts: qcom: qcm6490-idp: Add WSA8830 speakers and WCD9370 headset codec 2025-04-29 9:24 ` [PATCH v2 6/7] arm64: dts: qcom: qcm6490-idp: Add WSA8830 speakers and WCD9370 headset codec Prasad Kumpatla @ 2025-04-29 11:01 ` Konrad Dybcio 2025-05-08 17:01 ` Prasad Kumpatla 0 siblings, 1 reply; 26+ messages in thread From: Konrad Dybcio @ 2025-04-29 11:01 UTC (permalink / raw) To: Prasad Kumpatla, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 4/29/25 11:24 AM, Prasad Kumpatla wrote: > From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > > Add nodes for WSA8830 speakers and WCD9370 headset codec > on qcm6490-idp board. > > Enable lpass macros along with audio support pin controls. > > Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > --- > arch/arm64/boot/dts/qcom/qcm6490-idp.dts | 162 +++++++++++++++++++++++ > 1 file changed, 162 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts > index 7a155ef6492e..1a59080cbfaf 100644 > --- a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts > +++ b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts > @@ -18,6 +18,7 @@ > #include "pm7325.dtsi" > #include "pm8350c.dtsi" > #include "pmk8350.dtsi" > +#include "qcs6490-audioreach.dtsi" > > /delete-node/ &ipa_fw_mem; > /delete-node/ &rmtfs_mem; > @@ -169,6 +170,30 @@ > regulator-min-microvolt = <3700000>; > regulator-max-microvolt = <3700000>; > }; > + > + wcd9370: audio-codec-0 { > + compatible = "qcom,wcd9370-codec"; > + > + pinctrl-0 = <&wcd_reset_n>; > + pinctrl-1 = <&wcd_reset_n_sleep>; > + pinctrl-names = "default", "sleep"; Does audio work for you? For inexplicable reasons, it didn't for me on rb2 when the sleep state was defined > + > + reset-gpios = <&tlmm 83 GPIO_ACTIVE_HIGH>; > + > + vdd-buck-supply = <&vreg_l17b_1p7>; > + vdd-rxtx-supply = <&vreg_l18b_1p8>; > + vdd-px-supply = <&vreg_l18b_1p8>; > + vdd-mic-bias-supply = <&vreg_bob_3p296>; > + > + qcom,micbias1-microvolt = <1800000>; > + qcom,micbias2-microvolt = <1800000>; > + qcom,micbias3-microvolt = <1800000>; > + > + qcom,rx-device = <&wcd937x_rx>; > + qcom,tx-device = <&wcd937x_tx>; > + > + #sound-dai-cells = <1>; > + }; > }; > > &apps_rsc { > @@ -536,6 +561,76 @@ > firmware-name = "qcom/qcm6490/a660_zap.mbn"; > }; > > +&lpass_dmic01_clk { > + drive-strength = <8>; > + bias-disable; > +}; > + > +&lpass_dmic01_data { > + bias-pull-down; As a testament to these definitions belonging in the soc dtsi, you added them in the file you included already. [...] > &tlmm { > gpio-reserved-ranges = <32 2>, /* ADSP */ > <48 4>; /* NFC */ > @@ -725,6 +868,25 @@ > function = "gpio"; > bias-pull-up; > }; > + > + sw_ctrl: sw-ctrl-state { > + pins = "gpio86"; > + function = "gpio"; > + bias-pull-down; > + }; Again, unused > + > + wcd_reset_n: wcd-reset-n-state { > + pins = "gpio83"; > + function = "gpio"; > + drive-strength = <8>; Since the definition is otherwise identical to the sleep state, you should define the (other) bias type that should be set when active. Konrad > + }; > + > + wcd_reset_n_sleep: wcd-reset-n-sleep-state { > + pins = "gpio83"; > + function = "gpio"; > + drive-strength = <8>; > + bias-disable; > + }; > }; > > &uart5 { ^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v2 6/7] arm64: dts: qcom: qcm6490-idp: Add WSA8830 speakers and WCD9370 headset codec 2025-04-29 11:01 ` Konrad Dybcio @ 2025-05-08 17:01 ` Prasad Kumpatla 2025-05-08 17:46 ` Konrad Dybcio 0 siblings, 1 reply; 26+ messages in thread From: Prasad Kumpatla @ 2025-05-08 17:01 UTC (permalink / raw) To: Konrad Dybcio, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 4/29/2025 4:31 PM, Konrad Dybcio wrote: > On 4/29/25 11:24 AM, Prasad Kumpatla wrote: >> From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >> >> Add nodes for WSA8830 speakers and WCD9370 headset codec >> on qcm6490-idp board. >> >> Enable lpass macros along with audio support pin controls. >> >> Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >> --- >> arch/arm64/boot/dts/qcom/qcm6490-idp.dts | 162 +++++++++++++++++++++++ >> 1 file changed, 162 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts >> index 7a155ef6492e..1a59080cbfaf 100644 >> --- a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts >> +++ b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts >> @@ -18,6 +18,7 @@ >> #include "pm7325.dtsi" >> #include "pm8350c.dtsi" >> #include "pmk8350.dtsi" >> +#include "qcs6490-audioreach.dtsi" >> >> /delete-node/ &ipa_fw_mem; >> /delete-node/ &rmtfs_mem; >> @@ -169,6 +170,30 @@ >> regulator-min-microvolt = <3700000>; >> regulator-max-microvolt = <3700000>; >> }; >> + >> + wcd9370: audio-codec-0 { >> + compatible = "qcom,wcd9370-codec"; >> + >> + pinctrl-0 = <&wcd_reset_n>; >> + pinctrl-1 = <&wcd_reset_n_sleep>; >> + pinctrl-names = "default", "sleep"; > > Does audio work for you? For inexplicable reasons, it didn't for me > on rb2 when the sleep state was defined > For Qcm6490-IDP board Audio is working fine, Not sure about rb2, Could you please provide more details about rb2 ?>> + >> + reset-gpios = <&tlmm 83 GPIO_ACTIVE_HIGH>; >> + >> + vdd-buck-supply = <&vreg_l17b_1p7>; >> + vdd-rxtx-supply = <&vreg_l18b_1p8>; >> + vdd-px-supply = <&vreg_l18b_1p8>; >> + vdd-mic-bias-supply = <&vreg_bob_3p296>; >> + >> + qcom,micbias1-microvolt = <1800000>; >> + qcom,micbias2-microvolt = <1800000>; >> + qcom,micbias3-microvolt = <1800000>; >> + >> + qcom,rx-device = <&wcd937x_rx>; >> + qcom,tx-device = <&wcd937x_tx>; >> + >> + #sound-dai-cells = <1>; >> + }; >> }; >> >> &apps_rsc { >> @@ -536,6 +561,76 @@ >> firmware-name = "qcom/qcm6490/a660_zap.mbn"; >> }; >> >> +&lpass_dmic01_clk { >> + drive-strength = <8>; >> + bias-disable; >> +}; >> + >> +&lpass_dmic01_data { >> + bias-pull-down; > > As a testament to these definitions belonging in the soc dtsi, you > added them in the file you included already. > > [...] Ack, Will move to soc dtsi file.> >> &tlmm { >> gpio-reserved-ranges = <32 2>, /* ADSP */ >> <48 4>; /* NFC */ >> @@ -725,6 +868,25 @@ >> function = "gpio"; >> bias-pull-up; >> }; >> + >> + sw_ctrl: sw-ctrl-state { >> + pins = "gpio86"; >> + function = "gpio"; >> + bias-pull-down; >> + }; > > Again, unused Ack> >> + >> + wcd_reset_n: wcd-reset-n-state { >> + pins = "gpio83"; >> + function = "gpio"; >> + drive-strength = <8>; > > Since the definition is otherwise identical to the sleep state, > you should define the (other) bias type that should be set when > active. > Taken the reference from sc7280, which is working fine. Link for reference : https://elixir.bootlin.com/linux/v6.15-rc5/source/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi#L841 Will cross check and modify if required. Thanks, Prasad> Konrad > > + }; >> + >> + wcd_reset_n_sleep: wcd-reset-n-sleep-state { >> + pins = "gpio83"; >> + function = "gpio"; >> + drive-strength = <8>; >> + bias-disable; >> + }; >> }; >> >> &uart5 { ^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v2 6/7] arm64: dts: qcom: qcm6490-idp: Add WSA8830 speakers and WCD9370 headset codec 2025-05-08 17:01 ` Prasad Kumpatla @ 2025-05-08 17:46 ` Konrad Dybcio 0 siblings, 0 replies; 26+ messages in thread From: Konrad Dybcio @ 2025-05-08 17:46 UTC (permalink / raw) To: Prasad Kumpatla, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 5/8/25 7:01 PM, Prasad Kumpatla wrote: > > > On 4/29/2025 4:31 PM, Konrad Dybcio wrote: >> On 4/29/25 11:24 AM, Prasad Kumpatla wrote: >>> From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >>> >>> Add nodes for WSA8830 speakers and WCD9370 headset codec >>> on qcm6490-idp board. >>> >>> Enable lpass macros along with audio support pin controls. >>> >>> Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> >>> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >>> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> >>> --- >>> arch/arm64/boot/dts/qcom/qcm6490-idp.dts | 162 +++++++++++++++++++++++ >>> 1 file changed, 162 insertions(+) >>> >>> diff --git a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts >>> index 7a155ef6492e..1a59080cbfaf 100644 >>> --- a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts >>> +++ b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts >>> @@ -18,6 +18,7 @@ >>> #include "pm7325.dtsi" >>> #include "pm8350c.dtsi" >>> #include "pmk8350.dtsi" >>> +#include "qcs6490-audioreach.dtsi" >>> /delete-node/ &ipa_fw_mem; >>> /delete-node/ &rmtfs_mem; >>> @@ -169,6 +170,30 @@ >>> regulator-min-microvolt = <3700000>; >>> regulator-max-microvolt = <3700000>; >>> }; >>> + >>> + wcd9370: audio-codec-0 { >>> + compatible = "qcom,wcd9370-codec"; >>> + >>> + pinctrl-0 = <&wcd_reset_n>; >>> + pinctrl-1 = <&wcd_reset_n_sleep>; >>> + pinctrl-names = "default", "sleep"; >> >> Does audio work for you? For inexplicable reasons, it didn't for me >> on rb2 when the sleep state was defined >> > For Qcm6490-IDP board Audio is working fine, Not sure about rb2, Could you please provide more details about rb2 ? I just mentioned it as something to keep in mind. Someone else has taken over that work since. [...] >>> + wcd_reset_n: wcd-reset-n-state { >>> + pins = "gpio83"; >>> + function = "gpio"; >>> + drive-strength = <8>; >> >> Since the definition is otherwise identical to the sleep state, >> you should define the (other) bias type that should be set when >> active. >> > Taken the reference from sc7280, which is working fine. > Link for reference : https://elixir.bootlin.com/linux/v6.15-rc5/source/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi#L841 > > Will cross check and modify if required. sure it will work fine, but in the same spirit typing 147 as 1+1+1+... will work fine as well, please doublecheck :D Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH v2 7/7] arm64: dts: qcom: qcm6490-idp: Add sound card 2025-04-29 9:24 [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards Prasad Kumpatla ` (5 preceding siblings ...) 2025-04-29 9:24 ` [PATCH v2 6/7] arm64: dts: qcom: qcm6490-idp: Add WSA8830 speakers and WCD9370 headset codec Prasad Kumpatla @ 2025-04-29 9:24 ` Prasad Kumpatla 2025-04-29 11:02 ` Konrad Dybcio 2025-04-29 14:44 ` [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards Rob Herring (Arm) 2025-05-08 14:42 ` Alexey Klimov 8 siblings, 1 reply; 26+ messages in thread From: Prasad Kumpatla @ 2025-04-29 9:24 UTC (permalink / raw) To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik, Prasad Kumpatla From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Add the sound card node with tested playback over WSA8835 speakers, digital on-board mics along with wcd9370 headset playabck and record. Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> --- arch/arm64/boot/dts/qcom/qcm6490-idp.dts | 88 ++++++++++++++++++++++++ 1 file changed, 88 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts index 1a59080cbfaf..a83044020ead 100644 --- a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts +++ b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts @@ -811,6 +811,94 @@ cd-gpios = <&tlmm 91 GPIO_ACTIVE_LOW>; }; +&sound { + compatible = "qcom,qcm6490-idp-sndcard"; + model = "qcm6490-idp-snd-card"; + + audio-routing = + "SpkrLeft IN", "WSA_SPK1 OUT", + "SpkrRight IN", "WSA_SPK2 OUT", + "IN1_HPHL", "HPHL_OUT", + "IN2_HPHR", "HPHR_OUT", + "AMIC2", "MIC BIAS2", + "TX DMIC0", "MIC BIAS1", + "TX DMIC1", "MIC BIAS2", + "TX DMIC2", "MIC BIAS3", + "TX SWR_ADC1", "ADC2_OUTPUT", + "VA DMIC0", "VA MIC BIAS3", + "VA DMIC1", "VA MIC BIAS3", + "VA DMIC2", "VA MIC BIAS1", + "VA DMIC3", "VA MIC BIAS1"; + + #address-cells = <1>; + #size-cells = <0>; + + wsa-dai-link { + link-name = "WSA Playback"; + + codec { + sound-dai = <&left_spkr>, <&right_spkr>, <&swr2 0>, + <&lpass_wsa_macro 0>; + }; + + cpu { + sound-dai = <&q6apmbedai WSA_CODEC_DMA_RX_0>; + }; + + platform { + sound-dai = <&q6apm>; + }; + }; + + wcd-playback-dai-link { + link-name = "WCD Playback"; + + codec { + sound-dai = <&wcd9370 0>, <&swr0 0>, <&lpass_rx_macro 0>; + }; + + cpu { + sound-dai = <&q6apmbedai RX_CODEC_DMA_RX_0>; + }; + + platform { + sound-dai = <&q6apm>; + }; + }; + + wcd-capture-dai-link { + link-name = "WCD Capture"; + + codec { + sound-dai = <&wcd9370 1>, <&swr1 0>, <&lpass_tx_macro 0>; + }; + + cpu { + sound-dai = <&q6apmbedai TX_CODEC_DMA_TX_3>; + }; + + platform { + sound-dai = <&q6apm>; + }; + }; + + va-dai-link { + link-name = "VA Capture"; + + codec { + sound-dai = <&lpass_va_macro 0>; + }; + + cpu { + sound-dai = <&q6apmbedai VA_CODEC_DMA_TX_0>; + }; + + platform { + sound-dai = <&q6apm>; + }; + }; +}; + &swr0 { status = "okay"; -- 2.17.1 ^ permalink raw reply related [flat|nested] 26+ messages in thread
* Re: [PATCH v2 7/7] arm64: dts: qcom: qcm6490-idp: Add sound card 2025-04-29 9:24 ` [PATCH v2 7/7] arm64: dts: qcom: qcm6490-idp: Add sound card Prasad Kumpatla @ 2025-04-29 11:02 ` Konrad Dybcio 0 siblings, 0 replies; 26+ messages in thread From: Konrad Dybcio @ 2025-04-29 11:02 UTC (permalink / raw) To: Prasad Kumpatla, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel, Mohammad Rafi Shaik On 4/29/25 11:24 AM, Prasad Kumpatla wrote: > From: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > > Add the sound card node with tested playback over WSA8835 speakers, > digital on-board mics along with wcd9370 headset playabck and record. > > Signed-off-by: Mohammad Rafi Shaik <quic_mohs@quicinc.com> > Co-developed-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > Signed-off-by: Prasad Kumpatla <quic_pkumpatl@quicinc.com> > --- similar comments to patch 5 Konrad ^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards 2025-04-29 9:24 [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards Prasad Kumpatla ` (6 preceding siblings ...) 2025-04-29 9:24 ` [PATCH v2 7/7] arm64: dts: qcom: qcm6490-idp: Add sound card Prasad Kumpatla @ 2025-04-29 14:44 ` Rob Herring (Arm) 2025-05-08 14:42 ` Alexey Klimov 8 siblings, 0 replies; 26+ messages in thread From: Rob Herring (Arm) @ 2025-04-29 14:44 UTC (permalink / raw) To: Prasad Kumpatla Cc: Bjorn Andersson, Konrad Dybcio, cros-qcom-dts-watchers, linux-arm-msm, kernel, Krzysztof Kozlowski, linux-kernel, Conor Dooley, devicetree On Tue, 29 Apr 2025 14:54:23 +0530, Prasad Kumpatla wrote: > Audio support is now enabled on the qcs6490-RB3Gen2 and qcm6490-idp boards. > The updates include adding the necessary audio device tree support and the required > dependencies. > > Both the qcs6490-RB3Gen2 and qcm6490-idp boards are derived from the same SoC > platform. Therefore, the audio support changes are included in a single patch > set for consistency and ease of maintenance. > > Changes in [v2]: > - Created dtsi file to handle common audio nodes to support Audioreach. > - Addressed the review comments. > - Link to V1 : https://lore.kernel.org/linux-arm-msm/20250317054151.6095-2-quic_pkumpatl@quicinc.com/ > > Mohammad Rafi Shaik (7): > arm64: dts: qcom: qcs6490-audioreach: Add gpr node > arm64: dts: qcom: sc7280: Add WSA SoundWire and LPASS support > arm64: dts: qcom: qcs6490-audioreach: Modify LPASS macros clock > settings for audioreach > arm64: dts: qcom: qcs6490-rb3gen2: Add WSA8830 speakers amplifier > arm64: dts: qcom: qcs6490-rb3gen2: Add sound card > arm64: dts: qcom: qcm6490-idp: Add WSA8830 speakers and WCD9370 > headset codec > arm64: dts: qcom: qcm6490-idp: Add sound card > > arch/arm64/boot/dts/qcom/qcm6490-idp.dts | 250 ++++++++++++++++++ > .../boot/dts/qcom/qcs6490-audioreach.dtsi | 99 +++++++ > arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 119 +++++++++ > arch/arm64/boot/dts/qcom/sc7280.dtsi | 70 ++++- > 4 files changed, 537 insertions(+), 1 deletion(-) > create mode 100644 arch/arm64/boot/dts/qcom/qcs6490-audioreach.dtsi > > > base-commit: 33035b665157558254b3c21c3f049fd728e72368 > -- > 2.17.1 > > > My bot found new DTB warnings on the .dts files added or changed in this series. Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings are fixed by another series. Ultimately, it is up to the platform maintainer whether these warnings are acceptable or not. No need to reply unless the platform maintainer has comments. If you already ran DT checks and didn't see these error(s), then make sure dt-schema is up to date: pip3 install dtschema --upgrade This patch series was applied (using b4) to base: Base: using specified base-commit 33035b665157558254b3c21c3f049fd728e72368 If this is not the correct base, please add 'base-commit' tag (or use b4 which does this automatically) New warnings running 'make CHECK_DTBS=y for arch/arm64/boot/dts/qcom/' for 20250429092430.21477-1-quic_pkumpatl@quicinc.com: arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[224, 8], [224, 7], [225]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[224, 8], [224, 7], [225]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dtb: codec@3370000 (qcom,sc7280-lpass-va-macro): clock-names: ['mclk', 'macro', 'dcodec'] is too long from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-va-macro.yaml# arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dtb: codec@3370000 (qcom,sc7280-lpass-va-macro): clocks: [[224, 56, 1], [224, 102, 1], [224, 103, 1]] is too long from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-va-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-crd-pro.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-crd-pro.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-crd-pro.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[232, 8], [232, 7], [233]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-crd-pro.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-crd-pro.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[232, 8], [232, 7], [233]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[232, 8], [232, 7], [233]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[232, 8], [232, 7], [233]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dtb: pinctrl@33c0000 (qcom,sc7280-lpass-lpi-pinctrl): Unevaluated properties are not allowed ('clock-names', 'clocks' were unexpected) from schema $id: http://devicetree.org/schemas/pinctrl/qcom,sc7280-lpass-lpi-pinctrl.yaml# arch/arm64/boot/dts/qcom/qcm6490-idp.dtb: codec@3370000 (qcom,sc7280-lpass-va-macro): clock-names: ['mclk', 'macro', 'dcodec'] is too long from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-va-macro.yaml# arch/arm64/boot/dts/qcom/qcm6490-idp.dtb: codec@3370000 (qcom,sc7280-lpass-va-macro): clocks: [[203, 56, 1], [203, 102, 1], [203, 103, 1]] is too long from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-va-macro.yaml# arch/arm64/boot/dts/qcom/qcm6490-idp.dtb: pinctrl@33c0000 (qcom,sc7280-lpass-lpi-pinctrl): Unevaluated properties are not allowed ('clock-names', 'clocks' were unexpected) from schema $id: http://devicetree.org/schemas/pinctrl/qcom,sc7280-lpass-lpi-pinctrl.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-crd.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-crd.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-crd.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[232, 8], [232, 7], [233]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-crd.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-crd.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[232, 8], [232, 7], [233]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[236, 8], [236, 7], [237]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[236, 8], [236, 7], [237]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-idp2.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-idp2.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-idp2.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[221, 8], [221, 7], [222]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-idp2.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-idp2.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[221, 8], [221, 7], [222]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dtb: pmic@2 (qcom,pm8350c): pwm:nvmem: [[360, 361]] is too short from schema $id: http://devicetree.org/schemas/mfd/qcom,spmi-pmic.yaml# arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dtb: pwm (qcom,pm8350c-pwm): nvmem: [[360, 361]] is too short from schema $id: http://devicetree.org/schemas/leds/leds-qcom-lpg.yaml# arch/arm64/boot/dts/qcom/qcm6490-idp.dtb: sound (qcom,qcm6490-idp-sndcard): '#address-cells', '#size-cells' do not match any of the regexes: '.*-dai-link$', '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/sound/qcom,sm8250.yaml# arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dtb: sound (qcom,qcs6490-rb3gen2-sndcard): '#address-cells', '#size-cells' do not match any of the regexes: '.*-dai-link$', '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/sound/qcom,sm8250.yaml# arch/arm64/boot/dts/qcom/sc7280-idp.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-idp.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-idp.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[216, 8], [216, 7], [217]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-idp.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-idp.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[216, 8], [216, 7], [217]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcm6490-idp.dtb: audio-codec-0 (qcom,wcd9370-codec): 'qcom,micbias4-microvolt' is a required property from schema $id: http://devicetree.org/schemas/sound/qcom,wcd937x.yaml# arch/arm64/boot/dts/qcom/qcm6490-idp.dtb: audio-codec-0 (qcom,wcd9370-codec): Unevaluated properties are not allowed ('qcom,micbias1-microvolt', 'qcom,micbias2-microvolt', 'qcom,micbias3-microvolt', 'qcom,rx-device', 'qcom,tx-device', 'reset-gpios', 'vdd-buck-supply', 'vdd-mic-bias-supply', 'vdd-rxtx-supply' were unexpected) from schema $id: http://devicetree.org/schemas/sound/qcom,wcd937x.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[236, 8], [236, 7], [237]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[236, 8], [236, 7], [237]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[232, 8], [232, 7], [233]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[232, 8], [232, 7], [233]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-herobrine-r1.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-herobrine-r1.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-herobrine-r1.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[235, 8], [235, 7], [236]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-herobrine-r1.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-herobrine-r1.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[235, 8], [235, 7], [236]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-nvme-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-nvme-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-nvme-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[232, 8], [232, 7], [233]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-nvme-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-nvme-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[232, 8], [232, 7], [233]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-nvme.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-nvme.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-nvme.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[232, 8], [232, 7], [233]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-nvme.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie-nvme.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[232, 8], [232, 7], [233]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[218, 8], [218, 7], [219]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[218, 8], [218, 7], [219]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[233, 8], [233, 7], [234]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[233, 8], [233, 7], [234]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-crd-r3.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-crd-r3.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-crd-r3.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[224, 8], [224, 7], [225]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-crd-r3.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-crd-r3.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[224, 8], [224, 7], [225]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[223, 8], [223, 7], [224]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[223, 8], [223, 7], [224]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[233, 8], [233, 7], [234]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1-lte.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[233, 8], [233, 7], [234]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r0.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names:2: 'macro' was expected from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r0.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r0.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[233, 8], [233, 7], [234]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r0.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clock-names: ['mclk', 'npl', 'fsgen'] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r0.dtb: codec@3240000 (qcom,sc7280-lpass-wsa-macro): clocks: [[233, 8], [233, 7], [234]] is too short from schema $id: http://devicetree.org/schemas/sound/qcom,lpass-wsa-macro.yaml# ^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards 2025-04-29 9:24 [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards Prasad Kumpatla ` (7 preceding siblings ...) 2025-04-29 14:44 ` [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards Rob Herring (Arm) @ 2025-05-08 14:42 ` Alexey Klimov 8 siblings, 0 replies; 26+ messages in thread From: Alexey Klimov @ 2025-05-08 14:42 UTC (permalink / raw) To: Prasad Kumpatla, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel, kernel On Tue Apr 29, 2025 at 10:24 AM BST, Prasad Kumpatla wrote: > Audio support is now enabled on the qcs6490-RB3Gen2 and qcm6490-idp boards. > The updates include adding the necessary audio device tree support and the required > dependencies. > > Both the qcs6490-RB3Gen2 and qcm6490-idp boards are derived from the same SoC > platform. Therefore, the audio support changes are included in a single patch > set for consistency and ease of maintenance. > > Changes in [v2]: > - Created dtsi file to handle common audio nodes to support Audioreach. > - Addressed the review comments. > - Link to V1 : https://lore.kernel.org/linux-arm-msm/20250317054151.6095-2-quic_pkumpatl@quicinc.com/ > > Mohammad Rafi Shaik (7): > arm64: dts: qcom: qcs6490-audioreach: Add gpr node > arm64: dts: qcom: sc7280: Add WSA SoundWire and LPASS support > arm64: dts: qcom: qcs6490-audioreach: Modify LPASS macros clock > settings for audioreach > arm64: dts: qcom: qcs6490-rb3gen2: Add WSA8830 speakers amplifier > arm64: dts: qcom: qcs6490-rb3gen2: Add sound card > arm64: dts: qcom: qcm6490-idp: Add WSA8830 speakers and WCD9370 > headset codec > arm64: dts: qcom: qcm6490-idp: Add sound card Do you have topology file and mixers commands (or UCM profile) to test this? Thanks, Alexey ^ permalink raw reply [flat|nested] 26+ messages in thread
end of thread, other threads:[~2025-05-08 17:47 UTC | newest] Thread overview: 26+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2025-04-29 9:24 [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards Prasad Kumpatla 2025-04-29 9:24 ` [PATCH v2 1/7] arm64: dts: qcom: qcs6490-audioreach: Add gpr node Prasad Kumpatla 2025-04-29 10:48 ` Konrad Dybcio 2025-05-08 16:16 ` Prasad Kumpatla 2025-04-29 9:24 ` [PATCH v2 2/7] arm64: dts: qcom: sc7280: Add WSA SoundWire and LPASS support Prasad Kumpatla 2025-04-29 10:51 ` Konrad Dybcio 2025-05-08 16:31 ` Prasad Kumpatla 2025-05-08 17:47 ` Konrad Dybcio 2025-04-29 9:24 ` [PATCH v2 3/7] arm64: dts: qcom: qcs6490-audioreach: Modify LPASS macros clock settings for audioreach Prasad Kumpatla 2025-04-29 10:54 ` Konrad Dybcio 2025-05-08 16:38 ` Prasad Kumpatla 2025-05-08 16:45 ` Konrad Dybcio 2025-04-29 9:24 ` [PATCH v2 4/7] arm64: dts: qcom: qcs6490-rb3gen2: Add WSA8830 speakers amplifier Prasad Kumpatla 2025-04-29 10:57 ` Konrad Dybcio 2025-05-08 16:43 ` Prasad Kumpatla 2025-04-29 9:24 ` [PATCH v2 5/7] arm64: dts: qcom: qcs6490-rb3gen2: Add sound card Prasad Kumpatla 2025-04-29 10:58 ` Konrad Dybcio 2025-05-08 16:44 ` Prasad Kumpatla 2025-04-29 9:24 ` [PATCH v2 6/7] arm64: dts: qcom: qcm6490-idp: Add WSA8830 speakers and WCD9370 headset codec Prasad Kumpatla 2025-04-29 11:01 ` Konrad Dybcio 2025-05-08 17:01 ` Prasad Kumpatla 2025-05-08 17:46 ` Konrad Dybcio 2025-04-29 9:24 ` [PATCH v2 7/7] arm64: dts: qcom: qcm6490-idp: Add sound card Prasad Kumpatla 2025-04-29 11:02 ` Konrad Dybcio 2025-04-29 14:44 ` [PATCH v2 0/7] Enable audio on qcs6490-RB3Gen2 and qcm6490-idp boards Rob Herring (Arm) 2025-05-08 14:42 ` Alexey Klimov
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox