linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/5] fix dt_binding_check warnings
@ 2020-12-04  9:38 Zhen Lei
  2020-12-04  9:38 ` [PATCH 1/5] media: dt-bindings: add the required property 'additionalProperties' Zhen Lei
                   ` (4 more replies)
  0 siblings, 5 replies; 17+ messages in thread
From: Zhen Lei @ 2020-12-04  9:38 UTC (permalink / raw)
  To: Philipp Zabel, Mauro Carvalho Chehab, Rob Herring, Karol Gugala,
	Mateusz Holenko, Greg Kroah-Hartman, Pawel Czarnecki,
	Stafford Horne, Matthias Brugger, Lee Jones, linux-media,
	linux-serial, linux-arm-kernel, linux-mediatek, devicetree,
	linux-kernel
  Cc: Zhen Lei

These patches is based on the latest linux-next code.

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

/root/linux-next/Documentation/devicetree/bindings/serial/litex,liteuart.yaml: 'additionalProperties' is a required property
/root/linux-next/Documentation/devicetree/bindings/soc/mediatek/devapc.yaml: 'additionalProperties' is a required property
/root/linux-next/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml: 'additionalProperties' is a required property
/root/linux-next/Documentation/devicetree/bindings/media/coda.yaml: 'additionalProperties' is a required property
/root/linux-next/Documentation/devicetree/bindings/media/coda.yaml: ignoring, error in schema:
warning: no schema found in file: ./Documentation/devicetree/bindings/media/coda.yaml
/root/linux-next/Documentation/devicetree/bindings/serial/litex,liteuart.yaml: ignoring, error in schema:
warning: no schema found in file: ./Documentation/devicetree/bindings/serial/litex,liteuart.yaml
/root/linux-next/Documentation/devicetree/bindings/soc/mediatek/devapc.yaml: ignoring, error in schema:
warning: no schema found in file: ./Documentation/devicetree/bindings/soc/mediatek/devapc.yaml
/root/linux-next/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml: ignoring, error in schema:
warning: no schema found in file: ./Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml
/root/linux-next/Documentation/devicetree/bindings/mfd/iqs62x.example.dt.yaml: pwmleds: 'panel' does not match any of the regexes: '^led(-[0-9a-f]+)?$', 'pinctrl-[0-9]+'
        From schema: /root/linux-next/Documentation/devicetree/bindings/leds/leds-pwm.yaml

I have done the dtbs_check for all 5 yaml files on both arm32 and arm64.


Zhen Lei (5):
  media: dt-bindings: add the required property 'additionalProperties'
  dt-bindings: serial: add the required property 'additionalProperties'
  dt-bindings: soc: add the required property 'additionalProperties'
  dt-bindings: devapc: add the required property 'additionalProperties'
  dt-bindings: mfd: correct the node name of the panel led

 Documentation/devicetree/bindings/media/coda.yaml                   | 6 ++++++
 Documentation/devicetree/bindings/mfd/iqs62x.yaml                   | 2 +-
 Documentation/devicetree/bindings/serial/litex,liteuart.yaml        | 2 ++
 .../devicetree/bindings/soc/litex/litex,soc-controller.yaml         | 2 ++
 Documentation/devicetree/bindings/soc/mediatek/devapc.yaml          | 2 ++
 5 files changed, 13 insertions(+), 1 deletion(-)

-- 
1.8.3



^ permalink raw reply	[flat|nested] 17+ messages in thread

* [PATCH 1/5] media: dt-bindings: add the required property 'additionalProperties'
  2020-12-04  9:38 [PATCH 0/5] fix dt_binding_check warnings Zhen Lei
@ 2020-12-04  9:38 ` Zhen Lei
  2020-12-04 10:56   ` Philipp Zabel
  2020-12-04  9:38 ` [PATCH 2/5] dt-bindings: serial: " Zhen Lei
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 17+ messages in thread
From: Zhen Lei @ 2020-12-04  9:38 UTC (permalink / raw)
  To: Philipp Zabel, Mauro Carvalho Chehab, Rob Herring, Karol Gugala,
	Mateusz Holenko, Greg Kroah-Hartman, Pawel Czarnecki,
	Stafford Horne, Matthias Brugger, Lee Jones, linux-media,
	linux-serial, linux-arm-kernel, linux-mediatek, devicetree,
	linux-kernel
  Cc: Zhen Lei

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

xxx/media/coda.yaml: 'additionalProperties' is a required property
xxx/media/coda.yaml: ignoring, error in schema:
warning: no schema found in file: xxx/media/coda.yaml

There are three properties defined in allOf, they should be explicitly
declared. Otherwise, "additionalProperties: false" will prohibit them.

Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
---
 Documentation/devicetree/bindings/media/coda.yaml | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/Documentation/devicetree/bindings/media/coda.yaml b/Documentation/devicetree/bindings/media/coda.yaml
index 7bac0057faf7319..199b0826cd01cb6 100644
--- a/Documentation/devicetree/bindings/media/coda.yaml
+++ b/Documentation/devicetree/bindings/media/coda.yaml
@@ -52,6 +52,10 @@ properties:
     description: phandle pointing to the SRAM device node
     maxItems: 1
 
+  interrupts: true
+  interrupt-names: true
+  power-domains: true
+
 required:
   - compatible
   - reg
@@ -96,6 +100,8 @@ allOf:
           description: phandle pointing to the PU power domain
           maxItems: 1
 
+additionalProperties: false
+
 examples:
   - |
     vpu: video-codec@63ff4000 {
-- 
1.8.3



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 2/5] dt-bindings: serial: add the required property 'additionalProperties'
  2020-12-04  9:38 [PATCH 0/5] fix dt_binding_check warnings Zhen Lei
  2020-12-04  9:38 ` [PATCH 1/5] media: dt-bindings: add the required property 'additionalProperties' Zhen Lei
@ 2020-12-04  9:38 ` Zhen Lei
  2020-12-07 22:58   ` Rob Herring
  2020-12-18 21:16   ` Rob Herring
  2020-12-04  9:38 ` [PATCH 3/5] dt-bindings: soc: " Zhen Lei
                   ` (2 subsequent siblings)
  4 siblings, 2 replies; 17+ messages in thread
From: Zhen Lei @ 2020-12-04  9:38 UTC (permalink / raw)
  To: Philipp Zabel, Mauro Carvalho Chehab, Rob Herring, Karol Gugala,
	Mateusz Holenko, Greg Kroah-Hartman, Pawel Czarnecki,
	Stafford Horne, Matthias Brugger, Lee Jones, linux-media,
	linux-serial, linux-arm-kernel, linux-mediatek, devicetree,
	linux-kernel
  Cc: Zhen Lei

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

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

Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
---
 Documentation/devicetree/bindings/serial/litex,liteuart.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/serial/litex,liteuart.yaml b/Documentation/devicetree/bindings/serial/litex,liteuart.yaml
index bc79b3cca542437..c4f1f489dc2d091 100644
--- a/Documentation/devicetree/bindings/serial/litex,liteuart.yaml
+++ b/Documentation/devicetree/bindings/serial/litex,liteuart.yaml
@@ -29,6 +29,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     uart0: serial@e0001800 {
-- 
1.8.3



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 3/5] dt-bindings: soc: add the required property 'additionalProperties'
  2020-12-04  9:38 [PATCH 0/5] fix dt_binding_check warnings Zhen Lei
  2020-12-04  9:38 ` [PATCH 1/5] media: dt-bindings: add the required property 'additionalProperties' Zhen Lei
  2020-12-04  9:38 ` [PATCH 2/5] dt-bindings: serial: " Zhen Lei
@ 2020-12-04  9:38 ` Zhen Lei
  2020-12-07 22:59   ` Rob Herring
  2020-12-18 21:17   ` Rob Herring
  2020-12-04  9:38 ` [PATCH 4/5] dt-bindings: devapc: " Zhen Lei
  2020-12-04  9:38 ` [PATCH 5/5] dt-bindings: mfd: correct the node name of the panel led Zhen Lei
  4 siblings, 2 replies; 17+ messages in thread
From: Zhen Lei @ 2020-12-04  9:38 UTC (permalink / raw)
  To: Philipp Zabel, Mauro Carvalho Chehab, Rob Herring, Karol Gugala,
	Mateusz Holenko, Greg Kroah-Hartman, Pawel Czarnecki,
	Stafford Horne, Matthias Brugger, Lee Jones, linux-media,
	linux-serial, linux-arm-kernel, linux-mediatek, devicetree,
	linux-kernel
  Cc: Zhen Lei

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 <thunder.leizhen@huawei.com>
---
 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



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 4/5] dt-bindings: devapc: add the required property 'additionalProperties'
  2020-12-04  9:38 [PATCH 0/5] fix dt_binding_check warnings Zhen Lei
                   ` (2 preceding siblings ...)
  2020-12-04  9:38 ` [PATCH 3/5] dt-bindings: soc: " Zhen Lei
@ 2020-12-04  9:38 ` Zhen Lei
  2020-12-07 22:59   ` Rob Herring
  2020-12-18 21:17   ` Rob Herring
  2020-12-04  9:38 ` [PATCH 5/5] dt-bindings: mfd: correct the node name of the panel led Zhen Lei
  4 siblings, 2 replies; 17+ messages in thread
From: Zhen Lei @ 2020-12-04  9:38 UTC (permalink / raw)
  To: Philipp Zabel, Mauro Carvalho Chehab, Rob Herring, Karol Gugala,
	Mateusz Holenko, Greg Kroah-Hartman, Pawel Czarnecki,
	Stafford Horne, Matthias Brugger, Lee Jones, linux-media,
	linux-serial, linux-arm-kernel, linux-mediatek, devicetree,
	linux-kernel
  Cc: Zhen Lei

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

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

Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
---
 Documentation/devicetree/bindings/soc/mediatek/devapc.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/soc/mediatek/devapc.yaml b/Documentation/devicetree/bindings/soc/mediatek/devapc.yaml
index 6c763f873a632d2..31e4d3c339bfe72 100644
--- a/Documentation/devicetree/bindings/soc/mediatek/devapc.yaml
+++ b/Documentation/devicetree/bindings/soc/mediatek/devapc.yaml
@@ -44,6 +44,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
-- 
1.8.3



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 5/5] dt-bindings: mfd: correct the node name of the panel led
  2020-12-04  9:38 [PATCH 0/5] fix dt_binding_check warnings Zhen Lei
                   ` (3 preceding siblings ...)
  2020-12-04  9:38 ` [PATCH 4/5] dt-bindings: devapc: " Zhen Lei
@ 2020-12-04  9:38 ` Zhen Lei
  2020-12-07 23:00   ` Rob Herring
  2020-12-16  8:58   ` Lee Jones
  4 siblings, 2 replies; 17+ messages in thread
From: Zhen Lei @ 2020-12-04  9:38 UTC (permalink / raw)
  To: Philipp Zabel, Mauro Carvalho Chehab, Rob Herring, Karol Gugala,
	Mateusz Holenko, Greg Kroah-Hartman, Pawel Czarnecki,
	Stafford Horne, Matthias Brugger, Lee Jones, linux-media,
	linux-serial, linux-arm-kernel, linux-mediatek, devicetree,
	linux-kernel
  Cc: Zhen Lei

According to the definition in leds-pwm.yaml, the node name of each led
must match the regular expression "^led(-[0-9a-f]+)?$". "led" or "led-"
followed by a decimal or hexadecimal ID number.

Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
---
 Documentation/devicetree/bindings/mfd/iqs62x.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/mfd/iqs62x.yaml b/Documentation/devicetree/bindings/mfd/iqs62x.yaml
index 541b06d80e7337a..044cd7542c2bcf5 100644
--- a/Documentation/devicetree/bindings/mfd/iqs62x.yaml
+++ b/Documentation/devicetree/bindings/mfd/iqs62x.yaml
@@ -93,7 +93,7 @@ examples:
     pwmleds {
             compatible = "pwm-leds";
 
-            panel {
+            led-1 {
                     pwms = <&iqs620a_pwm 0 1000000>;
                     max-brightness = <255>;
             };
-- 
1.8.3



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* Re: [PATCH 1/5] media: dt-bindings: add the required property 'additionalProperties'
  2020-12-04  9:38 ` [PATCH 1/5] media: dt-bindings: add the required property 'additionalProperties' Zhen Lei
@ 2020-12-04 10:56   ` Philipp Zabel
  2020-12-06  5:32     ` Leizhen (ThunderTown)
  0 siblings, 1 reply; 17+ messages in thread
From: Philipp Zabel @ 2020-12-04 10:56 UTC (permalink / raw)
  To: Zhen Lei, Mauro Carvalho Chehab, Rob Herring, Karol Gugala,
	Mateusz Holenko, Greg Kroah-Hartman, Pawel Czarnecki,
	Stafford Horne, Matthias Brugger, Lee Jones, linux-media,
	linux-serial, linux-arm-kernel, linux-mediatek, devicetree,
	linux-kernel

On Fri, 2020-12-04 at 17:38 +0800, Zhen Lei wrote:
> When I do dt_binding_check for any YAML file, below wanring is always
> reported:
> 
> xxx/media/coda.yaml: 'additionalProperties' is a required property
> xxx/media/coda.yaml: ignoring, error in schema:
> warning: no schema found in file: xxx/media/coda.yaml
> 
> There are three properties defined in allOf, they should be explicitly
> declared. Otherwise, "additionalProperties: false" will prohibit them.
> 
> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>

Thank you, there already is a patch to fix this:

https://lore.kernel.org/linux-media/20201117200752.4004368-1-robh@kernel.org/

regards
Philipp

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 1/5] media: dt-bindings: add the required property 'additionalProperties'
  2020-12-04 10:56   ` Philipp Zabel
@ 2020-12-06  5:32     ` Leizhen (ThunderTown)
  0 siblings, 0 replies; 17+ messages in thread
From: Leizhen (ThunderTown) @ 2020-12-06  5:32 UTC (permalink / raw)
  To: Philipp Zabel, Mauro Carvalho Chehab, Rob Herring, Karol Gugala,
	Mateusz Holenko, Greg Kroah-Hartman, Pawel Czarnecki,
	Stafford Horne, Matthias Brugger, Lee Jones, linux-media,
	linux-serial, linux-arm-kernel, linux-mediatek, devicetree,
	linux-kernel



On 2020/12/4 18:56, Philipp Zabel wrote:
> On Fri, 2020-12-04 at 17:38 +0800, Zhen Lei wrote:
>> When I do dt_binding_check for any YAML file, below wanring is always
>> reported:
>>
>> xxx/media/coda.yaml: 'additionalProperties' is a required property
>> xxx/media/coda.yaml: ignoring, error in schema:
>> warning: no schema found in file: xxx/media/coda.yaml
>>
>> There are three properties defined in allOf, they should be explicitly
>> declared. Otherwise, "additionalProperties: false" will prohibit them.
>>
>> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
> 
> Thank you, there already is a patch to fix this:
> 
> https://lore.kernel.org/linux-media/20201117200752.4004368-1-robh@kernel.org/

OK. I found it temporarily during do a JSON conversion,I have not subscribed the
dt-binding maillist.

> 
> regards
> Philipp
> 
> .
> 


^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 2/5] dt-bindings: serial: add the required property 'additionalProperties'
  2020-12-04  9:38 ` [PATCH 2/5] dt-bindings: serial: " Zhen Lei
@ 2020-12-07 22:58   ` Rob Herring
  2020-12-18 21:16   ` Rob Herring
  1 sibling, 0 replies; 17+ messages in thread
From: Rob Herring @ 2020-12-07 22:58 UTC (permalink / raw)
  To: Zhen Lei
  Cc: Matthias Brugger, linux-media, linux-serial, Philipp Zabel,
	Mauro Carvalho Chehab, Karol Gugala, Greg Kroah-Hartman,
	Pawel Czarnecki, Stafford Horne, Lee Jones, linux-mediatek,
	Mateusz Holenko, devicetree, linux-arm-kernel, linux-kernel,
	Rob Herring

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

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 3/5] dt-bindings: soc: add the required property 'additionalProperties'
  2020-12-04  9:38 ` [PATCH 3/5] dt-bindings: soc: " Zhen Lei
@ 2020-12-07 22:59   ` Rob Herring
  2020-12-18 21:17   ` Rob Herring
  1 sibling, 0 replies; 17+ messages in thread
From: Rob Herring @ 2020-12-07 22:59 UTC (permalink / raw)
  To: Zhen Lei
  Cc: Pawel Czarnecki, Rob Herring, linux-arm-kernel,
	Mauro Carvalho Chehab, Stafford Horne, Matthias Brugger,
	Karol Gugala, Philipp Zabel, linux-mediatek, linux-kernel,
	devicetree, Greg Kroah-Hartman, Mateusz Holenko, linux-media,
	linux-serial, Lee Jones

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 <thunder.leizhen@huawei.com>
> ---
>  Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 4/5] dt-bindings: devapc: add the required property 'additionalProperties'
  2020-12-04  9:38 ` [PATCH 4/5] dt-bindings: devapc: " Zhen Lei
@ 2020-12-07 22:59   ` Rob Herring
  2020-12-18 21:17   ` Rob Herring
  1 sibling, 0 replies; 17+ messages in thread
From: Rob Herring @ 2020-12-07 22:59 UTC (permalink / raw)
  To: Zhen Lei
  Cc: linux-mediatek, Philipp Zabel, linux-serial, Mateusz Holenko,
	Karol Gugala, linux-arm-kernel, Matthias Brugger,
	Mauro Carvalho Chehab, Pawel Czarnecki, Stafford Horne,
	Lee Jones, devicetree, linux-kernel, Greg Kroah-Hartman,
	linux-media, Rob Herring

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

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 5/5] dt-bindings: mfd: correct the node name of the panel led
  2020-12-04  9:38 ` [PATCH 5/5] dt-bindings: mfd: correct the node name of the panel led Zhen Lei
@ 2020-12-07 23:00   ` Rob Herring
  2020-12-16  8:58   ` Lee Jones
  1 sibling, 0 replies; 17+ messages in thread
From: Rob Herring @ 2020-12-07 23:00 UTC (permalink / raw)
  To: Zhen Lei
  Cc: linux-kernel, Matthias Brugger, linux-arm-kernel, Stafford Horne,
	Karol Gugala, linux-media, devicetree, linux-mediatek,
	Pawel Czarnecki, linux-serial, Lee Jones, Rob Herring,
	Greg Kroah-Hartman, Mauro Carvalho Chehab, Mateusz Holenko,
	Philipp Zabel

On Fri, 04 Dec 2020 17:38:13 +0800, Zhen Lei wrote:
> According to the definition in leds-pwm.yaml, the node name of each led
> must match the regular expression "^led(-[0-9a-f]+)?$". "led" or "led-"
> followed by a decimal or hexadecimal ID number.
> 
> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
> ---
>  Documentation/devicetree/bindings/mfd/iqs62x.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 5/5] dt-bindings: mfd: correct the node name of the panel led
  2020-12-04  9:38 ` [PATCH 5/5] dt-bindings: mfd: correct the node name of the panel led Zhen Lei
  2020-12-07 23:00   ` Rob Herring
@ 2020-12-16  8:58   ` Lee Jones
  1 sibling, 0 replies; 17+ messages in thread
From: Lee Jones @ 2020-12-16  8:58 UTC (permalink / raw)
  To: Zhen Lei
  Cc: Philipp Zabel, Mauro Carvalho Chehab, Rob Herring, Karol Gugala,
	Mateusz Holenko, Greg Kroah-Hartman, Pawel Czarnecki,
	Stafford Horne, Matthias Brugger, linux-media, linux-serial,
	linux-arm-kernel, linux-mediatek, devicetree, linux-kernel

On Fri, 04 Dec 2020, Zhen Lei wrote:

> According to the definition in leds-pwm.yaml, the node name of each led
> must match the regular expression "^led(-[0-9a-f]+)?$". "led" or "led-"
> followed by a decimal or hexadecimal ID number.
> 
> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
> ---
>  Documentation/devicetree/bindings/mfd/iqs62x.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks.

-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 2/5] dt-bindings: serial: add the required property 'additionalProperties'
  2020-12-04  9:38 ` [PATCH 2/5] dt-bindings: serial: " Zhen Lei
  2020-12-07 22:58   ` Rob Herring
@ 2020-12-18 21:16   ` Rob Herring
  1 sibling, 0 replies; 17+ messages in thread
From: Rob Herring @ 2020-12-18 21:16 UTC (permalink / raw)
  To: Zhen Lei
  Cc: Matthias Brugger, Greg Kroah-Hartman, devicetree,
	linux-arm-kernel, Rob Herring, Pawel Czarnecki, Stafford Horne,
	Karol Gugala, linux-serial, linux-media, Mauro Carvalho Chehab,
	linux-mediatek, Philipp Zabel, Mateusz Holenko, Lee Jones,
	linux-kernel

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

Applied, thanks!

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 3/5] dt-bindings: soc: add the required property 'additionalProperties'
  2020-12-04  9:38 ` [PATCH 3/5] dt-bindings: soc: " Zhen Lei
  2020-12-07 22:59   ` Rob Herring
@ 2020-12-18 21:17   ` Rob Herring
  2020-12-19  9:27     ` Stafford Horne
  1 sibling, 1 reply; 17+ messages in thread
From: Rob Herring @ 2020-12-18 21:17 UTC (permalink / raw)
  To: Zhen Lei
  Cc: Karol Gugala, linux-mediatek, linux-kernel, Matthias Brugger,
	Lee Jones, linux-arm-kernel, Philipp Zabel,
	Mauro Carvalho Chehab, linux-serial, devicetree, Mateusz Holenko,
	Greg Kroah-Hartman, linux-media, Pawel Czarnecki, Rob Herring,
	Stafford Horne

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 <thunder.leizhen@huawei.com>
> ---
>  Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 

Applied, thanks!

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 4/5] dt-bindings: devapc: add the required property 'additionalProperties'
  2020-12-04  9:38 ` [PATCH 4/5] dt-bindings: devapc: " Zhen Lei
  2020-12-07 22:59   ` Rob Herring
@ 2020-12-18 21:17   ` Rob Herring
  1 sibling, 0 replies; 17+ messages in thread
From: Rob Herring @ 2020-12-18 21:17 UTC (permalink / raw)
  To: Zhen Lei
  Cc: Greg Kroah-Hartman, Mauro Carvalho Chehab, linux-serial,
	Karol Gugala, linux-media, Pawel Czarnecki, Philipp Zabel,
	linux-kernel, Matthias Brugger, linux-mediatek, linux-arm-kernel,
	Rob Herring, Mateusz Holenko, Lee Jones, devicetree,
	Stafford Horne

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

Applied, thanks!

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 3/5] dt-bindings: soc: add the required property 'additionalProperties'
  2020-12-18 21:17   ` Rob Herring
@ 2020-12-19  9:27     ` Stafford Horne
  0 siblings, 0 replies; 17+ messages in thread
From: Stafford Horne @ 2020-12-19  9:27 UTC (permalink / raw)
  To: Rob Herring
  Cc: Zhen Lei, Karol Gugala, linux-mediatek, linux-kernel,
	Matthias Brugger, Lee Jones, linux-arm-kernel, Philipp Zabel,
	Mauro Carvalho Chehab, linux-serial, devicetree, Mateusz Holenko,
	Greg Kroah-Hartman, linux-media, Pawel Czarnecki, Rob Herring

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 <thunder.leizhen@huawei.com>
> > ---
> >  Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> 
> Applied, thanks!

Thank you!

-Stafford

^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2020-12-19  9:28 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-04  9:38 [PATCH 0/5] fix dt_binding_check warnings Zhen Lei
2020-12-04  9:38 ` [PATCH 1/5] media: dt-bindings: add the required property 'additionalProperties' Zhen Lei
2020-12-04 10:56   ` Philipp Zabel
2020-12-06  5:32     ` Leizhen (ThunderTown)
2020-12-04  9:38 ` [PATCH 2/5] dt-bindings: serial: " Zhen Lei
2020-12-07 22:58   ` Rob Herring
2020-12-18 21:16   ` Rob Herring
2020-12-04  9:38 ` [PATCH 3/5] dt-bindings: soc: " Zhen Lei
2020-12-07 22:59   ` Rob Herring
2020-12-18 21:17   ` Rob Herring
2020-12-19  9:27     ` Stafford Horne
2020-12-04  9:38 ` [PATCH 4/5] dt-bindings: devapc: " Zhen Lei
2020-12-07 22:59   ` Rob Herring
2020-12-18 21:17   ` Rob Herring
2020-12-04  9:38 ` [PATCH 5/5] dt-bindings: mfd: correct the node name of the panel led Zhen Lei
2020-12-07 23:00   ` Rob Herring
2020-12-16  8:58   ` Lee Jones

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).