diff mbox series

[1/3] dt-bindings: regulator: Fix qcom,spmi-regulator schema

Message ID 20220822152224.507497-2-jean-philippe@linaro.org
State Not Applicable, archived
Headers show
Series dt-bindings: Some schema fixes | expand

Checks

Context Check Description
robh/checkpatch warning total: 0 errors, 2 warnings, 15 lines checked
robh/patch-applied success
robh/dtbs-check warning build log
robh/dt-meta-schema success

Commit Message

Jean-Philippe Brucker Aug. 22, 2022, 3:22 p.m. UTC
The DT validator reports an error in the schema:

Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.yaml: ignoring, error in schema: patternProperties: ^(5vs[1-2]|(l|s)[1-9][0-9]?|lvs[1-3])$: properties

Move the unevaluatedProperties statement out of the properties section
to fix it.

Fixes: 0b3bbd7646b0 ("regulator: qcom,spmi-regulator: Convert to dtschema")
Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
---
 .../devicetree/bindings/regulator/qcom,spmi-regulator.yaml     | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Rob Herring Aug. 22, 2022, 7:42 p.m. UTC | #1
On Mon, Aug 22, 2022 at 10:23 AM Jean-Philippe Brucker
<jean-philippe@linaro.org> wrote:
>
> The DT validator reports an error in the schema:
>
> Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.yaml: ignoring, error in schema: patternProperties: ^(5vs[1-2]|(l|s)[1-9][0-9]?|lvs[1-3])$: properties
>
> Move the unevaluatedProperties statement out of the properties section
> to fix it.
>
> Fixes: 0b3bbd7646b0 ("regulator: qcom,spmi-regulator: Convert to dtschema")
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> ---
>  .../devicetree/bindings/regulator/qcom,spmi-regulator.yaml     | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

Reviewed-by: Rob Herring <robh@kernel.org>
Rob Herring Aug. 30, 2022, 12:45 p.m. UTC | #2
On Mon, Aug 22, 2022 at 2:42 PM Rob Herring <robh+dt@kernel.org> wrote:
>
> On Mon, Aug 22, 2022 at 10:23 AM Jean-Philippe Brucker
> <jean-philippe@linaro.org> wrote:
> >
> > The DT validator reports an error in the schema:
> >
> > Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.yaml: ignoring, error in schema: patternProperties: ^(5vs[1-2]|(l|s)[1-9][0-9]?|lvs[1-3])$: properties
> >
> > Move the unevaluatedProperties statement out of the properties section
> > to fix it.
> >
> > Fixes: 0b3bbd7646b0 ("regulator: qcom,spmi-regulator: Convert to dtschema")
> > Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> > ---
> >  .../devicetree/bindings/regulator/qcom,spmi-regulator.yaml     | 3 +--
> >  1 file changed, 1 insertion(+), 2 deletions(-)
>
> Reviewed-by: Rob Herring <robh@kernel.org>

You sent this to the wrong Mar[kc]. Can you resend it and also Cc LKML.

Rob
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.yaml b/Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.yaml
index 8b7c4af4b551..faa4af9fd035 100644
--- a/Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.yaml
+++ b/Documentation/devicetree/bindings/regulator/qcom,spmi-regulator.yaml
@@ -35,6 +35,7 @@  patternProperties:
     description: List of regulators and its properties
     type: object
     $ref: regulator.yaml#
+    unevaluatedProperties: false
 
     properties:
       qcom,ocp-max-retries:
@@ -100,8 +101,6 @@  patternProperties:
           SAW controlled gang leader. Will be configured as SAW regulator.
         type: boolean
 
-      unevaluatedProperties: false
-
 required:
   - compatible