Skip to content

Commit

Permalink
CPUFREQ: Loongson2: drop set_cpus_allowed_ptr()
Browse files Browse the repository at this point in the history
It is pure mystery to me why we need to be on a specific CPU while
looking up a value in an array.
My best shot at this is that before commit d4019f0 ("cpufreq: move
freq change notifications to cpufreq core") it was required to invoke
cpufreq_notify_transition() on a special CPU.

Since it looks like a waste, remove it.

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Cc: Rafael J. Wysocki <rjw@rjwysocki.net>
Cc: tglx@linutronix.de
Cc: linux-pm@vger.kernel.org
Cc: linux-mips@linux-mips.org
Patchwork: https://patchwork.linux-mips.org/patch/15888/
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
  • Loading branch information
Sebastian Andrzej Siewior authored and Ralf Baechle committed Apr 12, 2017
1 parent 33679a5 commit 759f534
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions drivers/cpufreq/loongson2_cpufreq.c
Original file line number Diff line number Diff line change
Expand Up @@ -51,19 +51,12 @@ static int loongson2_cpu_freq_notifier(struct notifier_block *nb,
static int loongson2_cpufreq_target(struct cpufreq_policy *policy,
unsigned int index)
{
unsigned int cpu = policy->cpu;
cpumask_t cpus_allowed;
unsigned int freq;

cpus_allowed = current->cpus_allowed;
set_cpus_allowed_ptr(current, cpumask_of(cpu));

freq =
((cpu_clock_freq / 1000) *
loongson2_clockmod_table[index].driver_data) / 8;

set_cpus_allowed_ptr(current, &cpus_allowed);

/* setting the cpu frequency */
clk_set_rate(policy->clk, freq * 1000);

Expand Down

0 comments on commit 759f534

Please sign in to comment.