diff mbox series

dt-bindings: soc: qcom,smd: do not use pattern for simple rpm-requests string

Message ID 20220524070408.39505-1-krzysztof.kozlowski@linaro.org
State Accepted
Commit 9171c484dffd82ef20942ea6adbbc8f765691c69
Headers show
Series dt-bindings: soc: qcom,smd: do not use pattern for simple rpm-requests string | expand

Commit Message

Krzysztof Kozlowski May 24, 2022, 7:04 a.m. UTC
patternProperties should not be used for properties with a simple string
as name:

  Documentation/devicetree/bindings/soc/qcom/qcom,smd.yaml: patternProperties:^(.*-edge|rpm)$:patternProperties:
    '^rpm-requests$' should not be valid under {'pattern': '^\\^[a-zA-Z0-9,\\-._#]+\\$$'}

Fixes: 375eed5f51a8 ("dt-bindings: soc: qcom,smd: convert to dtschema")
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/soc/qcom/qcom,smd.yaml | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

Rob Herring (Arm) May 24, 2022, 6:33 p.m. UTC | #1
On Tue, 24 May 2022 09:04:08 +0200, Krzysztof Kozlowski wrote:
> patternProperties should not be used for properties with a simple string
> as name:
> 
>   Documentation/devicetree/bindings/soc/qcom/qcom,smd.yaml: patternProperties:^(.*-edge|rpm)$:patternProperties:
>     '^rpm-requests$' should not be valid under {'pattern': '^\\^[a-zA-Z0-9,\\-._#]+\\$$'}
> 
> Fixes: 375eed5f51a8 ("dt-bindings: soc: qcom,smd: convert to dtschema")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/soc/qcom/qcom,smd.yaml | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 

Acked-by: Rob Herring <robh@kernel.org>
Rob Herring (Arm) May 31, 2022, 1:53 p.m. UTC | #2
On Tue, 24 May 2022 09:04:08 +0200, Krzysztof Kozlowski wrote:
> patternProperties should not be used for properties with a simple string
> as name:
> 
>   Documentation/devicetree/bindings/soc/qcom/qcom,smd.yaml: patternProperties:^(.*-edge|rpm)$:patternProperties:
>     '^rpm-requests$' should not be valid under {'pattern': '^\\^[a-zA-Z0-9,\\-._#]+\\$$'}
> 
> Fixes: 375eed5f51a8 ("dt-bindings: soc: qcom,smd: convert to dtschema")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/soc/qcom/qcom,smd.yaml | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 

Now failing in Linus' tree and linux-next and no response, so I've 
applied.

Rob
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,smd.yaml b/Documentation/devicetree/bindings/soc/qcom/qcom,smd.yaml
index e6f9ffa1c0ea..bca07bb13ebf 100644
--- a/Documentation/devicetree/bindings/soc/qcom/qcom,smd.yaml
+++ b/Documentation/devicetree/bindings/soc/qcom/qcom,smd.yaml
@@ -66,9 +66,7 @@  patternProperties:
           The identifier for the remote processor as known by the rest of the
           system.
 
-    # Binding for edge subnodes is not complete
-    patternProperties:
-      "^rpm-requests$":
+      rpm-requests:
         type: object
         description:
           In turn, subnodes of the "edges" represent devices tied to SMD