Skip to contentSkip to conversation

dt-bindings: thermal: sprd: Remove redundant 'maxItems'

PATCH Applied Rob Herring 4 participants 5 messages Mar 13, 2020 – Mar 15, 2020

Subsystems (5)
Device tree bindings Arm Qualcomm mailing list Thermal Arm/spreadtrum SoC support +1 more subsystems
lore
Share & tools
lore

Rob Herring

author maintainer
#0 lore raw

PATCH 1/2 dt-bindings: thermal: sprd: Remove redundant 'maxItems'

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 <[email protected]>
Cc Baolin Wang <[email protected]>
Cc Chunyan Zhang <[email protected]>
Cc Zhang Rui <[email protected]>
Cc Daniel Lezcano <[email protected]>
Cc Amit Kucheria <[email protected]>
Cc [email protected]
Cc [email protected]
Signed-off-by Rob Herring <[email protected]>
---
 Documentation/devicetree/bindings/thermal/sprd-thermal.yaml | 1 -
 1 file changed, 1 deletion(-)
Documentation/devicetree/bindings/thermal/sprd-thermal.yaml +0 -1
index 3bfe05504456..058c4cc06ba6 100644
@@ -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
Signature · 1 line
2.20.1
author maintainer
#1 lore raw

PATCH 2/2 dt-bindings: thermal: qcom-tsens: Remove redundant 'maxItems'

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 <[email protected]>
Cc Bjorn Andersson <[email protected]>
Cc Amit Kucheria <[email protected]>
Cc Zhang Rui <[email protected]>
Cc Daniel Lezcano <[email protected]>
Cc [email protected]
Cc [email protected]
Cc [email protected]
Signed-off-by Rob Herring <[email protected]>
---
 Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 -
 1 file changed, 1 deletion(-)
Documentation/devicetree/bindings/thermal/qcom-tsens.yaml +0 -1
index c0ed030d0960..62b97a6d9b65 100644
@@ -43,7 +43,6 @@ properties:
           - const: qcom,tsens-v2
 
   reg:
-    maxItems: 2
     items:
       - description: TM registers
       - description: SROT registers
Signature · 1 line
2.20.1
Reviewed-by on v1 2/2
#2 lore raw

Re: [PATCH 2/2] dt-bindings: thermal: qcom-tsens: Remove redundant 'maxItems'

Quoted 3 lines · Rob Herring

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 <[email protected]>
Quoted 28 lines
Cc Andy Gross <[email protected]>
Cc Bjorn Andersson <[email protected]>
Cc Amit Kucheria <[email protected]>
Cc Zhang Rui <[email protected]>
Cc Daniel Lezcano <[email protected]>
Cc [email protected]
Cc [email protected]
Cc [email protected]
Signed-off-by Rob Herring <[email protected]>
---
 Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 -
 1 file changed, 1 deletion(-)
Documentation/devicetree/bindings/thermal/qcom-tsens.yaml +0 -2
index c0ed030d0960..62b97a6d9b65 100644
@@ -43,7 +43,6 @@ properties:
           - const: qcom,tsens-v2
 
   reg:
-    maxItems: 2
     items:
       - description: TM registers
       - description: SROT registers
--

2.20.1

maintainer Reviewed-by on v1 1/2
#3 lore raw
In reply to Rob Herring
Quoted 4 lines · Rob Herring

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.

Quoted 9 lines
Cc Orson Zhai <[email protected]>
Cc Baolin Wang <[email protected]>
Cc Chunyan Zhang <[email protected]>
Cc Zhang Rui <[email protected]>
Cc Daniel Lezcano <[email protected]>
Cc Amit Kucheria <[email protected]>
Cc [email protected]
Cc [email protected]
Signed-off-by Rob Herring <[email protected]>

Thanks Rob.

Reviewed-by Baolin Wang <[email protected]>
Quoted 19 lines
---
 Documentation/devicetree/bindings/thermal/sprd-thermal.yaml | 1 -
 1 file changed, 1 deletion(-)
Documentation/devicetree/bindings/thermal/sprd-thermal.yaml +0 -2
index 3bfe05504456..058c4cc06ba6 100644
@@ -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

Signature · 1 line
Baolin Wang
Applied
#4 lore raw
In reply to Rob Herring
Quoted 12 lines · Rob Herring

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 <[email protected]>
Cc Baolin Wang <[email protected]>
Cc Chunyan Zhang <[email protected]>
Cc Zhang Rui <[email protected]>
Cc Daniel Lezcano <[email protected]>
Cc Amit Kucheria <[email protected]>
Cc [email protected]
Cc [email protected]
Signed-off-by Rob Herring <[email protected]>

Applied, thanks

Quoted 17 lines
---
 Documentation/devicetree/bindings/thermal/sprd-thermal.yaml | 1 -
 1 file changed, 1 deletion(-)
Documentation/devicetree/bindings/thermal/sprd-thermal.yaml +0 -1
index 3bfe05504456..058c4cc06ba6 100644
@@ -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
Signature · 5 lines
 <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