diff mbox series

[v1,2/7] drivers: cpufreq: Add thermal_cooling_device pointer to struct cpufreq_policy

Message ID aadc9cff04dc310138b41979239ce51938afedce.1547078153.git.amit.kucheria@linaro.org
State New
Headers show
Series [v1,1/7] drivers: thermal: of-thermal: Print name of device node with error | expand

Commit Message

Amit Kucheria Jan. 10, 2019, midnight UTC
Several cpufreq drivers register themselves as thermal cooling devices.
Adding a pointer to struct cpufreq_policy removes the need for them to
store this pointer in a private data structure.

Signed-off-by: Amit Kucheria <amit.kucheria@linaro.org>

---
 include/linux/cpufreq.h | 2 ++
 1 file changed, 2 insertions(+)

-- 
2.17.1

Comments

Matthias Kaehlcke Jan. 10, 2019, 1:01 a.m. UTC | #1
On Thu, Jan 10, 2019 at 05:30:51AM +0530, Amit Kucheria wrote:
> Several cpufreq drivers register themselves as thermal cooling devices.

> Adding a pointer to struct cpufreq_policy removes the need for them to

> store this pointer in a private data structure.

> 

> Signed-off-by: Amit Kucheria <amit.kucheria@linaro.org>

> ---

>  include/linux/cpufreq.h | 2 ++

>  1 file changed, 2 insertions(+)

> 

> diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h

> index c86d6d8bdfed..2496549d7573 100644

> --- a/include/linux/cpufreq.h

> +++ b/include/linux/cpufreq.h

> @@ -95,6 +95,8 @@ struct cpufreq_policy {

>  	struct cpufreq_frequency_table	*freq_table;

>  	enum cpufreq_table_sorting freq_table_sorted;

>  

> +	struct thermal_cooling_device *cooldev; /* Pointer to the cooling

> +						 * device if used for thermal mitigation */

>  	struct list_head        policy_list;

>  	struct kobject		kobj;

>  	struct completion	kobj_unregister;


I've mixed feelings about this. It's definitely desirable to avoid
code duplication and tying the cooling device to the cpufreq_policy is
a convenient way to achieve that. However semantically it seems a bit
odd that a CPU cooling device is part of the cpufreq policy.

Anyway, unless there are better ideas we probably want to be pragmatic
here, so if Viresh is fine with it who am I to complain ;-)

Cheers

Matthias
diff mbox series

Patch

diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h
index c86d6d8bdfed..2496549d7573 100644
--- a/include/linux/cpufreq.h
+++ b/include/linux/cpufreq.h
@@ -95,6 +95,8 @@  struct cpufreq_policy {
 	struct cpufreq_frequency_table	*freq_table;
 	enum cpufreq_table_sorting freq_table_sorted;
 
+	struct thermal_cooling_device *cooldev; /* Pointer to the cooling
+						 * device if used for thermal mitigation */
 	struct list_head        policy_list;
 	struct kobject		kobj;
 	struct completion	kobj_unregister;