linux-pm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] cpufreq: Select IRQ_WORK if CPU_FREQ_GOV_COMMON is set
@ 2016-02-28  1:33 Rafael J. Wysocki
  2016-02-29  3:18 ` Viresh Kumar
  0 siblings, 1 reply; 2+ messages in thread
From: Rafael J. Wysocki @ 2016-02-28  1:33 UTC (permalink / raw)
  To: Linux PM list
  Cc: Viresh Kumar, Srinivas Pandruvada, Linux Kernel Mailing List

From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

Commit 8fb47ff100af (cpufreq: governor: Replace timers with utilization
update callbacks) made CPU_FREQ select IRQ_WORK, but that's not
necessary, as it is sufficient for IRQ_WORK to be selected by
CPU_FREQ_GOV_COMMON, so modify the cpufreq Kconfig to that effect.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
---

On top of linux-next.

Thanks,
Rafael

---
 drivers/cpufreq/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: linux-pm/drivers/cpufreq/Kconfig
===================================================================
--- linux-pm.orig/drivers/cpufreq/Kconfig
+++ linux-pm/drivers/cpufreq/Kconfig
@@ -3,7 +3,6 @@ menu "CPU Frequency scaling"
 config CPU_FREQ
 	bool "CPU Frequency scaling"
 	select SRCU
-	select IRQ_WORK
 	help
 	  CPU Frequency scaling allows you to change the clock speed of 
 	  CPUs on the fly. This is a nice method to save power, because 
@@ -20,6 +19,7 @@ config CPU_FREQ
 if CPU_FREQ
 
 config CPU_FREQ_GOV_COMMON
+	select IRQ_WORK
 	bool
 
 config CPU_FREQ_BOOST_SW


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] cpufreq: Select IRQ_WORK if CPU_FREQ_GOV_COMMON is set
  2016-02-28  1:33 [PATCH] cpufreq: Select IRQ_WORK if CPU_FREQ_GOV_COMMON is set Rafael J. Wysocki
@ 2016-02-29  3:18 ` Viresh Kumar
  0 siblings, 0 replies; 2+ messages in thread
From: Viresh Kumar @ 2016-02-29  3:18 UTC (permalink / raw)
  To: Rafael J. Wysocki
  Cc: Linux PM list, Srinivas Pandruvada, Linux Kernel Mailing List

On 28-02-16, 02:33, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> 
> Commit 8fb47ff100af (cpufreq: governor: Replace timers with utilization
> update callbacks) made CPU_FREQ select IRQ_WORK, but that's not
> necessary, as it is sufficient for IRQ_WORK to be selected by
> CPU_FREQ_GOV_COMMON, so modify the cpufreq Kconfig to that effect.
> 
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> ---
> 
> On top of linux-next.
> 
> Thanks,
> Rafael
> 
> ---
>  drivers/cpufreq/Kconfig |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Index: linux-pm/drivers/cpufreq/Kconfig
> ===================================================================
> --- linux-pm.orig/drivers/cpufreq/Kconfig
> +++ linux-pm/drivers/cpufreq/Kconfig
> @@ -3,7 +3,6 @@ menu "CPU Frequency scaling"
>  config CPU_FREQ
>  	bool "CPU Frequency scaling"
>  	select SRCU
> -	select IRQ_WORK
>  	help
>  	  CPU Frequency scaling allows you to change the clock speed of 
>  	  CPUs on the fly. This is a nice method to save power, because 
> @@ -20,6 +19,7 @@ config CPU_FREQ
>  if CPU_FREQ
>  
>  config CPU_FREQ_GOV_COMMON
> +	select IRQ_WORK
>  	bool
>  
>  config CPU_FREQ_BOOST_SW

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

-- 
viresh

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-02-29  3:18 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-28  1:33 [PATCH] cpufreq: Select IRQ_WORK if CPU_FREQ_GOV_COMMON is set Rafael J. Wysocki
2016-02-29  3:18 ` Viresh Kumar

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).