dt-bindings: i3c: silvaco,i3c-master: fix missing schema restriction

[PATCH] dt-bindings: i3c: silvaco,i3c-master: fix missing schema restriction

Posted by Krzysztof Kozlowski 2 years, 8 months ago

Each device schema must end with unevaluatedProperties: false, if it
references other common schema.  Otherwise it would allow any properties
to be listed.

Fixes: b8b0446f1f1a ("dt-bindings: i3c: Describe Silvaco master binding")
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/i3c/silvaco,i3c-master.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/i3c/silvaco,i3c-master.yaml b/Documentation/devicetree/bindings/i3c/silvaco,i3c-master.yaml
index 543b177f8610..133855f11b4f 100644
--- a/Documentation/devicetree/bindings/i3c/silvaco,i3c-master.yaml
+++ b/Documentation/devicetree/bindings/i3c/silvaco,i3c-master.yaml
@@ -44,7 +44,7 @@ required:
   - clock-names
   - clocks
 
-additionalProperties: true
+unevaluatedProperties: false
 
 examples:
   - |
-- 
2.34.1

Re: [PATCH] dt-bindings: i3c: silvaco,i3c-master: fix missing schema restriction

Posted by Rob Herring 2 years, 8 months ago

On Fri, 09 Jun 2023 16:11:07 +0200, Krzysztof Kozlowski wrote:
> Each device schema must end with unevaluatedProperties: false, if it
> references other common schema.  Otherwise it would allow any properties
> to be listed.
> 
> Fixes: b8b0446f1f1a ("dt-bindings: i3c: Describe Silvaco master binding")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/i3c/silvaco,i3c-master.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Applied, thanks!

Re: [PATCH] dt-bindings: i3c: silvaco,i3c-master: fix missing schema restriction

Posted by Alexandre Belloni 2 years, 8 months ago

On 09/06/2023 16:14:03-0600, Rob Herring wrote:
> 
> On Fri, 09 Jun 2023 16:11:07 +0200, Krzysztof Kozlowski wrote:
> > Each device schema must end with unevaluatedProperties: false, if it
> > references other common schema.  Otherwise it would allow any properties
> > to be listed.
> > 
> > Fixes: b8b0446f1f1a ("dt-bindings: i3c: Describe Silvaco master binding")
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > ---
> >  Documentation/devicetree/bindings/i3c/silvaco,i3c-master.yaml | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> 
> Applied, thanks!

As you took this one, I assume you are going to take the previous patch.

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com