On 10-02-16, 01:36, Rafael J. Wysocki wrote:
static int cpufreq_governor_stop(struct cpufreq_policy *policy) {
struct policy_dbs_info *policy_dbs = policy->governor_data;
/* State should be equivalent to START */
if (!policy_dbs->policy)
return -EBUSY;
gov_cancel_work(policy_dbs);
policy_dbs->policy = NULL;
gov_cancel_work(policy); return 0;
}
So maybe we can call gov_cancel_work(policy) from cpufreq_governor_dbs() directly and get rid of this wrapper too?
I thought about it, but left it for consistency. It wouldn't hurt, the compiler will anyway make it inline I believe.