Re: [PATCH v12 2/2] arm64: dts: qcom: ipq5018: Add tsens node

From: Konrad Dybcio
Date: Wed Jun 11 2025 - 11:09:56 EST


On 6/11/25 11:33 AM, George Moussalem via B4 Relay wrote:
> From: Sricharan Ramabadhran <quic_srichara@xxxxxxxxxxx>
>
> IPQ5018 has tsens V1.0 IP with 5 sensors, though 4 are in use.
> There is no RPM, so tsens has to be manually enabled. Adding the tsens
> and nvmem nodes and adding 4 thermal sensors (zones). The critical trip
> temperature is set to 120'C with an action to reboot.
>
> In addition, adding a cooling device to the CPU thermal zone which uses
> CPU frequency scaling.
>
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
> Signed-off-by: Sricharan Ramabadhran <quic_srichara@xxxxxxxxxxx>
> Signed-off-by: George Moussalem <george.moussalem@xxxxxxxxxxx>
> ---

[...]

> + cpu-thermal {
> + polling-delay-passive = <0>;

0 is also the default value for polling-delay-passive


> + thermal-sensors = <&tsens 2>;
> +
> + trips {
> + cpu-critical {
> + temperature = <120000>;
> + hysteresis = <2>;
> + type = "critical";
> + };
> +
> + cpu_alert: cpu-passive {
> + temperature = <100000>;
> + hysteresis = <2>;

that's 2 milicelcius, consider x1000

Konrad