diff mbox series

[net-next] dt-bindings: net: qca,ar803x: add missing unevaluatedProperties for each regulator

Message ID 20230725123711.149230-1-krzysztof.kozlowski@linaro.org
State New
Headers
Series [net-next] dt-bindings: net: qca,ar803x: add missing unevaluatedProperties for each regulator |

Commit Message

Krzysztof Kozlowski July 25, 2023, 12:37 p.m. UTC 
  Each regulator node, which references common regulator.yaml schema,
should disallow additional or unevaluated properties.  Otherwise
mistakes in properties will go unnoticed.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/net/qca,ar803x.yaml | 2 ++
 1 file changed, 2 insertions(+)

Comments

Conor Dooley July 25, 2023, 8:01 p.m. UTC | #1 
On Tue, Jul 25, 2023 at 02:37:11PM +0200, Krzysztof Kozlowski wrote:
> Each regulator node, which references common regulator.yaml schema,
> should disallow additional or unevaluated properties.  Otherwise
> mistakes in properties will go unnoticed.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Acked-by: Conor Dooley <conor.dooley@microchip.com>


> ---
>  Documentation/devicetree/bindings/net/qca,ar803x.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/net/qca,ar803x.yaml b/Documentation/devicetree/bindings/net/qca,ar803x.yaml
> index 161d28919316..3acd09f0da86 100644
> --- a/Documentation/devicetree/bindings/net/qca,ar803x.yaml
> +++ b/Documentation/devicetree/bindings/net/qca,ar803x.yaml
> @@ -75,6 +75,7 @@ properties:
>      description:
>        Initial data for the VDDIO regulator. Set this to 1.5V or 1.8V.
>      $ref: /schemas/regulator/regulator.yaml
> +    unevaluatedProperties: false
>  
>    vddh-regulator:
>      type: object
> @@ -82,6 +83,7 @@ properties:
>        Dummy subnode to model the external connection of the PHY VDDH
>        regulator to VDDIO.
>      $ref: /schemas/regulator/regulator.yaml
> +    unevaluatedProperties: false
>  
>  unevaluatedProperties: false
>  
> -- 
> 2.34.1
>
patchwork-bot+netdevbpf@kernel.org July 27, 2023, 4:50 a.m. UTC | #2 
Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Tue, 25 Jul 2023 14:37:11 +0200 you wrote:
> Each regulator node, which references common regulator.yaml schema,
> should disallow additional or unevaluated properties.  Otherwise
> mistakes in properties will go unnoticed.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/net/qca,ar803x.yaml | 2 ++
>  1 file changed, 2 insertions(+)

Here is the summary with links:
  - [net-next] dt-bindings: net: qca,ar803x: add missing unevaluatedProperties for each regulator
    https://git.kernel.org/netdev/net-next/c/c1b0b6118b48

You are awesome, thank you!
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/net/qca,ar803x.yaml b/Documentation/devicetree/bindings/net/qca,ar803x.yaml
index 161d28919316..3acd09f0da86 100644
--- a/Documentation/devicetree/bindings/net/qca,ar803x.yaml
+++ b/Documentation/devicetree/bindings/net/qca,ar803x.yaml
@@ -75,6 +75,7 @@  properties:
     description:
       Initial data for the VDDIO regulator. Set this to 1.5V or 1.8V.
     $ref: /schemas/regulator/regulator.yaml
+    unevaluatedProperties: false
 
   vddh-regulator:
     type: object
@@ -82,6 +83,7 @@  properties:
       Dummy subnode to model the external connection of the PHY VDDH
       regulator to VDDIO.
     $ref: /schemas/regulator/regulator.yaml
+    unevaluatedProperties: false
 
 unevaluatedProperties: false