2020-12-04 09:43:55

by Zhen Lei

[permalink] [raw]
Subject: [PATCH 3/5] dt-bindings: soc: add the required property 'additionalProperties'

When I do dt_binding_check for any YAML file, below wanring is always
reported:

xxx/soc/litex/litex,soc-controller.yaml: 'additionalProperties' is a required property
xxx/soc/litex/litex,soc-controller.yaml: ignoring, error in schema:
warning: no schema found in file: xxx/soc/litex/litex,soc-controller.yaml

Signed-off-by: Zhen Lei <[email protected]>
---
Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml | 2 ++
1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml b/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
index e2b788796e791ab..c8b57c7fd08c23d 100644
--- a/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
+++ b/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
@@ -28,6 +28,8 @@ required:
- compatible
- reg

+additionalProperties: false
+
examples:
- |
soc_ctrl0: soc-controller@f0000000 {
--
1.8.3



2020-12-07 23:03:31

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH 3/5] dt-bindings: soc: add the required property 'additionalProperties'

On Fri, 04 Dec 2020 17:38:11 +0800, Zhen Lei wrote:
> When I do dt_binding_check for any YAML file, below wanring is always
> reported:
>
> xxx/soc/litex/litex,soc-controller.yaml: 'additionalProperties' is a required property
> xxx/soc/litex/litex,soc-controller.yaml: ignoring, error in schema:
> warning: no schema found in file: xxx/soc/litex/litex,soc-controller.yaml
>
> Signed-off-by: Zhen Lei <[email protected]>
> ---
> Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml | 2 ++
> 1 file changed, 2 insertions(+)
>

Reviewed-by: Rob Herring <[email protected]>

2020-12-18 21:19:12

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH 3/5] dt-bindings: soc: add the required property 'additionalProperties'

On Fri, 04 Dec 2020 17:38:11 +0800, Zhen Lei wrote:
> When I do dt_binding_check for any YAML file, below wanring is always
> reported:
>
> xxx/soc/litex/litex,soc-controller.yaml: 'additionalProperties' is a required property
> xxx/soc/litex/litex,soc-controller.yaml: ignoring, error in schema:
> warning: no schema found in file: xxx/soc/litex/litex,soc-controller.yaml
>
> Signed-off-by: Zhen Lei <[email protected]>
> ---
> Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml | 2 ++
> 1 file changed, 2 insertions(+)
>

Applied, thanks!

2020-12-19 09:31:25

by Stafford Horne

[permalink] [raw]
Subject: Re: [PATCH 3/5] dt-bindings: soc: add the required property 'additionalProperties'

On Fri, Dec 18, 2020 at 03:17:06PM -0600, Rob Herring wrote:
> On Fri, 04 Dec 2020 17:38:11 +0800, Zhen Lei wrote:
> > When I do dt_binding_check for any YAML file, below wanring is always
> > reported:
> >
> > xxx/soc/litex/litex,soc-controller.yaml: 'additionalProperties' is a required property
> > xxx/soc/litex/litex,soc-controller.yaml: ignoring, error in schema:
> > warning: no schema found in file: xxx/soc/litex/litex,soc-controller.yaml
> >
> > Signed-off-by: Zhen Lei <[email protected]>
> > ---
> > Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml | 2 ++
> > 1 file changed, 2 insertions(+)
> >
>
> Applied, thanks!

Thank you!

-Stafford