cpufreq: Rename cpufreq_can_do_remote_dvfs()
authorViresh Kumar <viresh.kumar@linaro.org>
Tue, 22 May 2018 10:01:30 +0000 (15:31 +0530)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 23 May 2018 08:37:08 +0000 (10:37 +0200)
This routine checks if the CPU running this code belongs to the policy
of the target CPU or if not, can it do remote DVFS for it remotely. But
the current name of it implies as if it is only about doing remote
updates.

Rename it to make it more relevant.

Suggested-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/cpufreq/cpufreq_governor.c
include/linux/cpufreq.h
kernel/sched/cpufreq_schedutil.c

index ca38229b045ab288a2f250dddaf1b174e8c0572f..871bf9cf55cf0dc95ee9952774c406280a7cbcee 100644 (file)
@@ -278,7 +278,7 @@ static void dbs_update_util_handler(struct update_util_data *data, u64 time,
        struct policy_dbs_info *policy_dbs = cdbs->policy_dbs;
        u64 delta_ns, lst;
 
-       if (!cpufreq_can_do_remote_dvfs(policy_dbs->policy))
+       if (!cpufreq_this_cpu_can_update(policy_dbs->policy))
                return;
 
        /*
index 87f48dd932eb1ed424c062c8b3e9bb8ab4365cc4..882a9b9e34bc2c0dc8ad00eaa58d23c784b6c2f5 100644 (file)
@@ -571,7 +571,7 @@ struct governor_attr {
                         size_t count);
 };
 
-static inline bool cpufreq_can_do_remote_dvfs(struct cpufreq_policy *policy)
+static inline bool cpufreq_this_cpu_can_update(struct cpufreq_policy *policy)
 {
        /*
         * Allow remote callbacks if:
index 416b7d7853d497adedede94c713392736611592b..caf435c14a52c2222209e4161701ffda644607b3 100644 (file)
@@ -89,7 +89,7 @@ static bool sugov_should_update_freq(struct sugov_policy *sg_policy, u64 time)
         * schedule the kthread.
         */
        if (sg_policy->policy->fast_switch_enabled &&
-           !cpufreq_can_do_remote_dvfs(sg_policy->policy))
+           !cpufreq_this_cpu_can_update(sg_policy->policy))
                return false;
 
        if (sg_policy->work_in_progress)