dt-bindings: remoteproc: mtk,scp: Add missing additionalProperties on child node schemas
Subsystems (3)
Rob Herring
maintainerPATCH 1/1 dt-bindings: remoteproc: mtk,scp: Add missing additionalProperties on child node schemas
Just as unevaluatedProperties or additionalProperties are required at the top level of schemas, they should (and will) also be required for child node schemas. That ensures only documented properties are present for any node.
--- Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml | 1 + 1 file changed, 1 insertion(+)
Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml +1 -0
@@ -91,6 +91,7 @@ allOf: additionalProperties: type: object + additionalProperties: false description: Subnodes of the SCP represent rpmsg devices. The names of the devices are not important. The properties of these nodes are defined by the
Signature · 1 line
2.40.1
Il 26/09/23 18:45, Rob Herring ha scritto:
Quoted 6 lines
Just as unevaluatedProperties or additionalProperties are required at the top level of schemas, they should (and will) also be required for child node schemas. That ensures only documented properties are present for any node.
Signed-off-by Rob Herring <[email protected]>
Quoted 7 lines · Rob Herring
Just as unevaluatedProperties or additionalProperties are required at the top level of schemas, they should (and will) also be required for child node schemas. That ensures only documented properties are present for any node.
Signed-off-by Rob Herring <[email protected]>---
Quoted 18 lines
Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml | 1 + 1 file changed, 1 insertion(+)Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml +1
-0@@ -91,6 +91,7 @@ allOf: additionalProperties: type: object + additionalProperties: false description: Subnodes of the SCP represent rpmsg devices. The names of the devices are not important. The properties of these nodes are defined by theSignature · 1 line
2.40.1
Mathieu Poirier
maintainerHi Rob,
Quoted 15 lines · Rob Herring
Just as unevaluatedProperties or additionalProperties are required at the top level of schemas, they should (and will) also be required for child node schemas. That ensures only documented properties are present for any node.
Signed-off-by Rob Herring <[email protected]>--- Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml | 1 + 1 file changed, 1 insertion(+)Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml +0
-0
Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml allOf: · line 94
Quoted 4 lines
Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml +1
-0@@ -91,6 +91,7 @@ allOf: additionalProperties: type: object + additionalProperties: false
Things have changed in the remoteproc tree [1] and this patch doesn't apply. Please see if it is still needed.
Thanks,
Mathieu
Quoted 6 lines
Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml +0
-0description: Subnodes of the SCP represent rpmsg devices. The names of the devices are not important. The properties of these nodes are defined by theSignature · 1 line
2.40.1
Rob Herring
maintainerQuoted 20 lines · Mathieu Poirier
Hi Rob,
Quoted 16 lines · Rob Herring
Just as unevaluatedProperties or additionalProperties are required at the top level of schemas, they should (and will) also be required for child node schemas. That ensures only documented properties are present for any node.
Signed-off-by Rob Herring <[email protected]>--- Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml | 1 + 1 file changed, 1 insertion(+)Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml +0
-0@@ -91,6 +91,7 @@ allOf:
Quoted 6 lines
Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml allOf: · line 94
Quoted 3 lines
Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml +1
-0additionalProperties: type: object + additionalProperties: falseThings have changed in the remoteproc tree [1] and this patch doesn't apply. Please see if it is still needed.
It is not needed any more.
Thanks,
Rob