Re: [PATCH] arm64: dts: qcom: sc8280xp: Introduce additional tsens instances

From: Konrad Dybcio
Date: Fri Jan 26 2024 - 05:48:24 EST




On 1/26/24 11:00, Johan Hovold wrote:
On Fri, Jan 19, 2024 at 12:31:06AM +0100, Konrad Dybcio wrote:
On 1/19/24 00:00, Bjorn Andersson wrote:

+ gpu-thermal {
+ polling-delay-passive = <250>;
+ polling-delay = <1000>;

Hm, did tsens only gain support of non-polled reporting with 8450?

If not, we should definitely update all the relevant SoCs.

Did you determine whether the interrupts work as expected?

Yes, they seem to!


We don't want to be polling here unless we have to (i.e. polling-delay
should be 0 as per the binding).

Even better:

https://lore.kernel.org/lkml/20240125-topic-thermal-v1-0-3c9d4dced138@xxxxxxxxxx/

Believe me, I'm not omitting you from Cc on purpose.. I usually
realize I forgot right after pressing enter on "do you really
wanna send"..

Konrad