Skip to contentSkip to conversation

dt-bindings: thermal: Drop 'trips' node as required

PATCH Applied Rob Herring (Arm) 3 participants 3 messages Jul 9, 2024

Subsystems (2)
Device tree bindings Thermal
lore
Share & tools
lore

Rob Herring (Arm)

author maintainer
#0 lore raw

PATCH 1/1 dt-bindings: thermal: Drop 'trips' node as required

It is possible to have thermal zones which don't have any trip points. These zones in effect simply represent a temperature sensor without any action associated with it. While the schema has always required a 'trips' node, users have existed for a long time without it. Update the schema to match reality.

Signed-off-by Rob Herring (Arm) <[email protected]>
---
 Documentation/devicetree/bindings/thermal/thermal-zones.yaml | 1 -
 1 file changed, 1 deletion(-)
Documentation/devicetree/bindings/thermal/thermal-zones.yaml +0 -1
index 68398e7e8655..8315ab4b00eb 100644
@@ -229,7 +229,6 @@ patternProperties:
 
     required:
       - thermal-sensors
-      - trips
 
     additionalProperties: false
Signature · 1 line
2.43.0
Reviewed-by on v1 1/1
#1 lore raw
Quoted 11 lines · Rob Herring (Arm)

It is possible to have thermal zones which don't have any trip points. These zones in effect simply represent a temperature sensor without any action associated with it. While the schema has always required a 'trips' node, users have existed for a long time without it. Update the schema to match reality.

Signed-off-by Rob Herring (Arm) <[email protected]>
---
 Documentation/devicetree/bindings/thermal/thermal-zones.yaml | 1 -
 1 file changed, 1 deletion(-)

Indeed I noticed some new warnings after my recent fixes in Mediatek DTS.

Reviewed-by Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof

Applied
#2 lore raw
In reply to Rob Herring (Arm)
Quoted 8 lines · Rob Herring (Arm)

It is possible to have thermal zones which don't have any trip points. These zones in effect simply represent a temperature sensor without any action associated with it. While the schema has always required a 'trips' node, users have existed for a long time without it. Update the schema to match reality.

Signed-off-by Rob Herring (Arm) <[email protected]>

---

Applied, thanks

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