| Message ID | 20230725123711.149230-1-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | c1b0b6118b48100de0465b09201445df292647c1 |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [net-next] dt-bindings: net: qca,ar803x: add missing unevaluatedProperties for each regulator | expand |
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 >
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 --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
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(+)