[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240125-wildfowl-confined-25c6b276cdcb@spud>
Date: Thu, 25 Jan 2024 16:51:11 +0000
From: Conor Dooley <conor@...nel.org>
To: Konrad Dybcio <konrad.dybcio@...aro.org>
Cc: "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+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Conor Dooley <conor+dt@...nel.org>,
Bjorn Andersson <andersson@...nel.org>, linux-pm@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] dt-bindings: thermal-zones: Don't require
polling-delay(-passive)
On Thu, Jan 25, 2024 at 01:11:15PM +0100, Konrad Dybcio wrote:
> Currently, thermal zones associated with providers that have interrupts
> for signaling hot/critical trips are required to set a polling-delay
> of 0 to indicate no polling. This feels a bit backwards.
>
> Assume 0 (no polling) when these properties are not defined.
>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@...aro.org>
I think that makes sense.
Reviewed-by: Conor Dooley <conor.dooley@...rochip.com>
Cheers,
Conor.
> ---
> Documentation/devicetree/bindings/thermal/thermal-zones.yaml | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/thermal/thermal-zones.yaml b/Documentation/devicetree/bindings/thermal/thermal-zones.yaml
> index dbd52620d293..68398e7e8655 100644
> --- a/Documentation/devicetree/bindings/thermal/thermal-zones.yaml
> +++ b/Documentation/devicetree/bindings/thermal/thermal-zones.yaml
> @@ -228,8 +228,6 @@ patternProperties:
> additionalProperties: false
>
> required:
> - - polling-delay
> - - polling-delay-passive
> - thermal-sensors
> - trips
>
>
> --
> 2.40.1
>
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists