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

[PATCH] dt-bindings: thermal: Drop 'trips' node as required

Posted by Rob Herring (Arm) 1 year, 7 months ago

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

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
 
-- 
2.43.0

Re: [PATCH] dt-bindings: thermal: Drop 'trips' node as required

Posted by Daniel Lezcano 1 year, 7 months ago

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

Applied, thanks


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

Re: [PATCH] dt-bindings: thermal: Drop 'trips' node as required

Posted by Krzysztof Kozlowski 1 year, 7 months ago

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