On 4/16/25 10:33 PM, barnabas.cze...@mainlining.org wrote:
> On 2025-04-14 22:55, Konrad Dybcio wrote:
>> On 3/15/25 3:57 PM, Barnabás Czémán wrote:
>>> From: Dang Huynh <danc...@riseup.net>
>>>
>>> Add initial support for MSM8937 SoC.
>>>
>>> Signed-off-by: Dang Huynh <danc...@riseup.net>
>>> Co-developed-by: Barnabás Czémán <barnabas.cze...@mainlining.org>
>>> Signed-off-by: Barnabás Czémán <barnabas.cze...@mainlining.org>
>>> ---
[...]
>>> + thermal_zones: thermal-zones {
>>> + aoss-thermal {
>>> + polling-delay-passive = <250>;
>>
>> There are no passive trip points> +
> Should i remove polling-delay-passive?
yes, please do
>>> + thermal-sensors = <&tsens 0>;
>>> +
>>> + trips {
>>> + aoss_alert0: trip-point0 {
>>> + temperature = <85000>;
>>> + hysteresis = <2000>;
>>> + type = "hot";
>>> + };
>>
>> Please convert these to 'critical' instead
>>
>> [...]
>>
>>> + cpuss1-thermal {
>>> + polling-delay-passive = <250>;
>>
>> You can drop polling-delay-passive under CPU tzones, as threshold
>> crossing is interrupt-driven
> Should I remove polling-delay-passive then?
yeah
Konrad
