[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <5dae76ba-bd48-233d-4d4a-14111ff1b2ec@linaro.org>
Date: Tue, 15 Nov 2022 11:22:39 +0100
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Srinivas Kandagatla <srinivas.kandagatla@...aro.org>,
Banajit Goswami <bgoswami@...cinc.com>,
Andy Gross <agross@...nel.org>,
Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konrad.dybcio@...aro.org>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Liam Girdwood <lgirdwood@...il.com>,
Mark Brown <broonie@...nel.org>, alsa-devel@...a-project.org,
linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Cc: Patrick Lai <plai@....qualcomm.com>,
Srinivasa Rao Mandadapu <srivasam@....qualcomm.com>
Subject: Re: [PATCH 02/10] ASoC: dt-bindings: qcom,apr: Split services to
shared schema
On 11/11/2022 12:35, Krzysztof Kozlowski wrote:
> The APR/GPR nodes are organized like:
>
> apr-or-gpr-device-node <- qcom,apr.yaml
> apr-gpr-service@[0-9] <- qcom,apr.yaml
> service-specific-components <- /schemas/sound/qcom,q6*.yaml
>
(...)
> +additionalProperties: true
> diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,apr.yaml b/Documentation/devicetree/bindings/soc/qcom/qcom,apr.yaml
> index 0a7a34cb2497..9302ffe567d6 100644
> --- a/Documentation/devicetree/bindings/soc/qcom/qcom,apr.yaml
> +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,apr.yaml
> @@ -80,115 +80,13 @@ properties:
> '#size-cells':
> const: 0
>
> -#APR/GPR Services
> patternProperties:
> "^service@[1-9a-d]$":
> type: object
> + $ref: /schemas/soc/qcom/qcom,apr-services.yaml
> + additionalProperties: true
> description:
> - APR/GPR node's client devices use subnodes for desired static port services.
> -
> - properties:
> - compatible:
> - enum:
> - - qcom,q6core
> - - qcom,q6asm
> - - qcom,q6afe
> - - qcom,q6adm
> - - qcom,q6apm
> - - qcom,q6prm
I think that this piece could stay here. Otherwise we allow any
compatible which matches the qcom,apr-services.yaml binding, but that's
easy to achieve.
Best regards,
Krzysztof
Powered by blists - more mailing lists