All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] dt-bindings: thermal: sprd: Remove redundant 'maxItems'
@ 2020-03-13 21:45 Rob Herring
  2020-03-13 21:45 ` [PATCH 2/2] dt-bindings: thermal: qcom-tsens: " Rob Herring
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Rob Herring @ 2020-03-13 21:45 UTC (permalink / raw)
  To: Zhang Rui
  Cc: Orson Zhai, Baolin Wang, Chunyan Zhang, Daniel Lezcano,
	Amit Kucheria, linux-pm, devicetree

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(-)

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


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

* [PATCH 2/2] dt-bindings: thermal: qcom-tsens: Remove redundant 'maxItems'
  2020-03-13 21:45 [PATCH 1/2] dt-bindings: thermal: sprd: Remove redundant 'maxItems' Rob Herring
@ 2020-03-13 21:45 ` Rob Herring
  2020-03-15 19:38   ` Amit Kucheria
  2020-03-15  9:38 ` [PATCH 1/2] dt-bindings: thermal: sprd: " Baolin Wang
  2020-03-15 20:14 ` Daniel Lezcano
  2 siblings, 1 reply; 5+ messages in thread
From: Rob Herring @ 2020-03-13 21:45 UTC (permalink / raw)
  To: Zhang Rui
  Cc: Andy Gross, Bjorn Andersson, Amit Kucheria, Daniel Lezcano,
	linux-arm-msm, linux-pm, devicetree

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: Andy Gross <agross@kernel.org>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Amit Kucheria <amit.kucheria@linaro.org>
Cc: Zhang Rui <rui.zhang@intel.com>
Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: linux-pm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Signed-off-by: Rob Herring <robh@kernel.org>
---
 Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
index c0ed030d0960..62b97a6d9b65 100644
--- a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
+++ b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
@@ -43,7 +43,6 @@ properties:
           - const: qcom,tsens-v2
 
   reg:
-    maxItems: 2
     items:
       - description: TM registers
       - description: SROT registers
-- 
2.20.1


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

* Re: [PATCH 1/2] dt-bindings: thermal: sprd: Remove redundant 'maxItems'
  2020-03-13 21:45 [PATCH 1/2] dt-bindings: thermal: sprd: Remove redundant 'maxItems' Rob Herring
  2020-03-13 21:45 ` [PATCH 2/2] dt-bindings: thermal: qcom-tsens: " Rob Herring
@ 2020-03-15  9:38 ` Baolin Wang
  2020-03-15 20:14 ` Daniel Lezcano
  2 siblings, 0 replies; 5+ messages in thread
From: Baolin Wang @ 2020-03-15  9:38 UTC (permalink / raw)
  To: Rob Herring
  Cc: Zhang Rui, Orson Zhai, Chunyan Zhang, Daniel Lezcano,
	Amit Kucheria, Linux PM, devicetree

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
>


-- 
Baolin Wang

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

* Re: [PATCH 2/2] dt-bindings: thermal: qcom-tsens: Remove redundant 'maxItems'
  2020-03-13 21:45 ` [PATCH 2/2] dt-bindings: thermal: qcom-tsens: " Rob Herring
@ 2020-03-15 19:38   ` Amit Kucheria
  0 siblings, 0 replies; 5+ messages in thread
From: Amit Kucheria @ 2020-03-15 19:38 UTC (permalink / raw)
  To: Rob Herring
  Cc: Zhang Rui, Andy Gross, Bjorn Andersson, Daniel Lezcano,
	linux-arm-msm, Linux PM list,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS

On Sat, Mar 14, 2020 at 3:15 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.

Thanks Rob.

Reviewed-by: Amit Kucheria <amit.kucheria@linaro.org>

> Cc: Andy Gross <agross@kernel.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: Amit Kucheria <amit.kucheria@linaro.org>
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: linux-pm@vger.kernel.org
> Cc: devicetree@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>  Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
> index c0ed030d0960..62b97a6d9b65 100644
> --- a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
> +++ b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
> @@ -43,7 +43,6 @@ properties:
>            - const: qcom,tsens-v2
>
>    reg:
> -    maxItems: 2
>      items:
>        - description: TM registers
>        - description: SROT registers
> --
> 2.20.1
>

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

* Re: [PATCH 1/2] dt-bindings: thermal: sprd: Remove redundant 'maxItems'
  2020-03-13 21:45 [PATCH 1/2] dt-bindings: thermal: sprd: Remove redundant 'maxItems' Rob Herring
  2020-03-13 21:45 ` [PATCH 2/2] dt-bindings: thermal: qcom-tsens: " Rob Herring
  2020-03-15  9:38 ` [PATCH 1/2] dt-bindings: thermal: sprd: " Baolin Wang
@ 2020-03-15 20:14 ` Daniel Lezcano
  2 siblings, 0 replies; 5+ messages in thread
From: Daniel Lezcano @ 2020-03-15 20:14 UTC (permalink / raw)
  To: Rob Herring, Zhang Rui
  Cc: Orson Zhai, Baolin Wang, Chunyan Zhang, Amit Kucheria, linux-pm,
	devicetree

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
> 


-- 
 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog


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

end of thread, other threads:[~2020-03-15 20:14 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-13 21:45 [PATCH 1/2] dt-bindings: thermal: sprd: Remove redundant 'maxItems' Rob Herring
2020-03-13 21:45 ` [PATCH 2/2] dt-bindings: thermal: qcom-tsens: " Rob Herring
2020-03-15 19:38   ` Amit Kucheria
2020-03-15  9:38 ` [PATCH 1/2] dt-bindings: thermal: sprd: " Baolin Wang
2020-03-15 20:14 ` Daniel Lezcano

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.