* [PATCH 1/2] arm64: dts: qcom: sc7280-idp: drop incorrect dai-cells from WCD938x SDW
@ 2023-02-20 9:54 Krzysztof Kozlowski
2023-02-20 9:54 ` [PATCH 2/2] arm64: dts: qcom: sc7280-qcard: " Krzysztof Kozlowski
` (3 more replies)
0 siblings, 4 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2023-02-20 9:54 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, linux-arm-msm, devicetree, linux-kernel
Cc: Krzysztof Kozlowski
The WCD938x audio codec Soundwire interface part is not a DAI and does
not allow sound-dai-cells:
sc7280-idp.dtb: codec@0,4: '#sound-dai-cells' does not match any of the regexes: 'pinctrl-[0-9]+'
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
index 8ebcf763b3c7..cff0e0d31c15 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
@@ -480,7 +480,6 @@ &swr0 {
wcd_rx: codec@0,4 {
compatible = "sdw20217010d00";
reg = <0 4>;
- #sound-dai-cells = <1>;
qcom,rx-port-mapping = <1 2 3 4 5>;
};
};
@@ -491,7 +490,6 @@ &swr1 {
wcd_tx: codec@0,3 {
compatible = "sdw20217010d00";
reg = <0 3>;
- #sound-dai-cells = <1>;
qcom,tx-port-mapping = <1 2 3 4>;
};
};
--
2.34.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] arm64: dts: qcom: sc7280-qcard: drop incorrect dai-cells from WCD938x SDW
2023-02-20 9:54 [PATCH 1/2] arm64: dts: qcom: sc7280-idp: drop incorrect dai-cells from WCD938x SDW Krzysztof Kozlowski
@ 2023-02-20 9:54 ` Krzysztof Kozlowski
2023-02-20 10:01 ` Konrad Dybcio
2023-03-22 18:21 ` Doug Anderson
2023-02-20 10:01 ` [PATCH 1/2] arm64: dts: qcom: sc7280-idp: " Konrad Dybcio
` (2 subsequent siblings)
3 siblings, 2 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2023-02-20 9:54 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, linux-arm-msm, devicetree, linux-kernel
Cc: Krzysztof Kozlowski
The WCD938x audio codec Soundwire interface part is not a DAI and does
not allow sound-dai-cells:
sc7280-herobrine-crd.dtb: codec@0,4: '#sound-dai-cells' does not match any of the regexes: 'pinctrl-[0-9]+'
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm64/boot/dts/qcom/sc7280-qcard.dtsi | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sc7280-qcard.dtsi b/arch/arm64/boot/dts/qcom/sc7280-qcard.dtsi
index cb0cc2ba2fa3..95d9e4a19d76 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-qcard.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280-qcard.dtsi
@@ -419,7 +419,6 @@ &swr0 {
wcd_rx: codec@0,4 {
compatible = "sdw20217010d00";
reg = <0 4>;
- #sound-dai-cells = <1>;
qcom,rx-port-mapping = <1 2 3 4 5>;
};
};
@@ -428,7 +427,6 @@ &swr1 {
wcd_tx: codec@0,3 {
compatible = "sdw20217010d00";
reg = <0 3>;
- #sound-dai-cells = <1>;
qcom,tx-port-mapping = <1 2 3 4>;
};
};
--
2.34.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] arm64: dts: qcom: sc7280-idp: drop incorrect dai-cells from WCD938x SDW
2023-02-20 9:54 [PATCH 1/2] arm64: dts: qcom: sc7280-idp: drop incorrect dai-cells from WCD938x SDW Krzysztof Kozlowski
2023-02-20 9:54 ` [PATCH 2/2] arm64: dts: qcom: sc7280-qcard: " Krzysztof Kozlowski
@ 2023-02-20 10:01 ` Konrad Dybcio
2023-03-22 18:20 ` Doug Anderson
2023-05-23 3:03 ` (subset) " Bjorn Andersson
3 siblings, 0 replies; 7+ messages in thread
From: Konrad Dybcio @ 2023-02-20 10:01 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Rob Herring,
Krzysztof Kozlowski, linux-arm-msm, devicetree, linux-kernel
On 20.02.2023 10:54, Krzysztof Kozlowski wrote:
> The WCD938x audio codec Soundwire interface part is not a DAI and does
> not allow sound-dai-cells:
>
> sc7280-idp.dtb: codec@0,4: '#sound-dai-cells' does not match any of the regexes: 'pinctrl-[0-9]+'
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Konrad
> arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
> index 8ebcf763b3c7..cff0e0d31c15 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
> @@ -480,7 +480,6 @@ &swr0 {
> wcd_rx: codec@0,4 {
> compatible = "sdw20217010d00";
> reg = <0 4>;
> - #sound-dai-cells = <1>;
> qcom,rx-port-mapping = <1 2 3 4 5>;
> };
> };
> @@ -491,7 +490,6 @@ &swr1 {
> wcd_tx: codec@0,3 {
> compatible = "sdw20217010d00";
> reg = <0 3>;
> - #sound-dai-cells = <1>;
> qcom,tx-port-mapping = <1 2 3 4>;
> };
> };
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] arm64: dts: qcom: sc7280-qcard: drop incorrect dai-cells from WCD938x SDW
2023-02-20 9:54 ` [PATCH 2/2] arm64: dts: qcom: sc7280-qcard: " Krzysztof Kozlowski
@ 2023-02-20 10:01 ` Konrad Dybcio
2023-03-22 18:21 ` Doug Anderson
1 sibling, 0 replies; 7+ messages in thread
From: Konrad Dybcio @ 2023-02-20 10:01 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Rob Herring,
Krzysztof Kozlowski, linux-arm-msm, devicetree, linux-kernel
On 20.02.2023 10:54, Krzysztof Kozlowski wrote:
> The WCD938x audio codec Soundwire interface part is not a DAI and does
> not allow sound-dai-cells:
>
> sc7280-herobrine-crd.dtb: codec@0,4: '#sound-dai-cells' does not match any of the regexes: 'pinctrl-[0-9]+'
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Konrad
> arch/arm64/boot/dts/qcom/sc7280-qcard.dtsi | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7280-qcard.dtsi b/arch/arm64/boot/dts/qcom/sc7280-qcard.dtsi
> index cb0cc2ba2fa3..95d9e4a19d76 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280-qcard.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7280-qcard.dtsi
> @@ -419,7 +419,6 @@ &swr0 {
> wcd_rx: codec@0,4 {
> compatible = "sdw20217010d00";
> reg = <0 4>;
> - #sound-dai-cells = <1>;
> qcom,rx-port-mapping = <1 2 3 4 5>;
> };
> };
> @@ -428,7 +427,6 @@ &swr1 {
> wcd_tx: codec@0,3 {
> compatible = "sdw20217010d00";
> reg = <0 3>;
> - #sound-dai-cells = <1>;
> qcom,tx-port-mapping = <1 2 3 4>;
> };
> };
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] arm64: dts: qcom: sc7280-idp: drop incorrect dai-cells from WCD938x SDW
2023-02-20 9:54 [PATCH 1/2] arm64: dts: qcom: sc7280-idp: drop incorrect dai-cells from WCD938x SDW Krzysztof Kozlowski
2023-02-20 9:54 ` [PATCH 2/2] arm64: dts: qcom: sc7280-qcard: " Krzysztof Kozlowski
2023-02-20 10:01 ` [PATCH 1/2] arm64: dts: qcom: sc7280-idp: " Konrad Dybcio
@ 2023-03-22 18:20 ` Doug Anderson
2023-05-23 3:03 ` (subset) " Bjorn Andersson
3 siblings, 0 replies; 7+ messages in thread
From: Doug Anderson @ 2023-03-22 18:20 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, linux-arm-msm, devicetree, linux-kernel,
Srinivasa Rao Mandadapu, Venkata Prasad Potturu,
Matthias Kaehlcke, Judy Hsiao
Hi,
On Mon, Feb 20, 2023 at 1:54 AM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> The WCD938x audio codec Soundwire interface part is not a DAI and does
> not allow sound-dai-cells:
>
> sc7280-idp.dtb: codec@0,4: '#sound-dai-cells' does not match any of the regexes: 'pinctrl-[0-9]+'
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 2 --
> 1 file changed, 2 deletions(-)
I'm pretty soundwire ignorant but this looks right to me.
Reviewed-by: Douglas Anderson <dianders@chromium.org>
Seems like folks who were involved in landing the patch that
originally added those properties should be CCed on this, though.
Adding them.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] arm64: dts: qcom: sc7280-qcard: drop incorrect dai-cells from WCD938x SDW
2023-02-20 9:54 ` [PATCH 2/2] arm64: dts: qcom: sc7280-qcard: " Krzysztof Kozlowski
2023-02-20 10:01 ` Konrad Dybcio
@ 2023-03-22 18:21 ` Doug Anderson
1 sibling, 0 replies; 7+ messages in thread
From: Doug Anderson @ 2023-03-22 18:21 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, linux-arm-msm, devicetree, linux-kernel,
Srinivasa Rao Mandadapu, Venkata Prasad Potturu,
Matthias Kaehlcke, Judy Hsiao
Hi,
On Mon, Feb 20, 2023 at 1:54 AM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> The WCD938x audio codec Soundwire interface part is not a DAI and does
> not allow sound-dai-cells:
>
> sc7280-herobrine-crd.dtb: codec@0,4: '#sound-dai-cells' does not match any of the regexes: 'pinctrl-[0-9]+'
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/sc7280-qcard.dtsi | 2 --
> 1 file changed, 2 deletions(-)
Like with patch #1, I'm pretty soundwire ignorant but this looks right to me.
Reviewed-by: Douglas Anderson <dianders@chromium.org>
Seems like folks who were involved in landing the patch that
originally added those properties should be CCed on this, though.
Adding them.
-Doug
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: (subset) [PATCH 1/2] arm64: dts: qcom: sc7280-idp: drop incorrect dai-cells from WCD938x SDW
2023-02-20 9:54 [PATCH 1/2] arm64: dts: qcom: sc7280-idp: drop incorrect dai-cells from WCD938x SDW Krzysztof Kozlowski
` (2 preceding siblings ...)
2023-03-22 18:20 ` Doug Anderson
@ 2023-05-23 3:03 ` Bjorn Andersson
3 siblings, 0 replies; 7+ messages in thread
From: Bjorn Andersson @ 2023-05-23 3:03 UTC (permalink / raw)
To: linux-kernel, Krzysztof Kozlowski, Krzysztof Kozlowski,
Konrad Dybcio, Andy Gross, devicetree, linux-arm-msm, Rob Herring
On Mon, 20 Feb 2023 10:54:00 +0100, Krzysztof Kozlowski wrote:
> The WCD938x audio codec Soundwire interface part is not a DAI and does
> not allow sound-dai-cells:
>
> sc7280-idp.dtb: codec@0,4: '#sound-dai-cells' does not match any of the regexes: 'pinctrl-[0-9]+'
>
>
Applied, thanks!
[1/2] arm64: dts: qcom: sc7280-idp: drop incorrect dai-cells from WCD938x SDW
commit: ca8fc6814844d8787e7fec61b2544a871ea8b675
[2/2] arm64: dts: qcom: sc7280-qcard: drop incorrect dai-cells from WCD938x SDW
commit: 16bd455d0897d1b8b7a9aee2ed51d75b14a34563
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-05-23 2:59 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-02-20 9:54 [PATCH 1/2] arm64: dts: qcom: sc7280-idp: drop incorrect dai-cells from WCD938x SDW Krzysztof Kozlowski
2023-02-20 9:54 ` [PATCH 2/2] arm64: dts: qcom: sc7280-qcard: " Krzysztof Kozlowski
2023-02-20 10:01 ` Konrad Dybcio
2023-03-22 18:21 ` Doug Anderson
2023-02-20 10:01 ` [PATCH 1/2] arm64: dts: qcom: sc7280-idp: " Konrad Dybcio
2023-03-22 18:20 ` Doug Anderson
2023-05-23 3:03 ` (subset) " Bjorn Andersson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).