diff mbox series

[v1,1/2] dt-bindings: thermal: qcom-tsens: Add compatible for sm8350

Message ID 20210319104855.85075-1-robert.foss@linaro.org
State Superseded
Headers show
Series [v1,1/2] dt-bindings: thermal: qcom-tsens: Add compatible for sm8350 | expand

Commit Message

Robert Foss March 19, 2021, 10:48 a.m. UTC
Add tsens bindings for sm8350.

Signed-off-by: Robert Foss <robert.foss@linaro.org>

---
 Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 +
 1 file changed, 1 insertion(+)

-- 
2.27.0

Comments

Daniel Lezcano March 19, 2021, 11:27 a.m. UTC | #1
On 19/03/2021 12:24, Robert Foss wrote:
> On Fri, 19 Mar 2021 at 11:49, Robert Foss <robert.foss@linaro.org> wrote:

>>

>> sm8350 has 29 thermal sensors split across two tsens controllers. Add

>> the thermal zones to expose them and wireup the cpus to throttle their

>> frequencies on crossing passive temperature thresholds.

>>

>> Signed-off-by: Robert Foss <robert.foss@linaro.org>

>> ---


[ ... ]

>> +               };

>> +

>> +               // TODO: What is the NSP subsystem?

> 

> This comment should not have been included, will remove in v2


Please trim when replying to a large patch file.

[ ... ]

>> +                       trips {

>> +                               nspss1_alert0: trip-point0 {

>> +                                       temperature = <90000>;

>> +                                       hysteresis = <1000>;

>> +                                       type = "hot";

>> +                               };

>> +                       };

>> +               };

>> +

>> +               // TODO: What is the NSP subsystem?

> 

> This comment should not have been included, will remove in v2


[ ... ]

>> +                       trips {

>> +                               nspss2_alert0: trip-point0 {

>> +                                       temperature = <90000>;

>> +                                       hysteresis = <1000>;

>> +                                       type = "hot";

>> +                               };

>> +                       };

>> +               };

>> +

>> +               // TODO: What is the NSP subsystem?

> 

> This comment should not have been included, will remove in v2

> 


[ ... ]



-- 
<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
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
index 95462e071ab4..e788378eff8d 100644
--- a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
+++ b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
@@ -43,6 +43,7 @@  properties:
               - qcom,sdm845-tsens
               - qcom,sm8150-tsens
               - qcom,sm8250-tsens
+              - qcom,sm8350-tsens
           - const: qcom,tsens-v2
 
   reg: