Message ID | 20241212004727.2903846-9-alexey.klimov@linaro.org (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | qrb4210-rb2: add wsa audio playback and capture support | expand |
On Thu, Dec 12, 2024 at 12:47:21AM +0000, Alexey Klimov wrote: > Add support for qcom,wsa8815 and qcom,wsa8810 names to be recognised What is WSA8815 that? > as a valid compatibles. > > Signed-off-by: Alexey Klimov <alexey.klimov@linaro.org> > --- > Documentation/devicetree/bindings/arm/qcom-soc.yaml | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/arm/qcom-soc.yaml b/Documentation/devicetree/bindings/arm/qcom-soc.yaml > index 2ea6d3f65478..ebccafeb13e3 100644 > --- a/Documentation/devicetree/bindings/arm/qcom-soc.yaml > +++ b/Documentation/devicetree/bindings/arm/qcom-soc.yaml > @@ -23,7 +23,7 @@ description: | > select: > properties: > compatible: > - pattern: "^qcom,.*(apq|ipq|mdm|msm|qcm|qcs|q[dr]u|sa|sar|sc|sd[amx]|sm|x1e)[0-9]+.*$" > + pattern: "^qcom,.*(apq|ipq|mdm|msm|qcm|qcs|q[dr]u|sa|sar|sc|sd[amx]|sm|wsa|x1e)[0-9]+.*$" This is odd. WSA is not a SoC. Please provide description explaining this, if you think otherwise. Best regards, Krzysztof
On 12/12/2024 08:46, Krzysztof Kozlowski wrote: > On Thu, Dec 12, 2024 at 12:47:21AM +0000, Alexey Klimov wrote: >> Add support for qcom,wsa8815 and qcom,wsa8810 names to be recognised > > What is WSA8815 that? I guess I had some longer thought: "What is WSA8815 that it should be here?" > >> as a valid compatibles. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/arm/qcom-soc.yaml b/Documentation/devicetree/bindings/arm/qcom-soc.yaml index 2ea6d3f65478..ebccafeb13e3 100644 --- a/Documentation/devicetree/bindings/arm/qcom-soc.yaml +++ b/Documentation/devicetree/bindings/arm/qcom-soc.yaml @@ -23,7 +23,7 @@ description: | select: properties: compatible: - pattern: "^qcom,.*(apq|ipq|mdm|msm|qcm|qcs|q[dr]u|sa|sar|sc|sd[amx]|sm|x1e)[0-9]+.*$" + pattern: "^qcom,.*(apq|ipq|mdm|msm|qcm|qcs|q[dr]u|sa|sar|sc|sd[amx]|sm|wsa|x1e)[0-9]+.*$" required: - compatible @@ -34,6 +34,7 @@ properties: - pattern: "^qcom,(apq|ipq|mdm|msm|qcm|qcs|q[dr]u|sa|sc|sd[amx]|sm|x1e)[0-9]+(pro)?-.*$" - pattern: "^qcom,sar[0-9]+[a-z]?-.*$" - pattern: "^qcom,(sa|sc)8[0-9]+[a-z][a-z]?-.*$" + - pattern: "^qcom,wsa[0-9]+.*$" # Legacy namings - variations of existing patterns/compatibles are OK, # but do not add completely new entries to these:
Add support for qcom,wsa8815 and qcom,wsa8810 names to be recognised as a valid compatibles. Signed-off-by: Alexey Klimov <alexey.klimov@linaro.org> --- Documentation/devicetree/bindings/arm/qcom-soc.yaml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)