diff mbox series

[1/1] media: dt-bindings: media: renesas,drif: Fix a dt_binding_check warning

Message ID 20210520114953.8206-1-thunder.leizhen@huawei.com
State Superseded, archived
Headers show
Series [1/1] media: dt-bindings: media: renesas,drif: Fix a dt_binding_check warning | expand

Checks

Context Check Description
robh/checkpatch success
robh/dt-meta-schema success
robh/dtbs-check success

Commit Message

Leizhen (ThunderTown) May 20, 2021, 11:49 a.m. UTC
The value of the property 'clock-names' is a constant string, so the
'maxItems: 1' is not needed, should be removed. Otherwise, the following
warning is reported:
properties:clock-names:maxItems: False schema does not allow 1
ignoring, error in schema: properties: clock-names: maxItems
warning: no schema found in file: xxx/bindings/media/renesas,drif.yaml

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

Comments

Leizhen (ThunderTown) June 4, 2021, 3:15 a.m. UTC | #1
Hi all:
  Can someone review this patch?


On 2021/5/20 19:49, Zhen Lei wrote:
> The value of the property 'clock-names' is a constant string, so the
> 'maxItems: 1' is not needed, should be removed. Otherwise, the following
> warning is reported:
> properties:clock-names:maxItems: False schema does not allow 1
> ignoring, error in schema: properties: clock-names: maxItems
> warning: no schema found in file: xxx/bindings/media/renesas,drif.yaml
> 
> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
> ---
>  Documentation/devicetree/bindings/media/renesas,drif.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/media/renesas,drif.yaml b/Documentation/devicetree/bindings/media/renesas,drif.yaml
> index f1bdaeab4053..b7f07062922f 100644
> --- a/Documentation/devicetree/bindings/media/renesas,drif.yaml
> +++ b/Documentation/devicetree/bindings/media/renesas,drif.yaml
> @@ -67,7 +67,6 @@ properties:
>      maxItems: 1
>  
>    clock-names:
> -    maxItems: 1
>      items:
>        - const: fck
>  
>
Fabrizio Castro June 4, 2021, 8:54 a.m. UTC | #2
Hi Zhen,

Thanks for your patch!

> From: Zhen Lei <thunder.leizhen@huawei.com>
> Sent: 20 May 2021 12:50
> Subject: [PATCH 1/1] media: dt-bindings: media: renesas,drif: Fix a
> dt_binding_check warning
> 
> The value of the property 'clock-names' is a constant string, so the
> 'maxItems: 1' is not needed, should be removed. Otherwise, the following
> warning is reported:
> properties:clock-names:maxItems: False schema does not allow 1
> ignoring, error in schema: properties: clock-names: maxItems
> warning: no schema found in file: xxx/bindings/media/renesas,drif.yaml

A similar fix was sent and accepted a while ago, please see:
https://lkml.org/lkml/2021/4/8/1329

Best regards,
Fab

> 
> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
> ---
>  Documentation/devicetree/bindings/media/renesas,drif.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/media/renesas,drif.yaml
> b/Documentation/devicetree/bindings/media/renesas,drif.yaml
> index f1bdaeab4053..b7f07062922f 100644
> --- a/Documentation/devicetree/bindings/media/renesas,drif.yaml
> +++ b/Documentation/devicetree/bindings/media/renesas,drif.yaml
> @@ -67,7 +67,6 @@ properties:
>      maxItems: 1
> 
>    clock-names:
> -    maxItems: 1
>      items:
>        - const: fck
> 
> --
> 2.21.1
>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/media/renesas,drif.yaml b/Documentation/devicetree/bindings/media/renesas,drif.yaml
index f1bdaeab4053..b7f07062922f 100644
--- a/Documentation/devicetree/bindings/media/renesas,drif.yaml
+++ b/Documentation/devicetree/bindings/media/renesas,drif.yaml
@@ -67,7 +67,6 @@  properties:
     maxItems: 1
 
   clock-names:
-    maxItems: 1
     items:
       - const: fck