[1/2] dt-bindings: thermal: sprd: Remove redundant 'maxItems'
diff mbox series

Message ID 20200313214552.845-1-robh@kernel.org
State New
Delegated to: Daniel Lezcano
Headers show
Series
  • [1/2] dt-bindings: thermal: sprd: Remove redundant 'maxItems'
Related show

Commit Message

Rob Herring March 13, 2020, 9:45 p.m. UTC
There's no need to specify 'maxItems' with the same value as the number
of entries in 'items'. A meta-schema update will catch future cases.

Cc: Orson Zhai <orsonzhai@gmail.com>
Cc: Baolin Wang <baolin.wang7@gmail.com>
Cc: Chunyan Zhang <zhang.lyra@gmail.com>
Cc: Zhang Rui <rui.zhang@intel.com>
Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: Amit Kucheria <amit.kucheria@verdurent.com>
Cc: linux-pm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Signed-off-by: Rob Herring <robh@kernel.org>
---
 Documentation/devicetree/bindings/thermal/sprd-thermal.yaml | 1 -
 1 file changed, 1 deletion(-)

Comments

Baolin Wang March 15, 2020, 9:38 a.m. UTC | #1
On Sat, Mar 14, 2020 at 5:45 AM Rob Herring <robh@kernel.org> wrote:
>
> There's no need to specify 'maxItems' with the same value as the number
> of entries in 'items'. A meta-schema update will catch future cases.
>
> Cc: Orson Zhai <orsonzhai@gmail.com>
> Cc: Baolin Wang <baolin.wang7@gmail.com>
> Cc: Chunyan Zhang <zhang.lyra@gmail.com>
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: Amit Kucheria <amit.kucheria@verdurent.com>
> Cc: linux-pm@vger.kernel.org
> Cc: devicetree@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>

Thanks Rob.
Reviewed-by: Baolin Wang <baolin.wang7@gmail.com>

> ---
>  Documentation/devicetree/bindings/thermal/sprd-thermal.yaml | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/thermal/sprd-thermal.yaml b/Documentation/devicetree/bindings/thermal/sprd-thermal.yaml
> index 3bfe05504456..058c4cc06ba6 100644
> --- a/Documentation/devicetree/bindings/thermal/sprd-thermal.yaml
> +++ b/Documentation/devicetree/bindings/thermal/sprd-thermal.yaml
> @@ -31,7 +31,6 @@ properties:
>        Reference to nvmem nodes for the calibration data.
>
>    nvmem-cell-names:
> -    maxItems: 2
>      items:
>        - const: thm_sign_cal
>        - const: thm_ratio_cal
> --
> 2.20.1
>
Daniel Lezcano March 15, 2020, 8:14 p.m. UTC | #2
On 13/03/2020 22:45, Rob Herring wrote:
> There's no need to specify 'maxItems' with the same value as the number
> of entries in 'items'. A meta-schema update will catch future cases.
> 
> Cc: Orson Zhai <orsonzhai@gmail.com>
> Cc: Baolin Wang <baolin.wang7@gmail.com>
> Cc: Chunyan Zhang <zhang.lyra@gmail.com>
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: Amit Kucheria <amit.kucheria@verdurent.com>
> Cc: linux-pm@vger.kernel.org
> Cc: devicetree@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>

Applied, thanks

> ---
>  Documentation/devicetree/bindings/thermal/sprd-thermal.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/thermal/sprd-thermal.yaml b/Documentation/devicetree/bindings/thermal/sprd-thermal.yaml
> index 3bfe05504456..058c4cc06ba6 100644
> --- a/Documentation/devicetree/bindings/thermal/sprd-thermal.yaml
> +++ b/Documentation/devicetree/bindings/thermal/sprd-thermal.yaml
> @@ -31,7 +31,6 @@ properties:
>        Reference to nvmem nodes for the calibration data.
>  
>    nvmem-cell-names:
> -    maxItems: 2
>      items:
>        - const: thm_sign_cal
>        - const: thm_ratio_cal
>

Patch
diff mbox series

diff --git a/Documentation/devicetree/bindings/thermal/sprd-thermal.yaml b/Documentation/devicetree/bindings/thermal/sprd-thermal.yaml
index 3bfe05504456..058c4cc06ba6 100644
--- a/Documentation/devicetree/bindings/thermal/sprd-thermal.yaml
+++ b/Documentation/devicetree/bindings/thermal/sprd-thermal.yaml
@@ -31,7 +31,6 @@  properties:
       Reference to nvmem nodes for the calibration data.
 
   nvmem-cell-names:
-    maxItems: 2
     items:
       - const: thm_sign_cal
       - const: thm_ratio_cal