* [PATCH] arm64: dts: mediatek: mt8183: remove thermal zones without trips.
@ 2022-09-14 13:13 Amjad Ouled-Ameur
2022-09-14 14:15 ` Matthias Brugger
0 siblings, 1 reply; 5+ messages in thread
From: Amjad Ouled-Ameur @ 2022-09-14 13:13 UTC (permalink / raw)
To: Rob Herring
Cc: Amjad Ouled-Ameur, Krzysztof Kozlowski, Matthias Brugger,
devicetree, linux-arm-kernel, linux-mediatek, linux-kernel
Thermal zones without trip point are not registered by thermal core.
tzts1 ~ tzts6 zones of mt8183 were intially introduced for test-purpose
only but are not supposed to remain on DT.
Remove the zones above and keep only cpu_thermal.
Signed-off-by: Amjad Ouled-Ameur <aouledameur@baylibre.com>
---
arch/arm64/boot/dts/mediatek/mt8183.dtsi | 57 ------------------------
1 file changed, 57 deletions(-)
diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
index 9d32871973a2..f65fae8939de 100644
--- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
@@ -1182,63 +1182,6 @@ THERMAL_NO_LIMIT
};
};
};
-
- /* The tzts1 ~ tzts6 don't need to polling */
- /* The tzts1 ~ tzts6 don't need to thermal throttle */
-
- tzts1: tzts1 {
- polling-delay-passive = <0>;
- polling-delay = <0>;
- thermal-sensors = <&thermal 1>;
- sustainable-power = <5000>;
- trips {};
- cooling-maps {};
- };
-
- tzts2: tzts2 {
- polling-delay-passive = <0>;
- polling-delay = <0>;
- thermal-sensors = <&thermal 2>;
- sustainable-power = <5000>;
- trips {};
- cooling-maps {};
- };
-
- tzts3: tzts3 {
- polling-delay-passive = <0>;
- polling-delay = <0>;
- thermal-sensors = <&thermal 3>;
- sustainable-power = <5000>;
- trips {};
- cooling-maps {};
- };
-
- tzts4: tzts4 {
- polling-delay-passive = <0>;
- polling-delay = <0>;
- thermal-sensors = <&thermal 4>;
- sustainable-power = <5000>;
- trips {};
- cooling-maps {};
- };
-
- tzts5: tzts5 {
- polling-delay-passive = <0>;
- polling-delay = <0>;
- thermal-sensors = <&thermal 5>;
- sustainable-power = <5000>;
- trips {};
- cooling-maps {};
- };
-
- tztsABB: tztsABB {
- polling-delay-passive = <0>;
- polling-delay = <0>;
- thermal-sensors = <&thermal 6>;
- sustainable-power = <5000>;
- trips {};
- cooling-maps {};
- };
};
pwm0: pwm@1100e000 {
--
2.37.3
_______________________________________________
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] arm64: dts: mediatek: mt8183: remove thermal zones without trips.
2022-09-14 13:13 [PATCH] arm64: dts: mediatek: mt8183: remove thermal zones without trips Amjad Ouled-Ameur
@ 2022-09-14 14:15 ` Matthias Brugger
2022-09-14 19:44 ` Amjad Ouled-Ameur
2022-09-15 15:50 ` Daniel Lezcano
0 siblings, 2 replies; 5+ messages in thread
From: Matthias Brugger @ 2022-09-14 14:15 UTC (permalink / raw)
To: Amjad Ouled-Ameur, Rob Herring
Cc: Krzysztof Kozlowski, devicetree, linux-arm-kernel,
linux-mediatek, linux-kernel
On 14/09/2022 15:13, 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 but are not supposed to remain on DT.
>
> Remove the zones above and keep only cpu_thermal.
>
My understanding is that this thermal zones exist but are not used. DTS should
describe the HW as it is, so I propose to add
status = "disabled";
to the zones.
Regards,
Matthias
> Signed-off-by: Amjad Ouled-Ameur <aouledameur@baylibre.com>
> ---
> arch/arm64/boot/dts/mediatek/mt8183.dtsi | 57 ------------------------
> 1 file changed, 57 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> index 9d32871973a2..f65fae8939de 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> @@ -1182,63 +1182,6 @@ THERMAL_NO_LIMIT
> };
> };
> };
> -
> - /* The tzts1 ~ tzts6 don't need to polling */
> - /* The tzts1 ~ tzts6 don't need to thermal throttle */
> -
> - tzts1: tzts1 {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> - thermal-sensors = <&thermal 1>;
> - sustainable-power = <5000>;
> - trips {};
> - cooling-maps {};
> - };
> -
> - tzts2: tzts2 {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> - thermal-sensors = <&thermal 2>;
> - sustainable-power = <5000>;
> - trips {};
> - cooling-maps {};
> - };
> -
> - tzts3: tzts3 {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> - thermal-sensors = <&thermal 3>;
> - sustainable-power = <5000>;
> - trips {};
> - cooling-maps {};
> - };
> -
> - tzts4: tzts4 {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> - thermal-sensors = <&thermal 4>;
> - sustainable-power = <5000>;
> - trips {};
> - cooling-maps {};
> - };
> -
> - tzts5: tzts5 {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> - thermal-sensors = <&thermal 5>;
> - sustainable-power = <5000>;
> - trips {};
> - cooling-maps {};
> - };
> -
> - tztsABB: tztsABB {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> - thermal-sensors = <&thermal 6>;
> - sustainable-power = <5000>;
> - trips {};
> - cooling-maps {};
> - };
> };
>
> pwm0: pwm@1100e000 {
_______________________________________________
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] arm64: dts: mediatek: mt8183: remove thermal zones without trips.
2022-09-14 14:15 ` Matthias Brugger
@ 2022-09-14 19:44 ` Amjad Ouled-Ameur
2022-09-15 15:50 ` Daniel Lezcano
1 sibling, 0 replies; 5+ messages in thread
From: Amjad Ouled-Ameur @ 2022-09-14 19:44 UTC (permalink / raw)
To: Matthias Brugger, Rob Herring
Cc: Krzysztof Kozlowski, devicetree, linux-arm-kernel,
linux-mediatek, linux-kernel
Hi Matthias,
Thank you for your review.
On 9/14/22 16:15, Matthias Brugger wrote:
>
>
> On 14/09/2022 15:13, 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 but are not supposed to remain on DT.
>>
>> Remove the zones above and keep only cpu_thermal.
>>
>
> My understanding is that this thermal zones exist but are not used.
> DTS should describe the HW as it is, so I propose to add
> status = "disabled";
> to the zones.
>
Correct, they are unused unless for testing purposes. Thus, your
suggestion of disabling them seems more relevant
than removal.
Regards,
Amjad
> Regards,
> Matthias
>
>> Signed-off-by: Amjad Ouled-Ameur <aouledameur@baylibre.com>
>> ---
>> arch/arm64/boot/dts/mediatek/mt8183.dtsi | 57 ------------------------
>> 1 file changed, 57 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
>> b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
>> index 9d32871973a2..f65fae8939de 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
>> +++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
>> @@ -1182,63 +1182,6 @@ THERMAL_NO_LIMIT
>> };
>> };
>> };
>> -
>> - /* The tzts1 ~ tzts6 don't need to polling */
>> - /* The tzts1 ~ tzts6 don't need to thermal throttle */
>> -
>> - tzts1: tzts1 {
>> - polling-delay-passive = <0>;
>> - polling-delay = <0>;
>> - thermal-sensors = <&thermal 1>;
>> - sustainable-power = <5000>;
>> - trips {};
>> - cooling-maps {};
>> - };
>> -
>> - tzts2: tzts2 {
>> - polling-delay-passive = <0>;
>> - polling-delay = <0>;
>> - thermal-sensors = <&thermal 2>;
>> - sustainable-power = <5000>;
>> - trips {};
>> - cooling-maps {};
>> - };
>> -
>> - tzts3: tzts3 {
>> - polling-delay-passive = <0>;
>> - polling-delay = <0>;
>> - thermal-sensors = <&thermal 3>;
>> - sustainable-power = <5000>;
>> - trips {};
>> - cooling-maps {};
>> - };
>> -
>> - tzts4: tzts4 {
>> - polling-delay-passive = <0>;
>> - polling-delay = <0>;
>> - thermal-sensors = <&thermal 4>;
>> - sustainable-power = <5000>;
>> - trips {};
>> - cooling-maps {};
>> - };
>> -
>> - tzts5: tzts5 {
>> - polling-delay-passive = <0>;
>> - polling-delay = <0>;
>> - thermal-sensors = <&thermal 5>;
>> - sustainable-power = <5000>;
>> - trips {};
>> - cooling-maps {};
>> - };
>> -
>> - tztsABB: tztsABB {
>> - polling-delay-passive = <0>;
>> - polling-delay = <0>;
>> - thermal-sensors = <&thermal 6>;
>> - sustainable-power = <5000>;
>> - trips {};
>> - cooling-maps {};
>> - };
>> };
>> pwm0: pwm@1100e000 {
_______________________________________________
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] arm64: dts: mediatek: mt8183: remove thermal zones without trips.
2022-09-14 14:15 ` Matthias Brugger
2022-09-14 19:44 ` Amjad Ouled-Ameur
@ 2022-09-15 15:50 ` Daniel Lezcano
2022-09-20 15:16 ` Amjad Ouled-Ameur
1 sibling, 1 reply; 5+ messages in thread
From: Daniel Lezcano @ 2022-09-15 15:50 UTC (permalink / raw)
To: Matthias Brugger, Amjad Ouled-Ameur, Rob Herring
Cc: Krzysztof Kozlowski, devicetree, linux-arm-kernel,
linux-mediatek, linux-kernel
On 14/09/2022 16:15, Matthias Brugger wrote:
>
>
> On 14/09/2022 15:13, 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 but are not supposed to remain on DT.
>>
>> Remove the zones above and keep only cpu_thermal.
>>
>
> My understanding is that this thermal zones exist but are not used. DTS
> should describe the HW as it is, so I propose to add
> status = "disabled";
> to the zones.
>
At least the trip points must be declared as stated in the bindings
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/thermal/thermal-zones.yaml#n213
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
_______________________________________________
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] arm64: dts: mediatek: mt8183: remove thermal zones without trips.
2022-09-15 15:50 ` Daniel Lezcano
@ 2022-09-20 15:16 ` Amjad Ouled-Ameur
0 siblings, 0 replies; 5+ messages in thread
From: Amjad Ouled-Ameur @ 2022-09-20 15:16 UTC (permalink / raw)
To: Daniel Lezcano, Matthias Brugger, Rob Herring
Cc: Krzysztof Kozlowski, devicetree, linux-arm-kernel,
linux-mediatek, linux-kernel
Hi,
On 9/15/22 17:50, Daniel Lezcano wrote:
> On 14/09/2022 16:15, Matthias Brugger wrote:
>>
>>
>> On 14/09/2022 15:13, 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 but are not supposed to remain on DT.
>>>
>>> Remove the zones above and keep only cpu_thermal.
>>>
>>
>> My understanding is that this thermal zones exist but are not used.
>> DTS should describe the HW as it is, so I propose to add
>> status = "disabled";
>> to the zones.
>>
>
> At least the trip points must be declared as stated in the bindings
>
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/thermal/thermal-zones.yaml#n213
>
>
In the base case we should but there are no trip points for these
sensors according to MediaTek.
Regards,
Amjad
_______________________________________________
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-20 15:18 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-14 13:13 [PATCH] arm64: dts: mediatek: mt8183: remove thermal zones without trips Amjad Ouled-Ameur
2022-09-14 14:15 ` Matthias Brugger
2022-09-14 19:44 ` Amjad Ouled-Ameur
2022-09-15 15:50 ` Daniel Lezcano
2022-09-20 15:16 ` Amjad Ouled-Ameur
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).