Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1557968 > unrolled thread

[PATCH] arm64: dts: mt8173: Fix cpu_thermal cooling-maps contributions

Started byDaniel Kurtz <djkurtz@chromium.org>
First post2017-01-13 03:40 +0100
Last post2017-01-13 16:40 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] arm64: dts: mt8173: Fix cpu_thermal cooling-maps contributions Daniel Kurtz <djkurtz@chromium.org> - 2017-01-13 03:40 +0100
    Re: [PATCH] arm64: dts: mt8173: Fix cpu_thermal cooling-maps  contributions Matthias Brugger <matthias.bgg@gmail.com> - 2017-01-13 16:40 +0100

#1557968 — [PATCH] arm64: dts: mt8173: Fix cpu_thermal cooling-maps contributions

FromDaniel Kurtz <djkurtz@chromium.org>
Date2017-01-13 03:40 +0100
Subject[PATCH] arm64: dts: mt8173: Fix cpu_thermal cooling-maps contributions
Message-ID<sYZWx-3wg-5@gated-at.bofh.it>
According to [0], the contribution field for each cooling-device express
their relative power efficiency. Higher weights express higher power
efficiency.  Weighting is relative such that if each cooling device has a
weight of 1 they are considered equal. This is particularly useful in
heterogeneous systems where two cooling devices may perform the same kind
of compute, but with different efficiency.

[0] Documentation/thermal/power_allocator.txt

According to Mediatek IC designer, the power efficiency ratio between the
LITTLE core cluster (cooling-device cpu0) and big core cluster
(cooling-device cpu1) is around 3:1 (3072:1024).

Signed-off-by: Daniel Kurtz <djkurtz@chromium.org>
---
 arch/arm64/boot/dts/mediatek/mt8173.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
index 12e702771f5c..9a3b0d20f7a8 100644
--- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
@@ -182,12 +182,12 @@
 				map@0 {
 					trip = <&target>;
 					cooling-device = <&cpu0 0 0>;
-					contribution = <1024>;
+					contribution = <3072>;
 				};
 				map@1 {
 					trip = <&target>;
 					cooling-device = <&cpu2 0 0>;
-					contribution = <2048>;
+					contribution = <1024>;
 				};
 			};
 		};
-- 
2.11.0.390.gc69c2f50cf-goog

[toc] | [next] | [standalone]


#1558507 — Re: [PATCH] arm64: dts: mt8173: Fix cpu_thermal cooling-maps contributions

FromMatthias Brugger <matthias.bgg@gmail.com>
Date2017-01-13 16:40 +0100
SubjectRe: [PATCH] arm64: dts: mt8173: Fix cpu_thermal cooling-maps contributions
Message-ID<sZc7u-2zn-45@gated-at.bofh.it>
In reply to#1557968

On 13/01/17 03:30, Daniel Kurtz wrote:
> According to [0], the contribution field for each cooling-device express
> their relative power efficiency. Higher weights express higher power
> efficiency.  Weighting is relative such that if each cooling device has a
> weight of 1 they are considered equal. This is particularly useful in
> heterogeneous systems where two cooling devices may perform the same kind
> of compute, but with different efficiency.
>
> [0] Documentation/thermal/power_allocator.txt
>
> According to Mediatek IC designer, the power efficiency ratio between the
> LITTLE core cluster (cooling-device cpu0) and big core cluster
> (cooling-device cpu1) is around 3:1 (3072:1024).
>
> Signed-off-by: Daniel Kurtz <djkurtz@chromium.org>
> ---
>  arch/arm64/boot/dts/mediatek/mt8173.dtsi | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>

applied, thanks.

> diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> index 12e702771f5c..9a3b0d20f7a8 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> @@ -182,12 +182,12 @@
>  				map@0 {
>  					trip = <&target>;
>  					cooling-device = <&cpu0 0 0>;
> -					contribution = <1024>;
> +					contribution = <3072>;
>  				};
>  				map@1 {
>  					trip = <&target>;
>  					cooling-device = <&cpu2 0 0>;
> -					contribution = <2048>;
> +					contribution = <1024>;
>  				};
>  			};
>  		};
>

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web