mbox series

[v2,0/7] ACPI: thermal: Assorted cleanups

Message ID 2703629.mvXUDI8C0e@kreacher
Headers show
Series ACPI: thermal: Assorted cleanups | expand

Message

Rafael J. Wysocki June 4, 2023, 12:05 p.m. UTC 
Hi Folks,

This is the second iteration of

https://lore.kernel.org/linux-acpi/5675481.DvuYhMxLoT@kreacher

It contains two additional patches and fixes up two patches from the original
series.

Thanks!

Comments

Daniel Lezcano June 6, 2023, 4:41 p.m. UTC | #1 
On 04/06/2023 14:15, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> 
> Move all of the symbol definitions to the initial part of the code so
> they all can be found in one place.
> 
> While at it, consolidate white space used in there.
> 
> No functional impact.
> 
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> Reviewed-by: Michal Wilczynski <michal.wilczynski@intel.com>

Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Daniel Lezcano June 6, 2023, 4:46 p.m. UTC | #2 
On 04/06/2023 14:21, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> 
> Drop struct acpi_thermal_flags which is not really used (only one
> flag in it is ever set, but it is never read) and call
> acpi_execute_simple_method() directly to evaluate _SCP instead of
> using acpi_thermal_set_cooling_mode(), which has no callers after
> that change, so drop it.
> 
> No intentional functional impact.
> 
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>


Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>