linux-kernel - Re: [PATCH v10] arm64: dts: qcom: ipq5018: Add tsens node

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID:
 <DS7PR19MB8883150D952EC3C753211B7F9D6AA@DS7PR19MB8883.namprd19.prod.outlook.com>
Date: Tue, 10 Jun 2025 18:21:26 +0400
From: George Moussalem <george.moussalem@...look.com>
To: Konrad Dybcio <konrad.dybcio@....qualcomm.com>,
 Amit Kucheria <amitk@...nel.org>, Thara Gopinath <thara.gopinath@...il.com>,
 "Rafael J. Wysocki" <rafael@...nel.org>,
 Daniel Lezcano <daniel.lezcano@...aro.org>, Zhang Rui <rui.zhang@...el.com>,
 Lukasz Luba <lukasz.luba@....com>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Bjorn Andersson <andersson@...nel.org>,
 Konrad Dybcio <konradybcio@...nel.org>
Cc: linux-arm-msm@...r.kernel.org, linux-pm@...r.kernel.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
 Sricharan Ramabadhran <quic_srichara@...cinc.com>,
 Dmitry Baryshkov <lumag@...nel.org>
Subject: Re: [PATCH v10] arm64: dts: qcom: ipq5018: Add tsens node

On 6/10/25 18:09, Konrad Dybcio wrote:
> On 6/10/25 4:06 PM, George Moussalem wrote:
>>
>>
>> On 6/10/25 17:08, Konrad Dybcio wrote:
>>> On 6/10/25 3:03 PM, George Moussalem via B4 Relay wrote:
>>>> From: Sricharan Ramabadhran <quic_srichara@...cinc.com>
>>>>
>>>> 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). With the
>>>> critical temperature being 120'C and action is to reboot.
>>>>
>>>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
>>>> Signed-off-by: Sricharan Ramabadhran <quic_srichara@...cinc.com>
>>>> Signed-off-by: George Moussalem <george.moussalem@...look.com>
>>>> ---
> 
> [...]
> 
>>>> +        gephy-thermal {
>>>> +            polling-delay-passive = <0>;
>>>> +            polling-delay = <0>;
>>>> +            thermal-sensors = <&tsens 4>;
>>>> +
>>>> +            trips {
>>>> +                gephy-critical {
>>>> +                    temperature = <120000>;
>>>
>>> I'm not sure whether there's any firmware/hardware measure to shut
>>> down these beforehand. It's better to have a software trip at 120C
>>> than to not have any at all, but you may want to try and find a
>>> case_therm or so thermistor for your devices..
>>
>> these are the temps as defined for the critical trips by qcom. IPQ8074 has the temp set at 110C, while IPQ6018 even higher at 125C.
>> They've also defined lower temp trips in the downstream DTS files.
>> Should I try and add those? I could check if the driver supports them..
> 
> you can register the CPUs as a cooling device and add some sort of a
> trip point (see e.g. ipq6018.dtsi)

great, will add and test if the driver registers them appropriately.
Will use 100C as the passive trip temp which is the next value lower 
than the critical trip defined in the downstream dtsi.

> 
> Konrad

Thanks,
George