linux-kernel - Re: [PATCH V4 4/4] thermal/drivers/cpu_cooling: Rename to cpufreq

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]

Date:   Thu, 12 Dec 2019 08:34:41 +0100
From:   Martin Kepplinger <martin.kepplinger@...i.sm>
To:     Daniel Lezcano <daniel.lezcano@...aro.org>, edubezval@...il.com,
        rui.zhang@...el.com
Cc:     rjw@...ysocki.net, linux-pm@...r.kernel.org,
        viresh.kumar@...aro.org, amit.kucheria@...aro.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH V4 4/4] thermal/drivers/cpu_cooling: Rename to
 cpufreq_cooling



On 11.12.19 22:33, Daniel Lezcano wrote:
> 
> Hi Martin,
> 
> I've a bug in the code.
> 
> 
> diff --git a/drivers/thermal/cpuidle_cooling.c
> b/drivers/thermal/cpuidle_cooling.c
> index 369c5c613f6b..628ad707f247 100644
> --- a/drivers/thermal/cpuidle_cooling.c
> +++ b/drivers/thermal/cpuidle_cooling.c
> @@ -192,7 +192,7 @@ __init cpuidle_of_cooling_register(struct
> device_node *np,
>                 goto out_id;
>         }
> 
> -       idle_inject_set_duration(ii_dev, 0, TICK_USEC);
> +       idle_inject_set_duration(ii_dev, TICK_USEC, TICK_USEC);
> 
>         idle_cdev->ii_dev = ii_dev;
> 
> 
> Let me know if that solves your issue.
> 

It does. I now won't heat up over the hot trip point at all. That's what
I was expecting. thanks!

                                   martin