Re: [PATCH 5/5] cpufreq:boost:Kconfig: Enable boost support at Kconfig

2013-06-10 Thread Lukasz Majewski
Hi Viresh, > On 10 June 2013 18:50, Lukasz Majewski wrote: > > Hi, > > > >> On Thu, Jun 06, 2013 at 09:07:52AM +0200, Lukasz Majewski wrote: > >> > Enable/disable support for BOOST. New flag - CPU_FREQ_BOOST has > >> > been defined for that. > >> > >> What's the reason for this? Why would you wan

Re: [PATCH 5/5] cpufreq:boost:Kconfig: Enable boost support at Kconfig

2013-06-10 Thread Viresh Kumar
On 10 June 2013 18:50, Lukasz Majewski wrote: > Hi, > >> On Thu, Jun 06, 2013 at 09:07:52AM +0200, Lukasz Majewski wrote: >> > Enable/disable support for BOOST. New flag - CPU_FREQ_BOOST has been >> > defined for that. >> >> What's the reason for this? Why would you want to not compile-in the >> b

Re: [PATCH 5/5] cpufreq:boost:Kconfig: Enable boost support at Kconfig

2013-06-10 Thread Lukasz Majewski
Hi, > On Thu, Jun 06, 2013 at 09:07:52AM +0200, Lukasz Majewski wrote: > > Enable/disable support for BOOST. New flag - CPU_FREQ_BOOST has been > > defined for that. > > What's the reason for this? Why would you want to not compile-in the > boosting code? I think that this flag could be removed

Re: [PATCH 5/5] cpufreq:boost:Kconfig: Enable boost support at Kconfig

2013-06-06 Thread Borislav Petkov
On Thu, Jun 06, 2013 at 09:07:52AM +0200, Lukasz Majewski wrote: > Enable/disable support for BOOST. New flag - CPU_FREQ_BOOST has been > defined for that. What's the reason for this? Why would you want to not compile-in the boosting code? Also, at least on x86, even with this feature disabled, t

Re: [PATCH 5/5] cpufreq:boost:Kconfig: Enable boost support at Kconfig

2013-06-06 Thread Viresh Kumar
On 6 June 2013 20:51, Dave Jones wrote: > Given CPUFREQ is available on more platforms than X86/ARM, this option > could also use a depends. It should be disabled by default, so please add default n. But we don't need a depends here as everybody is open to use it. Its a core feature. -- To unsubs

Re: [PATCH 5/5] cpufreq:boost:Kconfig: Enable boost support at Kconfig

2013-06-06 Thread Dave Jones
On Thu, Jun 06, 2013 at 05:14:31PM +0200, Lukasz Majewski wrote: > Hi Dave, > > > On Thu, Jun 06, 2013 at 09:07:52AM +0200, Lukasz Majewski wrote: > > > > > +config CPU_FREQ_BOOST > > > + bool "CPU frequency boost support" > > > + help > > > + Switch to enable supp

Re: [PATCH 5/5] cpufreq:boost:Kconfig: Enable boost support at Kconfig

2013-06-06 Thread Lukasz Majewski
Hi Dave, > On Thu, Jun 06, 2013 at 09:07:52AM +0200, Lukasz Majewski wrote: > > > +config CPU_FREQ_BOOST > > + bool "CPU frequency boost support" > > + help > > +Switch to enable support for frequency boost > > + > > +If in doubt, say N. > > + > > This help text is devoid of any

Re: [PATCH 5/5] cpufreq:boost:Kconfig: Enable boost support at Kconfig

2013-06-06 Thread Dave Jones
On Thu, Jun 06, 2013 at 09:07:52AM +0200, Lukasz Majewski wrote: > +config CPU_FREQ_BOOST > +bool "CPU frequency boost support" > +help > + Switch to enable support for frequency boost > + > + If in doubt, say N. > + This help text is devoid of any useful information. On

[PATCH 5/5] cpufreq:boost:Kconfig: Enable boost support at Kconfig

2013-06-06 Thread Lukasz Majewski
Enable/disable support for BOOST. New flag - CPU_FREQ_BOOST has been defined for that. Signed-off-by: Lukasz Majewski Signed-off-by: Myungjoo Ham --- drivers/cpufreq/Kconfig |7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/cpufreq/Kconfig b/drivers/cpufreq/Kconfig index a14