public inbox for linux-pm@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 0/3] cpufreq: ondemand: Change the calculation of target frequency
@ 2013-06-05 16:01 Stratos Karafotis
  2013-06-06  6:54 ` Viresh Kumar
  0 siblings, 1 reply; 2+ messages in thread
From: Stratos Karafotis @ 2013-06-05 16:01 UTC (permalink / raw)
  To: Rafael J. Wysocki, Viresh Kumar
  Cc: Thomas Gleixner, Ingo Molnar, H. Peter Anvin, Borislav Petkov,
	linux-pm, cpufreq, linux-kernel

Changes since v2:
	- Reorder patches 2/3 and 3/3
	- Fix typos in patch changelog

Changes since v1:
	- Use policy->cpuinfo.max_freq in the calculation formula
		of target frequency instead of policy->max
	- Split the patch into 3 parts

Stratos Karafotis (3):
  cpufreq: ondemand: Change the calculation of target frequency
  cpufreq: Remove unused APERF/MPERF support
  cpufreq: Remove unused function __cpufreq_driver_getavg

 arch/x86/include/asm/processor.h   | 29 ----------------------
 drivers/cpufreq/Makefile           |  2 +-
 drivers/cpufreq/acpi-cpufreq.c     |  5 ----
 drivers/cpufreq/cpufreq.c          | 12 ---------
 drivers/cpufreq/cpufreq_governor.c | 10 +-------
 drivers/cpufreq/cpufreq_governor.h |  1 -
 drivers/cpufreq/cpufreq_ondemand.c | 39 ++++++-----------------------
 drivers/cpufreq/mperf.c            | 51 --------------------------------------
 drivers/cpufreq/mperf.h            |  9 -------
 include/linux/cpufreq.h            |  6 -----
 10 files changed, 9 insertions(+), 155 deletions(-)
 delete mode 100644 drivers/cpufreq/mperf.c
 delete mode 100644 drivers/cpufreq/mperf.h

-- 
1.8.1.4


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

* Re: [PATCH v3 0/3] cpufreq: ondemand: Change the calculation of target frequency
  2013-06-05 16:01 [PATCH v3 0/3] cpufreq: ondemand: Change the calculation of target frequency Stratos Karafotis
@ 2013-06-06  6:54 ` Viresh Kumar
  0 siblings, 0 replies; 2+ messages in thread
From: Viresh Kumar @ 2013-06-06  6:54 UTC (permalink / raw)
  To: Stratos Karafotis
  Cc: Rafael J. Wysocki, Thomas Gleixner, Ingo Molnar, H. Peter Anvin,
	Borislav Petkov, linux-pm, cpufreq, linux-kernel

On 5 June 2013 21:31, Stratos Karafotis <stratosk@semaphore.gr> wrote:
> Changes since v2:
>         - Reorder patches 2/3 and 3/3
>         - Fix typos in patch changelog
>
> Changes since v1:
>         - Use policy->cpuinfo.max_freq in the calculation formula
>                 of target frequency instead of policy->max
>         - Split the patch into 3 parts
>
> Stratos Karafotis (3):
>   cpufreq: ondemand: Change the calculation of target frequency
>   cpufreq: Remove unused APERF/MPERF support
>   cpufreq: Remove unused function __cpufreq_driver_getavg
>
>  arch/x86/include/asm/processor.h   | 29 ----------------------
>  drivers/cpufreq/Makefile           |  2 +-
>  drivers/cpufreq/acpi-cpufreq.c     |  5 ----
>  drivers/cpufreq/cpufreq.c          | 12 ---------
>  drivers/cpufreq/cpufreq_governor.c | 10 +-------
>  drivers/cpufreq/cpufreq_governor.h |  1 -
>  drivers/cpufreq/cpufreq_ondemand.c | 39 ++++++-----------------------
>  drivers/cpufreq/mperf.c            | 51 --------------------------------------
>  drivers/cpufreq/mperf.h            |  9 -------
>  include/linux/cpufreq.h            |  6 -----
>  10 files changed, 9 insertions(+), 155 deletions(-)
>  delete mode 100644 drivers/cpufreq/mperf.c
>  delete mode 100644 drivers/cpufreq/mperf.h

Functionally your patchset looks fine to me.. I know there are few
concerns about the real impact on power/performance, and I am
not talking about them here (Though they are very important :) ).

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

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

end of thread, other threads:[~2013-06-06  6:54 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-05 16:01 [PATCH v3 0/3] cpufreq: ondemand: Change the calculation of target frequency Stratos Karafotis
2013-06-06  6:54 ` Viresh Kumar

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox