diff mbox series

dt-bindings: remoteproc: qcom,sm8550-pas: correct power domains

Message ID 20230206193313.30667-1-krzysztof.kozlowski@linaro.org (mailing list archive)
State Accepted
Headers show
Series dt-bindings: remoteproc: qcom,sm8550-pas: correct power domains | expand

Commit Message

Krzysztof Kozlowski Feb. 6, 2023, 7:33 p.m. UTC
Correct CDSP and MPSS power domains to match what is used in DTS and the
Linux driver:

  sm8550-mtp.dtb: remoteproc@32300000: power-domain-names: ['cx', 'mxc', 'nsp'] is too long

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml   | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Krzysztof Kozlowski Feb. 6, 2023, 7:36 p.m. UTC | #1
On 06/02/2023 20:33, Krzysztof Kozlowski wrote:
> Correct CDSP and MPSS power domains to match what is used in DTS and the
> Linux driver:
> 
>   sm8550-mtp.dtb: remoteproc@32300000: power-domain-names: ['cx', 'mxc', 'nsp'] is too long
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

And probably:

Fixes: 084258d60712 ("dt-bindings: remoteproc: qcom: adsp: document sm8550 adsp, cdsp & mpss compatible")

Best regards,
Krzysztof
Neil Armstrong Feb. 7, 2023, 9:38 a.m. UTC | #2
On 06/02/2023 20:36, Krzysztof Kozlowski wrote:
> On 06/02/2023 20:33, Krzysztof Kozlowski wrote:
>> Correct CDSP and MPSS power domains to match what is used in DTS and the
>> Linux driver:
>>
>>    sm8550-mtp.dtb: remoteproc@32300000: power-domain-names: ['cx', 'mxc', 'nsp'] is too long
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> And probably:
> 
> Fixes: 084258d60712 ("dt-bindings: remoteproc: qcom: adsp: document sm8550 adsp, cdsp & mpss compatible")
> 
> Best regards,
> Krzysztof
> 

You were to fast, I'm preparing a serie fixing all checks on sm8550, including this one.

Since it's the same fix, with Fixes tag:
Acked-by: Neil Armstrong <neil.armstrong@linaro.org>

Neil
Rob Herring (Arm) Feb. 7, 2023, 9:09 p.m. UTC | #3
On Mon, 06 Feb 2023 20:33:13 +0100, Krzysztof Kozlowski wrote:
> Correct CDSP and MPSS power domains to match what is used in DTS and the
> Linux driver:
> 
>   sm8550-mtp.dtb: remoteproc@32300000: power-domain-names: ['cx', 'mxc', 'nsp'] is too long
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml   | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 

Acked-by: Rob Herring <robh@kernel.org>
Bjorn Andersson Feb. 14, 2023, 5:30 p.m. UTC | #4
On Mon, 6 Feb 2023 20:33:13 +0100, Krzysztof Kozlowski wrote:
> Correct CDSP and MPSS power domains to match what is used in DTS and the
> Linux driver:
> 
>   sm8550-mtp.dtb: remoteproc@32300000: power-domain-names: ['cx', 'mxc', 'nsp'] is too long
> 
> 

Applied, thanks!

[1/1] dt-bindings: remoteproc: qcom,sm8550-pas: correct power domains
      commit: 6f998eb2fd02a7a7a529abb4c7d8bbf44fc3ff13

Best regards,
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml
index ae612809e260..fe216aa531ed 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml
@@ -99,22 +99,22 @@  allOf:
       properties:
         compatible:
           enum:
-            - qcom,sm8550-cdsp-pas
+            - qcom,sm8550-mpss-pas
     then:
       properties:
         power-domains:
           items:
             - description: CX power domain
-            - description: MXC power domain
+            - description: MSS power domain
         power-domain-names:
           items:
             - const: cx
-            - const: mxc
+            - const: mss
   - if:
       properties:
         compatible:
           enum:
-            - qcom,sm8550-mpss-pas
+            - qcom,sm8550-cdsp-pas
     then:
       properties:
         power-domains: