linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 0/1] arm64: dts: mediatek: mt8183: disable thermal zones without trips.
@ 2022-09-21  9:05 Amjad Ouled-Ameur
  2022-09-21  9:05 ` [PATCH v2 1/1] " Amjad Ouled-Ameur
  2022-09-21  9:16 ` [PATCH v2 0/1] " Krzysztof Kozlowski
  0 siblings, 2 replies; 5+ messages in thread
From: Amjad Ouled-Ameur @ 2022-09-21  9:05 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Matthias Brugger, Rob Herring
  Cc: linux-mediatek, linux-kernel, devicetree, linux-arm-kernel,
	Amjad Ouled-Ameur, daniel.lezcano

Thermal zones without trip point are not registered by thermal core.

tzts1 ~ tzts6 zones of mt8183 were intially introduced for test-purpose
only.

Disable the zones above and keep only cpu_thermal enabled.

Signed-off-by: Amjad Ouled-Ameur <aouledameur@baylibre.com>
---
Amjad Ouled-Ameur (1):
      arm64: dts: mediatek: mt8183: disable thermal zones without trips.

 arch/arm64/boot/dts/mediatek/mt8183.dtsi | 6 ++++++
 1 file changed, 6 insertions(+)
---
base-commit: 60891ec99e141b74544d11e897a245ef06263052
change-id: 20220921-mt8183-disables-thermal-zones-up-6ac720a305d5

Best regards,
-- 
Amjad Ouled-Ameur <aouledameur@baylibre.com>

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH v2 1/1] arm64: dts: mediatek: mt8183: disable thermal zones without trips.
  2022-09-21  9:05 [PATCH v2 0/1] arm64: dts: mediatek: mt8183: disable thermal zones without trips Amjad Ouled-Ameur
@ 2022-09-21  9:05 ` Amjad Ouled-Ameur
  2022-09-25  7:57   ` Krzysztof Kozlowski
  2022-09-21  9:16 ` [PATCH v2 0/1] " Krzysztof Kozlowski
  1 sibling, 1 reply; 5+ messages in thread
From: Amjad Ouled-Ameur @ 2022-09-21  9:05 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Matthias Brugger, Rob Herring
  Cc: linux-mediatek, linux-kernel, devicetree, linux-arm-kernel,
	Amjad Ouled-Ameur, daniel.lezcano

Thermal zones without trip point are not registered by thermal core.

tzts1 ~ tzts6 zones of mt8183 were intially introduced for test-purpose
only.

Disable the zones above and keep only cpu_thermal enabled.

Signed-off-by: Amjad Ouled-Ameur <aouledameur@baylibre.com>

diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
index 9d32871973a2..44a197e92c99 100644
--- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
@@ -1193,6 +1193,7 @@ tzts1: tzts1 {
 				sustainable-power = <5000>;
 				trips {};
 				cooling-maps {};
+				status = "disabled";
 			};
 
 			tzts2: tzts2 {
@@ -1202,6 +1203,7 @@ tzts2: tzts2 {
 				sustainable-power = <5000>;
 				trips {};
 				cooling-maps {};
+				status = "disabled";
 			};
 
 			tzts3: tzts3 {
@@ -1211,6 +1213,7 @@ tzts3: tzts3 {
 				sustainable-power = <5000>;
 				trips {};
 				cooling-maps {};
+				status = "disabled";
 			};
 
 			tzts4: tzts4 {
@@ -1220,6 +1223,7 @@ tzts4: tzts4 {
 				sustainable-power = <5000>;
 				trips {};
 				cooling-maps {};
+				status = "disabled";
 			};
 
 			tzts5: tzts5 {
@@ -1229,6 +1233,7 @@ tzts5: tzts5 {
 				sustainable-power = <5000>;
 				trips {};
 				cooling-maps {};
+				status = "disabled";
 			};
 
 			tztsABB: tztsABB {
@@ -1238,6 +1243,7 @@ tztsABB: tztsABB {
 				sustainable-power = <5000>;
 				trips {};
 				cooling-maps {};
+				status = "disabled";
 			};
 		};
 

-- 
b4 0.10.0

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH v2 0/1] arm64: dts: mediatek: mt8183: disable thermal zones without trips.
  2022-09-21  9:05 [PATCH v2 0/1] arm64: dts: mediatek: mt8183: disable thermal zones without trips Amjad Ouled-Ameur
  2022-09-21  9:05 ` [PATCH v2 1/1] " Amjad Ouled-Ameur
@ 2022-09-21  9:16 ` Krzysztof Kozlowski
  1 sibling, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-21  9:16 UTC (permalink / raw)
  To: Amjad Ouled-Ameur, Krzysztof Kozlowski, Matthias Brugger, Rob Herring
  Cc: linux-mediatek, linux-kernel, devicetree, linux-arm-kernel,
	daniel.lezcano

On 21/09/2022 11:05, Amjad Ouled-Ameur wrote:
> Thermal zones without trip point are not registered by thermal core.
> 
> tzts1 ~ tzts6 zones of mt8183 were intially introduced for test-purpose

No need for cover letter for single patch.

Best regards,
Krzysztof


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2 1/1] arm64: dts: mediatek: mt8183: disable thermal zones without trips.
  2022-09-21  9:05 ` [PATCH v2 1/1] " Amjad Ouled-Ameur
@ 2022-09-25  7:57   ` Krzysztof Kozlowski
  2022-09-25  8:20     ` Amjad Ouled-Ameur
  0 siblings, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-25  7:57 UTC (permalink / raw)
  To: Amjad Ouled-Ameur, Krzysztof Kozlowski, Matthias Brugger, Rob Herring
  Cc: linux-mediatek, linux-kernel, devicetree, linux-arm-kernel,
	daniel.lezcano

On 21/09/2022 11:05, Amjad Ouled-Ameur wrote:
> Thermal zones without trip point are not registered by thermal core.
> 
> tzts1 ~ tzts6 zones of mt8183 were intially introduced for test-purpose
> only.
> 
> Disable the zones above and keep only cpu_thermal enabled.

Please test your patches before sending upstream. :(

Best regards,
Krzysztof


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2 1/1] arm64: dts: mediatek: mt8183: disable thermal zones without trips.
  2022-09-25  7:57   ` Krzysztof Kozlowski
@ 2022-09-25  8:20     ` Amjad Ouled-Ameur
  0 siblings, 0 replies; 5+ messages in thread
From: Amjad Ouled-Ameur @ 2022-09-25  8:20 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Krzysztof Kozlowski, Matthias Brugger, Rob Herring
  Cc: linux-mediatek, linux-kernel, devicetree, linux-arm-kernel,
	daniel.lezcano

Hi Krzysztof,

On 9/25/22 09:57, Krzysztof Kozlowski wrote:
> On 21/09/2022 11:05, Amjad Ouled-Ameur wrote:
>> Thermal zones without trip point are not registered by thermal core.
>>
>> tzts1 ~ tzts6 zones of mt8183 were intially introduced for test-purpose
>> only.
>>
>> Disable the zones above and keep only cpu_thermal enabled.
> Please test your patches before sending upstream. :(
My apologies for the inconvenience, I mistakenly missed the error,

will send a proper V3.


Regards,

Amjad

> Best regards,
> Krzysztof
>

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2022-09-25  8:22 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-21  9:05 [PATCH v2 0/1] arm64: dts: mediatek: mt8183: disable thermal zones without trips Amjad Ouled-Ameur
2022-09-21  9:05 ` [PATCH v2 1/1] " Amjad Ouled-Ameur
2022-09-25  7:57   ` Krzysztof Kozlowski
2022-09-25  8:20     ` Amjad Ouled-Ameur
2022-09-21  9:16 ` [PATCH v2 0/1] " Krzysztof Kozlowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).