Re: [PATCH v1] arm64: dts: qcom: ipq5424: add cooling maps for CPU thermal zones

[Manikanta Mylavarapu <quic_mmanikan@quicinc.com>]

On 10/23/2025 2:09 PM, Konrad Dybcio wrote:
> On 10/23/25 6:38 AM, Manikanta Mylavarapu wrote:
>> Add cooling-maps to the cpu1, cpu2, and cpu3 thermal zones to associate
>> passive trip points with CPU cooling devices. This enables proper
>> thermal mitigation by allowing the thermal framework to throttle CPUs
>> based on temperature thresholds. Also, label the trip points to allow
>> referencing them in the cooling maps.
>>
>> Signed-off-by: Manikanta Mylavarapu <quic_mmanikan@quicinc.com>
>> ---
> 
> [...]
> 
>>  		cpu1-thermal {
>> @@ -1254,18 +1269,28 @@ cpu1-thermal {
>>  			thermal-sensors = <&tsens 12>;
>>  
>>  			trips {
>> -				cpu-critical {
>> +				cpu1_crit: cpu-critical {
>>  					temperature = <120000>;
>>  					hysteresis = <9000>;
>>  					type = "critical";
>>  				};
>>  
>> -				cpu-passive {
>> +				cpu1_alert: cpu-passive {
>>  					temperature = <110000>;
>>  					hysteresis = <9000>;
>>  					type = "passive";
>>  				};
> 
> This means "software will start throttling the cpufreq at 110C"
> 
> Is this what you want?
> 

Hi Konrad,

Thank you for reviewing the patch.

Yes, the intent is to initiate software-based cpufreq throttling at 110°C.

> Also, is there nothing like LMH on this platform that throttles the
> device way earlier, without OS intervention?
> 

IPQ5424 platform does not support LMH.

Thanks & Regards,
Manikanta.