On Thu, May 19, 2011 at 08:12, Kevin Hilman <khilman@xxxxxx> wrote:
> Nishanth Menon <nm@xxxxxx> writes:
>
>> OMAP2 does not use OPP tables at the moment for DVFS. Currently,
>> we depend on opp table initialization to give us the freq_table,
>> which makes sense for OMAP3+. for OMAP2, we should be using
>> clk_init_cpufreq_table - so if the opp based frequency table
>> initilization fails, fall back to clk_init_cpufreq_table to give
>> us the table.
>>
>> Signed-off-by: Nishanth Menon <nm@xxxxxx>
>
> This is a good approach, but for readability, the OPP version and the
> clk version should probably be separated into separate functions, along
> with their error handling.
Was thinking more of the lines of splitting the file up. OMAP3+ all
have OPPs defined. only one pending is OMAP2
Either we introduce OPPs to OMAP2 OR we split it up and depend the OPP
based stuff on ARCH_HAS_OPP and CPUFREQ
>
> Minor: please capitalize acronyms: OPP, CPU, OMAP, etc...
will do.
Regards,
Nishanth Menon
>
> Kevin
>
>> ---
>> arch/arm/mach-omap2/omap2plus-cpufreq.c | 9 ++++++++-
>> 1 files changed, 8 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/omap2plus-cpufreq.c b/arch/arm/mach-omap2/omap2plus-cpufreq.c
>> index 45f1e9e..854f4b3 100644
>> --- a/arch/arm/mach-omap2/omap2plus-cpufreq.c
>> +++ b/arch/arm/mach-omap2/omap2plus-cpufreq.c
>> @@ -180,7 +180,13 @@ static int __cpuinit omap_cpu_init(struct cpufreq_policy *policy)
>> pr_warning("%s: unable to get the mpu device\n", __func__);
>> return -EINVAL;
>> }
>> - opp_init_cpufreq_table(mpu_dev, &freq_table);
>> +
>> + /*
>> + * if we dont get cpufreq table using opp, use traditional omap2 lookup
>> + * as a fallback
>> + */
>> + if (opp_init_cpufreq_table(mpu_dev, &freq_table))
>> + clk_init_cpufreq_table(&freq_table);
>>
>> if (freq_table) {
>> result = cpufreq_frequency_table_cpuinfo(policy, freq_table);
>> @@ -188,6 +194,7 @@ static int __cpuinit omap_cpu_init(struct cpufreq_policy *policy)
>> cpufreq_frequency_table_get_attr(freq_table,
>> policy->cpu);
>> } else {
>> + clk_exit_cpufreq_table(&freq_table);
>> WARN(true, "%s: fallback to clk_round(freq_table=%d)\n",
>> __func__, result);
>> kfree(freq_table);
>
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
