Skip to content

Commit

Permalink
cpufreq: pcc-cpufreq: update default value of cpuinfo_transition_latency
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Jacob Tanenbaum authored and Rafael J. Wysocki committed Dec 9, 2015
1 parent 2f7e8a1 commit 790d849
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Documentation/cpu-freq/pcc-cpufreq.txt
Original file line number Diff line number Diff line change
Expand Up @@ -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:
---------------------
Expand Down
2 changes: 2 additions & 0 deletions drivers/cpufreq/pcc-cpufreq.c
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand Down

0 comments on commit 790d849

Please sign in to comment.