mbox series

[v2,0/4] Add upper and lower limits in IPA power budget calculation

Message ID 20201015112441.4056-1-lukasz.luba@arm.com (mailing list archive)
Headers show
Series Add upper and lower limits in IPA power budget calculation | expand

Message

Lukasz Luba Oct. 15, 2020, 11:24 a.m. UTC
Hi all,

This patch set makes thermal governor Intelligent Power Allocation (IPA)
aware of cooling device limits for upper and lower bounds and respects them
in the internal power budget calculation.

This v2 contains for completeness a patch posted by Michael. The patches
are re-based on top of current thermal/linux-next branch.

Changes:
v2:
- added check for cdev_is_power_actor(), before using cdev->ops->state2power()
  pointed out by Daniel
- added Michael's patch for consistency
- re-based on top of current thermal/linux-next
v1:
 can be found here [2]

Regards,
Lukasz

[1] https://lore.kernel.org/linux-pm/20201007024332.30322-1-michael.kao@mediatek.com/
[2] https://lore.kernel.org/linux-pm/20201007122256.28080-1-lukasz.luba@arm.com/

Lukasz Luba (3):
  thermal: power_allocator: respect upper and lower bounds for cooling
    device
  thermal: core: remove unused functions in power actor section
  thermal: move power_actor_set_power into IPA

Michael Kao (1):
  thermal: core: add upper and lower limits to power_actor_set_power

 drivers/thermal/gov_power_allocator.c | 40 +++++++++++-
 drivers/thermal/thermal_core.c        | 88 ---------------------------
 drivers/thermal/thermal_core.h        |  6 --
 3 files changed, 38 insertions(+), 96 deletions(-)

Comments

Daniel Lezcano Oct. 26, 2020, 8:42 p.m. UTC | #1
Applied, thanks

On 15/10/2020 13:24, Lukasz Luba wrote:
> Hi all,
> 
> This patch set makes thermal governor Intelligent Power Allocation (IPA)
> aware of cooling device limits for upper and lower bounds and respects them
> in the internal power budget calculation.
> 
> This v2 contains for completeness a patch posted by Michael. The patches
> are re-based on top of current thermal/linux-next branch.
> 
> Changes:
> v2:
> - added check for cdev_is_power_actor(), before using cdev->ops->state2power()
>   pointed out by Daniel
> - added Michael's patch for consistency
> - re-based on top of current thermal/linux-next
> v1:
>  can be found here [2]
> 
> Regards,
> Lukasz
> 
> [1] https://lore.kernel.org/linux-pm/20201007024332.30322-1-michael.kao@mediatek.com/
> [2] https://lore.kernel.org/linux-pm/20201007122256.28080-1-lukasz.luba@arm.com/
> 
> Lukasz Luba (3):
>   thermal: power_allocator: respect upper and lower bounds for cooling
>     device
>   thermal: core: remove unused functions in power actor section
>   thermal: move power_actor_set_power into IPA
> 
> Michael Kao (1):
>   thermal: core: add upper and lower limits to power_actor_set_power
> 
>  drivers/thermal/gov_power_allocator.c | 40 +++++++++++-
>  drivers/thermal/thermal_core.c        | 88 ---------------------------
>  drivers/thermal/thermal_core.h        |  6 --
>  3 files changed, 38 insertions(+), 96 deletions(-)
>
Lukasz Luba Nov. 2, 2020, 9 a.m. UTC | #2
On 10/26/20 8:42 PM, Daniel Lezcano wrote:
> 
> Applied, thanks
> 

Thank you Daniel!