Re: [PATCH v2 6/10] cpufreq: governor: Rework cpufreq_governor_dbs()

From: Viresh Kumar
Date: Fri Feb 05 2016 - 03:15:12 EST


On 05-02-16, 03:18, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
>
> Since it is possible to obtain a pointer to struct dbs_governor
> from a pointer to the struct governor embedded in it via
> container_of(), the second argument of cpufreq_governor_init()
> is not necessary. Accordingly, cpufreq_governor_dbs() doesn't
> need its second argument either and the ->governor callbacks
> for both the ondemand and conservative governors may be set
> to cpufreq_governor_dbs() directly. Make that happen.
>
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
> Acked-by: Saravana Kannan <skannan@xxxxxxxxxxxxxx>
> ---
> drivers/cpufreq/cpufreq_conservative.c | 11 +----------
> drivers/cpufreq/cpufreq_governor.c | 10 +++++-----
> drivers/cpufreq/cpufreq_governor.h | 3 +--
> drivers/cpufreq/cpufreq_ondemand.c | 11 +----------
> 4 files changed, 8 insertions(+), 27 deletions(-)

Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>

--
viresh