| Message ID | 20241101170309.382782-1-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | dt-bindings: remoteproc: qcom,sm8550-pas: Add SM8750 ADSP | expand |
On 11/1/2024 10:03 AM, Krzysztof Kozlowski wrote: > Document compatible for Qualcomm SM8750 SoC ADSP PAS which looks fully > compatible with SM8550 variant. The only difference from bindings point > of view is one more interrupt ("shutdown-ack"). Marking devices as > compatible, using SM8550 ADSP PAS fallback, requires changing some of > the conditionals in "if:then:" to "contains". > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > --- > > Cc: Melody Olvera <quic_molvera@quicinc.com> > Cc: Satya Durga Srinivasu Prabhala <quic_satyap@quicinc.com> > Cc: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> > --- > .../bindings/remoteproc/qcom,sm8550-pas.yaml | 48 +++++++++++++------ > 1 file changed, 34 insertions(+), 14 deletions(-) > Reviewed-by: Melody Olvera <quic_molvera@quicinc.com>
On Fri, 01 Nov 2024 18:03:09 +0100, Krzysztof Kozlowski wrote: > Document compatible for Qualcomm SM8750 SoC ADSP PAS which looks fully > compatible with SM8550 variant. The only difference from bindings point > of view is one more interrupt ("shutdown-ack"). Marking devices as > compatible, using SM8550 ADSP PAS fallback, requires changing some of > the conditionals in "if:then:" to "contains". > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > --- > > Cc: Melody Olvera <quic_molvera@quicinc.com> > Cc: Satya Durga Srinivasu Prabhala <quic_satyap@quicinc.com> > Cc: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> > --- > .../bindings/remoteproc/qcom,sm8550-pas.yaml | 48 +++++++++++++------ > 1 file changed, 34 insertions(+), 14 deletions(-) > Acked-by: Rob Herring (Arm) <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml index d7fad7b3c2c6..a24cbb61bda7 100644 --- a/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml +++ b/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml @@ -15,16 +15,20 @@ description: properties: compatible: - enum: - - qcom,sdx75-mpss-pas - - qcom,sm8550-adsp-pas - - qcom,sm8550-cdsp-pas - - qcom,sm8550-mpss-pas - - qcom,sm8650-adsp-pas - - qcom,sm8650-cdsp-pas - - qcom,sm8650-mpss-pas - - qcom,x1e80100-adsp-pas - - qcom,x1e80100-cdsp-pas + oneOf: + - enum: + - qcom,sdx75-mpss-pas + - qcom,sm8550-adsp-pas + - qcom,sm8550-cdsp-pas + - qcom,sm8550-mpss-pas + - qcom,sm8650-adsp-pas + - qcom,sm8650-cdsp-pas + - qcom,sm8650-mpss-pas + - qcom,x1e80100-adsp-pas + - qcom,x1e80100-cdsp-pas + - items: + - const: qcom,sm8750-adsp-pas + - const: qcom,sm8550-adsp-pas reg: maxItems: 1 @@ -82,6 +86,20 @@ allOf: maxItems: 5 memory-region: maxItems: 2 + - if: + properties: + compatible: + contains: + enum: + - qcom,sm8750-adsp-pas + then: + properties: + interrupts: + maxItems: 6 + interrupt-names: + maxItems: 6 + memory-region: + maxItems: 2 - if: properties: compatible: @@ -129,10 +147,12 @@ allOf: - if: properties: compatible: - enum: - - qcom,sm8550-adsp-pas - - qcom,sm8650-adsp-pas - - qcom,x1e80100-adsp-pas + contains: + enum: + - qcom,sm8550-adsp-pas + - qcom,sm8650-adsp-pas + - qcom,sm8750-adsp-pas + - qcom,x1e80100-adsp-pas then: properties: power-domains:
Document compatible for Qualcomm SM8750 SoC ADSP PAS which looks fully compatible with SM8550 variant. The only difference from bindings point of view is one more interrupt ("shutdown-ack"). Marking devices as compatible, using SM8550 ADSP PAS fallback, requires changing some of the conditionals in "if:then:" to "contains". Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- Cc: Melody Olvera <quic_molvera@quicinc.com> Cc: Satya Durga Srinivasu Prabhala <quic_satyap@quicinc.com> Cc: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> --- .../bindings/remoteproc/qcom,sm8550-pas.yaml | 48 +++++++++++++------ 1 file changed, 34 insertions(+), 14 deletions(-)