mbox series

[v2,0/4] Thermal cpufreq & devfreq cooling minor clean-ups

Message ID 20220613124327.30766-1-lukasz.luba@arm.com
Headers show
Series Thermal cpufreq & devfreq cooling minor clean-ups | expand

Message

Lukasz Luba June 13, 2022, 12:43 p.m. UTC
Hi all,

This is v2 of some minor clean-ups for the thermal cpufreq and devfreq
cooling code.

Changes:
v2:
- extened the cpufreq_cooling_device with private ops field, to not waste
  memory and simplify allocation/free code (Viresh)
- added devfreq_cooling clean-up to align with cpufreq cooling code
- added ACKs from Viresh for patch 2/4 and path 3/4
- added missing maintainers of tracing to CC list

Regards,
Lukasz

Lukasz Luba (4):
  thermal: cpufreq_cooling: Use private callback ops for each cooling
    device
  thermal: cpufreq_cooling : Refactor thermal_power_cpu_get_power
    tracing
  thermal: cpufreq_cooling: Update outdated comments
  thermal: devfreq_cooling: Extend the devfreq_cooling_device with ops

 drivers/thermal/cpufreq_cooling.c | 77 ++++++++++---------------------
 drivers/thermal/devfreq_cooling.c | 27 ++++-------
 include/trace/events/thermal.h    | 28 ++++-------
 3 files changed, 42 insertions(+), 90 deletions(-)

Comments

Lukasz Luba June 15, 2022, 7:47 a.m. UTC | #1
On 6/14/22 19:26, Daniel Lezcano wrote:
> On 14/06/2022 14:51, Lukasz Luba wrote:
>> Hi Daniel,
>>
>>
>> On 6/13/22 13:43, Lukasz Luba wrote:
>>> Hi all,
>>>
>>> This is v2 of some minor clean-ups for the thermal cpufreq and devfreq
>>> cooling code.
>>>
>>> Changes:
>>> v2:
>>> - extened the cpufreq_cooling_device with private ops field, to not 
>>> waste
>>>    memory and simplify allocation/free code (Viresh)
>>> - added devfreq_cooling clean-up to align with cpufreq cooling code
>>> - added ACKs from Viresh for patch 2/4 and path 3/4
>>> - added missing maintainers of tracing to CC list
>>>
>>> Regards,
>>> Lukasz
>>>
>>> Lukasz Luba (4):
>>>    thermal: cpufreq_cooling: Use private callback ops for each cooling
>>>      device
>>>    thermal: cpufreq_cooling : Refactor thermal_power_cpu_get_power
>>>      tracing
>>>    thermal: cpufreq_cooling: Update outdated comments
>>>    thermal: devfreq_cooling: Extend the devfreq_cooling_device with ops
>>>
>>>   drivers/thermal/cpufreq_cooling.c | 77 ++++++++++---------------------
>>>   drivers/thermal/devfreq_cooling.c | 27 ++++-------
>>>   include/trace/events/thermal.h    | 28 ++++-------
>>>   3 files changed, 42 insertions(+), 90 deletions(-)
>>>
>>
>> Could you have a look and take the patches into your tree, please?
>> The 3 of 4 patches got ACKs, the last one is devfreq cooling, which
>> is pretty minor change.
> 
> Applied, thanks
> 
> 

Thanks Daniel!