cpufreq: remove redundant 'policy' field from user_policy
authorViresh Kumar <viresh.kumar@linaro.org>
Mon, 3 Aug 2015 03:06:18 +0000 (08:36 +0530)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 1 Sep 2015 13:50:39 +0000 (15:50 +0200)
Its always same as policy->policy, and there is no need to keep another
copy of it. Remove it.

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

index 3033952391fe3c07a6e06771eef1fea7f573b506..a80bd68bbd747be4e2216ef5d3f9587edb497901 100644 (file)
@@ -671,11 +671,7 @@ static ssize_t store_scaling_governor(struct cpufreq_policy *policy,
                return -EINVAL;
 
        ret = cpufreq_set_policy(policy, &new_policy);
-       if (ret)
-               return ret;
-
-       policy->user_policy.policy = policy->policy;
-       return count;
+       return ret ? ret : count;
 }
 
 /**
@@ -1322,9 +1318,6 @@ static int cpufreq_online(unsigned int cpu)
                goto out_exit_policy;
        }
 
-       if (new_policy)
-               policy->user_policy.policy = policy->policy;
-
        up_write(&policy->rwsem);
 
        kobject_uevent(&policy->kobj, KOBJ_ADD);
@@ -2302,7 +2295,6 @@ int cpufreq_update_policy(unsigned int cpu)
        memcpy(&new_policy, policy, sizeof(*policy));
        new_policy.min = policy->user_policy.min;
        new_policy.max = policy->user_policy.max;
-       new_policy.policy = policy->user_policy.policy;
 
        /*
         * BIOS might change freq behind our back
index 752bf8a5c3146b30bd27a3db31fc6d939b517fd4..54dbbff0a55ee4cf137306c6688e68dd40c2cd64 100644 (file)
@@ -54,7 +54,6 @@ struct cpufreq_cpuinfo {
 struct cpufreq_real_policy {
        unsigned int            min;    /* in kHz */
        unsigned int            max;    /* in kHz */
-       unsigned int            policy; /* see above */
 };
 
 struct cpufreq_policy {