[PATCH net] dt-bindings: net: pse-pd: ti,tps23881: Fix missing "additionalProperties" constraints

Rob Herring (Arm) posted 1 patch 1 year, 8 months ago
.../bindings/net/pse-pd/ti,tps23881.yaml       | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)
[PATCH net] dt-bindings: net: pse-pd: ti,tps23881: Fix missing "additionalProperties" constraints
Posted by Rob Herring (Arm) 1 year, 8 months ago
The child nodes are missing "additionalProperties" constraints which
means any undocumented properties or child nodes are allowed. Add the
constraints and all the undocumented properties exposed by the fix.

Fixes: f562202fedad ("dt-bindings: net: pse-pd: Add bindings for TPS23881 PSE controller")
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 .../bindings/net/pse-pd/ti,tps23881.yaml       | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/Documentation/devicetree/bindings/net/pse-pd/ti,tps23881.yaml b/Documentation/devicetree/bindings/net/pse-pd/ti,tps23881.yaml
index 4147adb11e10..6992d56832bf 100644
--- a/Documentation/devicetree/bindings/net/pse-pd/ti,tps23881.yaml
+++ b/Documentation/devicetree/bindings/net/pse-pd/ti,tps23881.yaml
@@ -29,13 +29,31 @@ properties:
       of the ports conversion matrix that establishes relationship between
       the logical ports and the physical channels.
     type: object
+    additionalProperties: false
+
+    properties:
+      "#address-cells":
+        const: 1
+
+      "#size-cells":
+        const: 0
 
     patternProperties:
       '^channel@[0-7]$':
         type: object
+        additionalProperties: false
+
+        properties:
+          reg:
+            maxItems: 1
+
         required:
           - reg
 
+    required:
+      - "#address-cells"
+      - "#size-cells"
+
 unevaluatedProperties: false
 
 required:
-- 
2.43.0
Re: [PATCH net] dt-bindings: net: pse-pd: ti,tps23881: Fix missing "additionalProperties" constraints
Posted by Kory Maincent 1 year, 8 months ago
On Thu, 23 May 2024 12:17:50 -0500
"Rob Herring (Arm)" <robh@kernel.org> wrote:

> The child nodes are missing "additionalProperties" constraints which
> means any undocumented properties or child nodes are allowed. Add the
> constraints and all the undocumented properties exposed by the fix.

Acked-by: Kory Maincent <kory.maincent@bootlin.com>

Regards,
-- 
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com