[RFC,4/7] cpufreq: add per-governor private data

Message ID 1413958051-7103-5-git-send-email-mturquette@linaro.org
State New
Headers show

Commit Message

Mike Turquette Oct. 22, 2014, 6:07 a.m.
Cc: Viresh Kumar <viresh.kumar@linaro.org>
Cc: Rafael J. Wysocki <rjw@rjwysocki.net>
Signed-off-by: Mike Turquette <mturquette@linaro.org>
---
 include/linux/cpufreq.h | 3 +++
 1 file changed, 3 insertions(+)

Comments

Viresh Kumar Oct. 22, 2014, 6:26 a.m. | #1
On 22 October 2014 11:37, Mike Turquette <mturquette@linaro.org> wrote:
> Cc: Viresh Kumar <viresh.kumar@linaro.org>
> Cc: Rafael J. Wysocki <rjw@rjwysocki.net>
> Signed-off-by: Mike Turquette <mturquette@linaro.org>
> ---
>  include/linux/cpufreq.h | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h
> index 138336b..91d173c 100644
> --- a/include/linux/cpufreq.h
> +++ b/include/linux/cpufreq.h
> @@ -115,6 +115,9 @@ struct cpufreq_policy {
>
>         /* For cpufreq driver's internal use */
>         void                    *driver_data;
> +
> +       /* For cpufreq governor's internal use */
> +       void                    *gov_data;

Its already there: governor_data ..

Am I missing something ?
Mike Turquette Oct. 22, 2014, 6:35 a.m. | #2
On Tue, Oct 21, 2014 at 11:26 PM, Viresh Kumar <viresh.kumar@linaro.org> wrote:
> On 22 October 2014 11:37, Mike Turquette <mturquette@linaro.org> wrote:
>> Cc: Viresh Kumar <viresh.kumar@linaro.org>
>> Cc: Rafael J. Wysocki <rjw@rjwysocki.net>
>> Signed-off-by: Mike Turquette <mturquette@linaro.org>
>> ---
>>  include/linux/cpufreq.h | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h
>> index 138336b..91d173c 100644
>> --- a/include/linux/cpufreq.h
>> +++ b/include/linux/cpufreq.h
>> @@ -115,6 +115,9 @@ struct cpufreq_policy {
>>
>>         /* For cpufreq driver's internal use */
>>         void                    *driver_data;
>> +
>> +       /* For cpufreq governor's internal use */
>> +       void                    *gov_data;
>
> Its already there: governor_data ..
>
> Am I missing something ?

Oops. Thats what I get for hacking while jetlagged. Please disregard the noise.

Regards,
Mike

Patch

diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h
index 138336b..91d173c 100644
--- a/include/linux/cpufreq.h
+++ b/include/linux/cpufreq.h
@@ -115,6 +115,9 @@  struct cpufreq_policy {
 
 	/* For cpufreq driver's internal use */
 	void			*driver_data;
+
+	/* For cpufreq governor's internal use */
+	void			*gov_data;
 };
 
 /* Only for ACPI */