diff mbox series

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

Message ID 20240709150154.3272825-1-robh@kernel.org
State Accepted
Commit 6f48290b1a18dba5870920188725eee6fe603ee6
Headers show
Series dt-bindings: thermal: Drop 'trips' node as required | expand

Commit Message

Rob Herring (Arm) July 9, 2024, 3:01 p.m. UTC
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) <robh@kernel.org>
---
 Documentation/devicetree/bindings/thermal/thermal-zones.yaml | 1 -
 1 file changed, 1 deletion(-)

Comments

Krzysztof Kozlowski July 9, 2024, 3:03 p.m. UTC | #1
On 09/07/2024 17:01, Rob Herring (Arm) wrote:
> 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) <robh@kernel.org>
> ---
>  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 <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/thermal/thermal-zones.yaml b/Documentation/devicetree/bindings/thermal/thermal-zones.yaml
index 68398e7e8655..8315ab4b00eb 100644
--- a/Documentation/devicetree/bindings/thermal/thermal-zones.yaml
+++ b/Documentation/devicetree/bindings/thermal/thermal-zones.yaml
@@ -229,7 +229,6 @@  patternProperties:
 
     required:
       - thermal-sensors
-      - trips
 
     additionalProperties: false