[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b4dba1d5-448a-4a4b-94d5-f27c6ff0010d@linaro.org>
Date: Fri, 10 May 2024 13:49:56 +0100
From: Bryan O'Donoghue <bryan.odonoghue@...aro.org>
To: Konrad Dybcio <konrad.dybcio@...aro.org>,
Bjorn Andersson <andersson@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Conor Dooley <conor+dt@...nel.org>, cros-qcom-dts-watchers@...omium.org,
Krzysztof Kozlowski <krzk+dt@...nel.org>
Cc: linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 00/31] Clean up thermal zone polling-delay
On 10/05/2024 12:59, Konrad Dybcio wrote:
> A trivial follow-up on the changes introduced in Commit 488164006a28
> ("thermal/of: Assume polling-delay(-passive) 0 when absent").
>
> Should probably wait until v6.9-rc1 so that the patch in question is
> in the base tree, otherwise TZs will fail to register.
>
> FWIW, Compile-tested only (except 8280).
>
> To: Bjorn Andersson <andersson@...nel.org>
> To: Rob Herring <robh@...nel.org>
> To: Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>
> To: Conor Dooley <conor+dt@...nel.org>
> To: cros-qcom-dts-watchers@...omium.org
> To: Krzysztof Kozlowski <krzk+dt@...nel.org>
> Cc: linux-arm-msm@...r.kernel.org
> Cc: devicetree@...r.kernel.org
> Cc: linux-kernel@...r.kernel.org
> Signed-off-by: Konrad Dybcio <konrad.dybcio@...aro.org>
>
> Changes in v2:
> - Un-drop passive delays. Whether they're useful where they're enabled
> is a topic for another patchset, as it requires examination on a case-
> -by-case basis.
> - Better unify the style (newlines between properties)
> - Link to v1: https://lore.kernel.org/r/20240319-topic-msm-polling-cleanup-v1-0-e0aee1dbcd78@linaro.org
So perhaps you can answer the question I have.
Right now, we have non-zero delay values, doesn't this mean the thermal
framework driver has a delay between evaluating dT/dt values per
Documentation/devicetree/bindings/thermal/thermal-zones.yaml
Your commit log implies or my reading of it is, there's no functional
change because its currently driven by an IRQ but, is that actually _so_
with non-zero values in the DT?
---
bod
Powered by blists - more mailing lists