linux-kernel - Re: [PATCH v4] thermal: k3_j72xx_bandgap: implement suspend/resume support

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] [day] [month] [year] [list]

Message-ID: <854d1b7b-074a-4bfb-8dec-5432a832e221@linaro.org>
Date: Mon, 3 Jun 2024 11:41:54 +0200
From: Daniel Lezcano <daniel.lezcano@...aro.org>
To: Thomas Richard <thomas.richard@...tlin.com>, rafael@...nel.org
Cc: rui.zhang@...el.com, lukasz.luba@....com, linux-pm@...r.kernel.org,
 linux-kernel@...r.kernel.org, gregory.clement@...tlin.com,
 thomas.petazzoni@...tlin.com, theo.lebrun@...tlin.com, u-kumar1@...com,
 j-keerthy@...com
Subject: Re: [PATCH v4] thermal: k3_j72xx_bandgap: implement suspend/resume
 support

On 25/04/2024 17:32, Thomas Richard wrote:
> From: Théo Lebrun <theo.lebrun@...tlin.com>
> 
> This add suspend-to-ram support.
> 
> The derived_table is kept-as is, so the resume is only about
> pm_runtime_* calls and restoring the same registers as the probe.
> 
> Extract the hardware initialization procedure to a function called at
> both probe-time & resume-time.
> 
> The probe-time loop is split in two to ensure doing the hardware
> initialization before registering thermal zones. That ensures our
> callbacks cannot be called while in bad state.
> 
> The 100ms delay in the hardware initialization sequence was removed.
> It was initially added to be sure the thresholds are programmed before
> enabling the interrupt, but in fact it's not needed (tested on J7200
> platform).
> 
> Signed-off-by: Théo Lebrun <theo.lebrun@...tlin.com>
> Acked-by: Keerthy <j-keerthy@...com>
> Signed-off-by: Thomas Richard <thomas.richard@...tlin.com>
> ---

Applied, sorry for the delay

-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog