mbox series

[v4,0/2] Add thermal management support for STi platform

Message ID 20240716-thermal-v4-0-947b327e165c@gmail.com (mailing list archive)
Headers show
Series Add thermal management support for STi platform | expand

Message

Raphaël Gallais-Pou July 16, 2024, 5:34 p.m. UTC
This patch series enhances the st_thermal driver in order to enable
support for thermal zones. The changes include:

1. Replace deprecated PM runtime macros with their updated counterparts.
2. Implementing devm_* based thermal of zone functions within the driver.
3. Updating the stih418 device-tree.

The device-tree patch depends on an earlier patch sent to the mailing
list [1].

As it is currently implemented, an alert threshold of 85°C is set to
trigger the CPU throttling, and when the temperature exceeds the
critical threshold of 95°C, the system shuts down. There is for now no
active cooling device on the platform, which explains the use of the
cpufreq framework.

[1] https://lore.kernel.org/lkml/20240320-thermal-v3-2-700296694c4a@gmail.com

Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>
---
Changes in v4:
- [2/2] optimize dependencies
- [2/2] do not return devm_* exit code
- Link to v3: https://lore.kernel.org/r/20240714-thermal-v3-0-88f2489ef7d5@gmail.com

Changes in v3:
- Fix unmet dependency in [2/2]
- Remove no more used variable in [2/2]
- Remove already merged patch in soc tree
- Link to v2: https://lore.kernel.org/r/20240625-thermal-v2-0-bf8354ed51ee@gmail.com

Changes in v2:
- Add Patrice's R-b
- Edit patch [2/3] to remove unused struct
- Link to v1: https://lore.kernel.org/r/20240518-thermal-v1-0-7dfca3ed454b@gmail.com

---
Raphael Gallais-Pou (2):
      thermal: st: switch from CONFIG_PM_SLEEP guards to pm_sleep_ptr()
      thermal: sti: depend on THERMAL_OF subsystem

 drivers/thermal/Kconfig                |  2 +-
 drivers/thermal/st/st_thermal.c        | 32 ++++++++++++--------------------
 drivers/thermal/st/st_thermal_memmap.c |  2 +-
 drivers/thermal/st/stm_thermal.c       |  8 +++-----
 4 files changed, 17 insertions(+), 27 deletions(-)
---
base-commit: 4f40be61af99a67d5580c1448acd9b74c0376389
change-id: 20240518-thermal-8f625428acf9

Best regards,

Comments

Raphaël Gallais-Pou Aug. 19, 2024, 8:21 p.m. UTC | #1
Hello,

Le 16/07/2024 à 19:34, Raphael Gallais-Pou a écrit :
> This patch series enhances the st_thermal driver in order to enable
> support for thermal zones. The changes include:
> 
> 1. Replace deprecated PM runtime macros with their updated counterparts.
> 2. Implementing devm_* based thermal of zone functions within the driver.
> 3. Updating the stih418 device-tree.
> 
> The device-tree patch depends on an earlier patch sent to the mailing
> list [1].
> 
> As it is currently implemented, an alert threshold of 85°C is set to
> trigger the CPU throttling, and when the temperature exceeds the
> critical threshold of 95°C, the system shuts down. There is for now no
> active cooling device on the platform, which explains the use of the
> cpufreq framework.
> 
> [1] https://lore.kernel.org/lkml/20240320-thermal-v3-2-700296694c4a@gmail.com
> 
> Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>
> ---
> Changes in v4:
> - [2/2] optimize dependencies
> - [2/2] do not return devm_* exit code
> - Link to v3: https://lore.kernel.org/r/20240714-thermal-v3-0-88f2489ef7d5@gmail.com

Gentle ping on this serie :)

Thanks for your time,

Regards,
Raphaël

> 
> Changes in v3:
> - Fix unmet dependency in [2/2]
> - Remove no more used variable in [2/2]
> - Remove already merged patch in soc tree
> - Link to v2: https://lore.kernel.org/r/20240625-thermal-v2-0-bf8354ed51ee@gmail.com
> 
> Changes in v2:
> - Add Patrice's R-b
> - Edit patch [2/3] to remove unused struct
> - Link to v1: https://lore.kernel.org/r/20240518-thermal-v1-0-7dfca3ed454b@gmail.com
> 
> ---
> Raphael Gallais-Pou (2):
>        thermal: st: switch from CONFIG_PM_SLEEP guards to pm_sleep_ptr()
>        thermal: sti: depend on THERMAL_OF subsystem
> 
>   drivers/thermal/Kconfig                |  2 +-
>   drivers/thermal/st/st_thermal.c        | 32 ++++++++++++--------------------
>   drivers/thermal/st/st_thermal_memmap.c |  2 +-
>   drivers/thermal/st/stm_thermal.c       |  8 +++-----
>   4 files changed, 17 insertions(+), 27 deletions(-)
> ---
> base-commit: 4f40be61af99a67d5580c1448acd9b74c0376389
> change-id: 20240518-thermal-8f625428acf9
> 
> Best regards,
Daniel Lezcano Aug. 20, 2024, 8:05 a.m. UTC | #2
On 19/08/2024 22:21, Raphaël Gallais-Pou wrote:
> Hello,
> 
> Le 16/07/2024 à 19:34, Raphael Gallais-Pou a écrit :
>> This patch series enhances the st_thermal driver in order to enable
>> support for thermal zones. The changes include:
>>
>> 1. Replace deprecated PM runtime macros with their updated counterparts.
>> 2. Implementing devm_* based thermal of zone functions within the driver.
>> 3. Updating the stih418 device-tree.
>>
>> The device-tree patch depends on an earlier patch sent to the mailing
>> list [1].
>>
>> As it is currently implemented, an alert threshold of 85°C is set to
>> trigger the CPU throttling, and when the temperature exceeds the
>> critical threshold of 95°C, the system shuts down. There is for now no
>> active cooling device on the platform, which explains the use of the
>> cpufreq framework.
>>
>> [1] https://lore.kernel.org/lkml/20240320-thermal- 
>> v3-2-700296694c4a@gmail.com
>>
>> Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>
>> ---
>> Changes in v4:
>> - [2/2] optimize dependencies
>> - [2/2] do not return devm_* exit code
>> - Link to v3: https://lore.kernel.org/r/20240714-thermal- 
>> v3-0-88f2489ef7d5@gmail.com
> 
> Gentle ping on this serie :)
> 
> Thanks for your time,

Applied, thanks

Thanks for the patch 2/2 ;)