Commit 78e30d12 authored by Ulf Hansson's avatar Ulf Hansson Committed by Mike Turquette

cpufreq: db8500: Use armss clk to update frequency

Using the armss clk to update the frequency makes the driver no more
directly dependant on the prmcu API.
Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Cc: Rafael J. Wysocki <rjw@sisk.pl>
Acked-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Signed-off-by: default avatarMike Turquette <mturquette@linaro.org>
parent d6e99fa4
...@@ -14,10 +14,11 @@ ...@@ -14,10 +14,11 @@
#include <linux/delay.h> #include <linux/delay.h>
#include <linux/slab.h> #include <linux/slab.h>
#include <linux/platform_device.h> #include <linux/platform_device.h>
#include <linux/mfd/dbx500-prcmu.h> #include <linux/clk.h>
#include <mach/id.h> #include <mach/id.h>
static struct cpufreq_frequency_table *freq_table; static struct cpufreq_frequency_table *freq_table;
static struct clk *armss_clk;
static struct freq_attr *db8500_cpufreq_attr[] = { static struct freq_attr *db8500_cpufreq_attr[] = {
&cpufreq_freq_attr_scaling_available_freqs, &cpufreq_freq_attr_scaling_available_freqs,
...@@ -58,9 +59,9 @@ static int db8500_cpufreq_target(struct cpufreq_policy *policy, ...@@ -58,9 +59,9 @@ static int db8500_cpufreq_target(struct cpufreq_policy *policy,
for_each_cpu(freqs.cpu, policy->cpus) for_each_cpu(freqs.cpu, policy->cpus)
cpufreq_notify_transition(&freqs, CPUFREQ_PRECHANGE); cpufreq_notify_transition(&freqs, CPUFREQ_PRECHANGE);
/* request the PRCM unit for opp change */ /* update armss clk frequency */
if (prcmu_set_arm_opp(freq_table[idx].index)) { if (clk_set_rate(armss_clk, freq_table[idx].frequency * 1000)) {
pr_err("db8500-cpufreq: Failed to set OPP level\n"); pr_err("db8500-cpufreq: Failed to update armss clk\n");
return -EINVAL; return -EINVAL;
} }
...@@ -74,16 +75,16 @@ static int db8500_cpufreq_target(struct cpufreq_policy *policy, ...@@ -74,16 +75,16 @@ static int db8500_cpufreq_target(struct cpufreq_policy *policy,
static unsigned int db8500_cpufreq_getspeed(unsigned int cpu) static unsigned int db8500_cpufreq_getspeed(unsigned int cpu)
{ {
int i = 0; int i = 0;
/* request the prcm to get the current ARM opp */ unsigned long freq = clk_get_rate(armss_clk) / 1000;
int opp = prcmu_get_arm_opp();
while (freq_table[i].frequency != CPUFREQ_TABLE_END) { while (freq_table[i].frequency != CPUFREQ_TABLE_END) {
if (opp == freq_table[i].index) if (freq <= freq_table[i].frequency)
return freq_table[i].frequency; return freq_table[i].frequency;
i++; i++;
} }
/* We could not find a corresponding opp frequency. */ /* We could not find a corresponding frequency. */
pr_err("db8500-cpufreq: Failed to find cpufreq speed\n");
return 0; return 0;
} }
...@@ -92,6 +93,12 @@ static int __cpuinit db8500_cpufreq_init(struct cpufreq_policy *policy) ...@@ -92,6 +93,12 @@ static int __cpuinit db8500_cpufreq_init(struct cpufreq_policy *policy)
int i = 0; int i = 0;
int res; int res;
armss_clk = clk_get(NULL, "armss");
if (IS_ERR(armss_clk)) {
pr_err("db8500-cpufreq : Failed to get armss clk\n");
return PTR_ERR(armss_clk);
}
pr_info("db8500-cpufreq : Available frequencies:\n"); pr_info("db8500-cpufreq : Available frequencies:\n");
while (freq_table[i].frequency != CPUFREQ_TABLE_END) { while (freq_table[i].frequency != CPUFREQ_TABLE_END) {
pr_info(" %d Mhz\n", freq_table[i].frequency/1000); pr_info(" %d Mhz\n", freq_table[i].frequency/1000);
...@@ -104,6 +111,7 @@ static int __cpuinit db8500_cpufreq_init(struct cpufreq_policy *policy) ...@@ -104,6 +111,7 @@ static int __cpuinit db8500_cpufreq_init(struct cpufreq_policy *policy)
cpufreq_frequency_table_get_attr(freq_table, policy->cpu); cpufreq_frequency_table_get_attr(freq_table, policy->cpu);
else { else {
pr_err("db8500-cpufreq : Failed to read policy table\n"); pr_err("db8500-cpufreq : Failed to read policy table\n");
clk_put(armss_clk);
return res; return res;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment