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: <CAHc4DN+N7oLNiiVL-T+T0dEr2nipph5f2q5zu37NowPQC+=mkw@mail.gmail.com>
Date: Wed, 10 Apr 2024 16:27:53 +0800
From: Hsin-Te Yuan <yuanhsinte@...omium.org>
To: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
Cc: Hsin-Te Yuan <yuanhsinte@...omium.org>, Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>, Conor Dooley <conor+dt@...nel.org>, 
	Matthias Brugger <matthias.bgg@...il.com>, 
	Nícolas F. R. A. Prado <nfraprado@...labora.com>, 
	Bernhard Rosenkränzer <bero@...libre.com>, 
	Balsam CHIHI <bchihi@...libre.com>, Alexandre Mergnat <amergnat@...libre.com>, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, 
	linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH] arm64: dts: mediatek: mt8192: Add missing trip point in
 thermal zone

Hi Angelo,

According to the document
(https://docs.kernel.org/driver-api/thermal/power_allocator.html),
there should be two passive trip points. It seems that the switch-on
temperature will be 0 if we only have one trip point, which hurts
performance. I'll send v2 to explain this more clearly.

Regards,
Hsin-Te


On Wed, Apr 10, 2024 at 3:43 PM AngeloGioacchino Del Regno
<angelogioacchino.delregno@...labora.com> wrote:
>
> Il 10/04/24 08:20, Hsin-Te Yuan ha scritto:
> > Add the missing passive trip point which is expected by kernel.
> >
> > Fixes: c7a728051f4e ("arm64: dts: mediatek: mt8192: Add thermal nodes and thermal zones")
> > Signed-off-by: Hsin-Te Yuan <yuanhsinte@...omium.org>
>
> Sorry, I don't understand what you're trying to solve here.
>
> All of the thermal zones in mt8192.dtsi already do have a passive trip point
> which is also used in cooling-maps.
>
> Can you please describe the issue?
>
> Thanks,
> Angelo
>
> > ---
> >   arch/arm64/boot/dts/mediatek/mt8192.dtsi | 40 ++++++++++++++++++++++++++++++++
> >   1 file changed, 40 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/mediatek/mt8192.dtsi b/arch/arm64/boot/dts/mediatek/mt8192.dtsi
> > index 05e401670bced..08d8bccc84669 100644
> > --- a/arch/arm64/boot/dts/mediatek/mt8192.dtsi
> > +++ b/arch/arm64/boot/dts/mediatek/mt8192.dtsi
> > @@ -1959,6 +1959,11 @@ cpu0-thermal {
> >                       thermal-sensors = <&lvts_mcu MT8192_MCU_LITTLE_CPU0>;
> >
> >                       trips {
> > +                             cpu0_thres: trip-point {
> > +                                     temperature = <68000>;
> > +                                     hysteresis = <2000>;
> > +                                     type = "passive";
> > +                             };
> >                               cpu0_alert: trip-alert {
> >                                       temperature = <85000>;
> >                                       hysteresis = <2000>;
> > @@ -1989,6 +1994,11 @@ cpu1-thermal {
> >                       thermal-sensors = <&lvts_mcu MT8192_MCU_LITTLE_CPU1>;
> >
> >                       trips {
> > +                             cpu1_thres: trip-point {
> > +                                     temperature = <68000>;
> > +                                     hysteresis = <2000>;
> > +                                     type = "passive";
> > +                             };
> >                               cpu1_alert: trip-alert {
> >                                       temperature = <85000>;
> >                                       hysteresis = <2000>;
> > @@ -2019,6 +2029,11 @@ cpu2-thermal {
> >                       thermal-sensors = <&lvts_mcu MT8192_MCU_LITTLE_CPU2>;
> >
> >                       trips {
> > +                             cpu2_thres: trip-point {
> > +                                     temperature = <68000>;
> > +                                     hysteresis = <2000>;
> > +                                     type = "passive";
> > +                             };
> >                               cpu2_alert: trip-alert {
> >                                       temperature = <85000>;
> >                                       hysteresis = <2000>;
> > @@ -2049,6 +2064,11 @@ cpu3-thermal {
> >                       thermal-sensors = <&lvts_mcu MT8192_MCU_LITTLE_CPU3>;
> >
> >                       trips {
> > +                             cpu3_thres: trip-point {
> > +                                     temperature = <68000>;
> > +                                     hysteresis = <2000>;
> > +                                     type = "passive";
> > +                             };
> >                               cpu3_alert: trip-alert {
> >                                       temperature = <85000>;
> >                                       hysteresis = <2000>;
> > @@ -2079,6 +2099,11 @@ cpu4-thermal {
> >                       thermal-sensors = <&lvts_mcu MT8192_MCU_BIG_CPU0>;
> >
> >                       trips {
> > +                             cpu4_thres: trip-point {
> > +                                     temperature = <68000>;
> > +                                     hysteresis = <2000>;
> > +                                     type = "passive";
> > +                             };
> >                               cpu4_alert: trip-alert {
> >                                       temperature = <85000>;
> >                                       hysteresis = <2000>;
> > @@ -2109,6 +2134,11 @@ cpu5-thermal {
> >                       thermal-sensors = <&lvts_mcu MT8192_MCU_BIG_CPU1>;
> >
> >                       trips {
> > +                             cpu5_thres: trip-point {
> > +                                     temperature = <68000>;
> > +                                     hysteresis = <2000>;
> > +                                     type = "passive";
> > +                             };
> >                               cpu5_alert: trip-alert {
> >                                       temperature = <85000>;
> >                                       hysteresis = <2000>;
> > @@ -2139,6 +2169,11 @@ cpu6-thermal {
> >                       thermal-sensors = <&lvts_mcu MT8192_MCU_BIG_CPU2>;
> >
> >                       trips {
> > +                             cpu6_thres: trip-point {
> > +                                     temperature = <68000>;
> > +                                     hysteresis = <2000>;
> > +                                     type = "passive";
> > +                             };
> >                               cpu6_alert: trip-alert {
> >                                       temperature = <85000>;
> >                                       hysteresis = <2000>;
> > @@ -2169,6 +2204,11 @@ cpu7-thermal {
> >                       thermal-sensors = <&lvts_mcu MT8192_MCU_BIG_CPU3>;
> >
> >                       trips {
> > +                             cpu7_thres: trip-point {
> > +                                     temperature = <68000>;
> > +                                     hysteresis = <2000>;
> > +                                     type = "passive";
> > +                             };
> >                               cpu7_alert: trip-alert {
> >                                       temperature = <85000>;
> >                                       hysteresis = <2000>;
> >
> > ---
> > base-commit: 20cb38a7af88dc40095da7c2c9094da3873fea23
> > change-id: 20240410-upstream-torvalds-master-40aeff5416c7
> >
> > Best regards,
>

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ