On 12/16/2011 02:52 AM, Jamie Iles wrote:
+static DEFINE_PER_CPU(unsigned long, l_p_j_ref); +static unsigned long l_p_j_ref_freq;
+static struct clk *cpu_clk;
This assumes that all CPU's share the same clk and run at the same rate. Is that a fair/safe assumption? I honestly don't know the answer to this so it's just a question!!!
On MSM, cpus independently scale both frequency and voltage. Our clock driver isn't upstream yet. David Brown has a preliminary version here:
https://www.codeaurora.org/gitweb/quic/kernel/?p=davidb/linux-msm.git%3Ba=sh...
Once we get our driver upstream, MSM will be an exception and not select ARM_GENERIC_CPUFREQ. We'll probably have a separate msm-cpufreq.c driver under drivers/cpufreq/.
- Bryan