Re: [PATCH 0/4] cpufreq/opp: Turbo/boost mode support

[Viresh Kumar <viresh.kumar@linaro.org>]

On 28-07-15, 15:38, Bartlomiej Zolnierkiewicz wrote:
> I did look a them and they are lacking enabling of boost support flag
> in cpufreq_dt driver structure.

That's insulting now, I forgot to add this:

diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
index 217928de2f0e..db4390ca5243 100644
--- a/drivers/cpufreq/cpufreq.c
+++ b/drivers/cpufreq/cpufreq.c
@@ -2495,6 +2495,8 @@ int cpufreq_enable_boost_support(void)
 	if (cpufreq_boost_supported())
 		return 0;
 
+	cpufreq_driver->boost_supported = true;
+
 	/* This will get removed on driver unregister */
 	return create_boost_sysfs_file();
 }

So that is done from the above routine now.

> There is also CONFIG_CPU_FREQ_BOOST_SW which needs to be enabled to
> have boost frequencies in sysfs (code for that is in freq_table.c).

Yeah, we should get rid of that as well..

> > those config options required. Get rid of them :)
> 
> Do you mean that we should now depend only on dts info for enabling
> boost support and that CONFIG_CPU_FREQ_BOOST_SW should be removed
> (together with CONFIG_ARM_EXYNOS_CPU_FREQ_BOOST_SW)?

Yes.

-- 
viresh