cpufreq: pcc-cpufreq: update default value of cpuinfo_transition_latency
authorJacob Tanenbaum <jtanenba@redhat.com>
Thu, 19 Nov 2015 15:29:01 +0000 (10:29 -0500)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 9 Dec 2015 23:17:03 +0000 (00:17 +0100)
The cpufreq documentation specifies

policy->cpuinfo.transition_latency   the time it takes on this CPU to
                                switch between two frequencies in
                                nanoseconds (if appropriate, else
                                specify CPUFREQ_ETERNAL)

currently pcc-cpufreq does not expose the value and sets it to zero. I
changed the pcc-cpufreq driver and it's documentation to conform to the
default value specified in Documentation/cpu-freq/cpu-drivers.txt

Signed-off-by: Jacob Tanenbaum <jtanenba@redhat.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Documentation/cpu-freq/pcc-cpufreq.txt
drivers/cpufreq/pcc-cpufreq.c

index 9e3c3b33514c688dc253861af1c11b650af361de..0a94224ad2965bff7c25e2137c3834ea3d132c72 100644 (file)
@@ -159,8 +159,8 @@ to be strictly associated with a P-state.
 
 2.2 cpuinfo_transition_latency:
 -------------------------------
-The cpuinfo_transition_latency field is 0. The PCC specification does
-not include a field to expose this value currently.
+The cpuinfo_transition_latency field is CPUFREQ_ETERNAL. The PCC specification
+does not include a field to expose this value currently.
 
 2.3 cpuinfo_cur_freq:
 ---------------------
index 2a0d58959acfe2861cdf9e826ef397958fd6fef2..808a320e9d5d0cdb0a943de7d9af5c89e9b444d5 100644 (file)
@@ -555,6 +555,8 @@ static int pcc_cpufreq_cpu_init(struct cpufreq_policy *policy)
        policy->min = policy->cpuinfo.min_freq =
                ioread32(&pcch_hdr->minimum_frequency) * 1000;
 
+       policy->cpuinfo.transition_latency = CPUFREQ_ETERNAL;
+
        pr_debug("init: policy->max is %d, policy->min is %d\n",
                policy->max, policy->min);
 out: