[PATCH v13 1/2] dt-bindings: thermal: qcom-tsens: make ipq5018 tsens standalone compatible

George Moussalem via B4 Relay posted 2 patches 4 months ago
There is a newer version of this series
[PATCH v13 1/2] dt-bindings: thermal: qcom-tsens: make ipq5018 tsens standalone compatible
Posted by George Moussalem via B4 Relay 4 months ago
From: George Moussalem <george.moussalem@outlook.com>

IPQ5018 tsens should not use qcom,tsens-v1 as fallback since it has no RPM
and, as such, must deviates from the standard v1 init routine in the
driver as this version of tsens needs to be explicitly reset and enabled
in the driver.

So let's make qcom,ipq5018-tsens a standalone compatible in the bindings.

Fixes: 77c6d28192ef ("dt-bindings: thermal: qcom-tsens: Add ipq5018 compatible")
Signed-off-by: George Moussalem <george.moussalem@outlook.com>
---
 Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
index 0e653bbe9884953b58c4d8569b8d096db47fd54f..3cf16f6734fcfee20735d3da2cfb0f708a11bca4 100644
--- a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
+++ b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
@@ -36,10 +36,15 @@ properties:
               - qcom,msm8974-tsens
           - const: qcom,tsens-v0_1
 
+      - description:
+          v1 of TSENS without RPM which requires to be explicitly reset
+          and enabled in the driver.
+        enum:
+          - qcom,ipq5018-tsens
+
       - description: v1 of TSENS
         items:
           - enum:
-              - qcom,ipq5018-tsens
               - qcom,msm8937-tsens
               - qcom,msm8956-tsens
               - qcom,msm8976-tsens

-- 
2.49.0
Re: [PATCH v13 1/2] dt-bindings: thermal: qcom-tsens: make ipq5018 tsens standalone compatible
Posted by Krzysztof Kozlowski 4 months ago
On Thu, Jun 12, 2025 at 10:46:13AM GMT, George Moussalem wrote:
> IPQ5018 tsens should not use qcom,tsens-v1 as fallback since it has no RPM
> and, as such, must deviates from the standard v1 init routine in the
> driver as this version of tsens needs to be explicitly reset and enabled
> in the driver.
> 
> So let's make qcom,ipq5018-tsens a standalone compatible in the bindings.
> 
> Fixes: 77c6d28192ef ("dt-bindings: thermal: qcom-tsens: Add ipq5018 compatible")
> Signed-off-by: George Moussalem <george.moussalem@outlook.com>

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof