linux-kernel - Re: [PATCH] arm64: dts: meson: vim3: reduce cpu thermal fan trigger temperature

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 11 Jun 2021 09:15:13 +0200
From:   Neil Armstrong <narmstrong@...libre.com>
To:     Nick Xie <xieqinick@...il.com>
Cc:     robh+dt@...nel.org, khilman@...libre.com, jbrunet@...libre.com,
        martin.blumenstingl@...glemail.com, devicetree@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org,
        linux-amlogic@...ts.infradead.org, linux-kernel@...r.kernel.org,
        Nick <nick@...das.com>, artem@...das.com
Subject: Re: [PATCH] arm64: dts: meson: vim3: reduce cpu thermal fan trigger
 temperature

Hi,

On 11/06/2021 08:54, Nick Xie wrote:
> Hello Neil,
> 
> The MCU FAN won't work until the temperature reach to 80 degree centigrade,
> Do you think the temperature is too high?

I think 80 is tool high, but I fear 50 is way too low, but if you think 50 is OK after you ran some tests, just add it to the commit log and it's ok for me.

Neil

> 
> Thanks.
> 
> Neil Armstrong <narmstrong@...libre.com <mailto:narmstrong@...libre.com>> 于2021年6月11日周五 下午2:44写道：
> 
>     Hi Nick,
> 
>     On 09/06/2021 03:28, xieqinick@...il.com <mailto:xieqinick@...il.com> wrote:
>     > From: Nick Xie <nick@...das.com <mailto:nick@...das.com>>
>     >
>     > Reduce the MCU FAN trigger temperature from
>     > 80 degree centigrade to 50 degree centigrade.
> 
>     Can you add some more rationale to this change ?
> 
>     >
>     > Signed-off-by: Nick Xie <nick@...das.com <mailto:nick@...das.com>>
>     > ---
>     >  arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi | 2 +-
>     >  1 file changed, 1 insertion(+), 1 deletion(-)
>     >
>     > diff --git a/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi b/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
>     > index 66d67524b031..a9c34fee91f4 100644
>     > --- a/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
>     > +++ b/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
>     > @@ -256,7 +256,7 @@ &cecb_AO {
>     >  &cpu_thermal {
>     >       trips {
>     >               cpu_active: cpu-active {
>     > -                     temperature = <80000>; /* millicelsius */
>     > +                     temperature = <50000>; /* millicelsius */
>     >                       hysteresis = <2000>; /* millicelsius */
>     >                       type = "active";
>     >               };
>     >
> 
>     With that:
>     Reviewed-by: Neil Armstrong <narmstrong@...libre.com <mailto:narmstrong@...libre.com>>
>