* [PATCH] arm64: defconfig: Enable BANG BANG Thermal Governor
@ 2025-01-13 18:56 Garrett Giordano
2025-05-05 8:18 ` [Upstream] " Daniel Schultz
0 siblings, 1 reply; 4+ messages in thread
From: Garrett Giordano @ 2025-01-13 18:56 UTC (permalink / raw)
To: catalin.marinas, will, quic_bjorande, geert+renesas,
dmitry.baryshkov, krzysztof.kozlowski, shawnguo, neil.armstrong,
alexander.stein, biju.das.jz, javier.carrasco, elinor.montmasson,
linux-arm-kernel
Cc: linux-kernel, upstream
Enable the BANG BANG Thermal Governor to manage the GPIO Fan using
hysteresis on the PHYTEC phyBOARD-Lyra AM625.
Signed-off-by: Garrett Giordano <ggiordano@phytec.com>
---
arch/arm64/configs/defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index c62831e61586..3fd04c689269 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -690,6 +690,7 @@ CONFIG_SENSORS_RASPBERRYPI_HWMON=m
CONFIG_SENSORS_SL28CPLD=m
CONFIG_SENSORS_INA2XX=m
CONFIG_SENSORS_INA3221=m
+CONFIG_THERMAL_GOV_BANG_BANG=y
CONFIG_THERMAL_GOV_POWER_ALLOCATOR=y
CONFIG_CPU_THERMAL=y
CONFIG_DEVFREQ_THERMAL=y
--
2.25.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Upstream] [PATCH] arm64: defconfig: Enable BANG BANG Thermal Governor
2025-01-13 18:56 [PATCH] arm64: defconfig: Enable BANG BANG Thermal Governor Garrett Giordano
@ 2025-05-05 8:18 ` Daniel Schultz
2025-05-05 8:33 ` Krzysztof Kozlowski
0 siblings, 1 reply; 4+ messages in thread
From: Daniel Schultz @ 2025-05-05 8:18 UTC (permalink / raw)
To: Garrett Giordano, catalin.marinas, will, quic_bjorande,
geert+renesas, dmitry.baryshkov, krzysztof.kozlowski, shawnguo,
neil.armstrong, alexander.stein, biju.das.jz, javier.carrasco,
elinor.montmasson, linux-arm-kernel
Cc: linux-kernel, upstream
Hey,
can someone please take a look at this patch? I can also re-send in case
it doesn't apply anymore. Thanks!
- Daniel
On 1/13/25 19:56, Garrett Giordano wrote:
> Enable the BANG BANG Thermal Governor to manage the GPIO Fan using
> hysteresis on the PHYTEC phyBOARD-Lyra AM625.
>
> Signed-off-by: Garrett Giordano <ggiordano@phytec.com>
> ---
> arch/arm64/configs/defconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index c62831e61586..3fd04c689269 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -690,6 +690,7 @@ CONFIG_SENSORS_RASPBERRYPI_HWMON=m
> CONFIG_SENSORS_SL28CPLD=m
> CONFIG_SENSORS_INA2XX=m
> CONFIG_SENSORS_INA3221=m
> +CONFIG_THERMAL_GOV_BANG_BANG=y
> CONFIG_THERMAL_GOV_POWER_ALLOCATOR=y
> CONFIG_CPU_THERMAL=y
> CONFIG_DEVFREQ_THERMAL=y
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Upstream] [PATCH] arm64: defconfig: Enable BANG BANG Thermal Governor
2025-05-05 8:18 ` [Upstream] " Daniel Schultz
@ 2025-05-05 8:33 ` Krzysztof Kozlowski
2025-05-05 13:40 ` Daniel Schultz
0 siblings, 1 reply; 4+ messages in thread
From: Krzysztof Kozlowski @ 2025-05-05 8:33 UTC (permalink / raw)
To: Daniel Schultz, Garrett Giordano, catalin.marinas, will,
quic_bjorande, geert+renesas, dmitry.baryshkov, shawnguo,
neil.armstrong, alexander.stein, biju.das.jz, javier.carrasco,
elinor.montmasson, linux-arm-kernel
Cc: linux-kernel, upstream
On 05/05/2025 10:18, Daniel Schultz wrote:
> Hey,
>
> can someone please take a look at this patch? I can also re-send in case
> it doesn't apply anymore. Thanks!
Sounds like a product thingy, so I do not see much need to enable it in
the defconfig.
BTW, the more people you CC, the bigger chance no one will fell
responsible. You Cc-eed ~7 persons not related to your subsystem.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Upstream] [PATCH] arm64: defconfig: Enable BANG BANG Thermal Governor
2025-05-05 8:33 ` Krzysztof Kozlowski
@ 2025-05-05 13:40 ` Daniel Schultz
0 siblings, 0 replies; 4+ messages in thread
From: Daniel Schultz @ 2025-05-05 13:40 UTC (permalink / raw)
To: Krzysztof Kozlowski, Garrett Giordano, catalin.marinas, will,
quic_bjorande, geert+renesas, dmitry.baryshkov, shawnguo,
neil.armstrong, alexander.stein, biju.das.jz, javier.carrasco,
elinor.montmasson, linux-arm-kernel
Cc: linux-kernel, upstream
On 5/5/25 10:33, Krzysztof Kozlowski wrote:
> On 05/05/2025 10:18, Daniel Schultz wrote:
>> Hey,
>>
>> can someone please take a look at this patch? I can also re-send in case
>> it doesn't apply anymore. Thanks!
>
> Sounds like a product thingy, so I do not see much need to enable it in
> the defconfig.
I'm okay with that. We can handle that downstream but wasn't sure if
this config would be accepted.
>
> BTW, the more people you CC, the bigger chance no one will fell
> responsible. You Cc-eed ~7 persons not related to your subsystem.
Ah, yes that makes sense. Thank you for the hint!
- Daniel
>
>
> Best regards,
> Krzysztof
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-05-05 13:51 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-01-13 18:56 [PATCH] arm64: defconfig: Enable BANG BANG Thermal Governor Garrett Giordano
2025-05-05 8:18 ` [Upstream] " Daniel Schultz
2025-05-05 8:33 ` Krzysztof Kozlowski
2025-05-05 13:40 ` Daniel Schultz
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox