* [PATCH 1/2] cpufreq: Make ARM big.LITTLE switcher depend on ARM
@ 2013-12-11 22:12 Mark Brown
2013-12-11 22:12 ` [PATCH 2/2] cpufreq: Select PM_OPP rather than depending on it Mark Brown
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Mark Brown @ 2013-12-11 22:12 UTC (permalink / raw)
To: Rob Herring, Viresh Kumar, Rafael J. Wysocki
Cc: cpufreq, linux-pm, linaro-kernel, Mark Brown
From: Mark Brown <broonie@linaro.org>
The patch currently under review to enable ARM cpufreq drivers for ARM64
which is useful due to the large amount of shared IP between ARM and ARM64
SoCs. However the big.LITTLE switcher relies on an architecture interface
to build which is not present on ARM64. Add a dependency until that is
resolved.
Signed-off-by: Mark Brown <broonie@linaro.org>
---
drivers/cpufreq/Kconfig.arm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/cpufreq/Kconfig.arm b/drivers/cpufreq/Kconfig.arm
index ce52ed949249..0a141109fd29 100644
--- a/drivers/cpufreq/Kconfig.arm
+++ b/drivers/cpufreq/Kconfig.arm
@@ -4,7 +4,7 @@
config ARM_BIG_LITTLE_CPUFREQ
tristate "Generic ARM big LITTLE CPUfreq driver"
- depends on ARM_CPU_TOPOLOGY && PM_OPP && HAVE_CLK
+ depends on ARM && ARM_CPU_TOPOLOGY && PM_OPP && HAVE_CLK
help
This enables the Generic CPUfreq driver for ARM big.LITTLE platforms.
--
1.8.5.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/2] cpufreq: Select PM_OPP rather than depending on it
2013-12-11 22:12 [PATCH 1/2] cpufreq: Make ARM big.LITTLE switcher depend on ARM Mark Brown
@ 2013-12-11 22:12 ` Mark Brown
2013-12-16 9:20 ` Viresh Kumar
2013-12-16 9:20 ` [PATCH 1/2] cpufreq: Make ARM big.LITTLE switcher depend on ARM Viresh Kumar
2013-12-22 1:06 ` Rafael J. Wysocki
2 siblings, 1 reply; 5+ messages in thread
From: Mark Brown @ 2013-12-11 22:12 UTC (permalink / raw)
To: Rob Herring, Viresh Kumar, Rafael J. Wysocki
Cc: cpufreq, linux-pm, linaro-kernel, Mark Brown
From: Mark Brown <broonie@linaro.org>
PM_OPP is a helper library used by several of the existing cpufreq drivers.
Some of the drivers select this symbol while others depend on it and rely
on the architecture to enable it. Make this behaviour more consistent and
obvious by having all the drivers select the symbol. This will also allow
better build coverage of the affected drivers.
Signed-off-by: Mark Brown <broonie@linaro.org>
---
drivers/cpufreq/Kconfig | 3 ++-
drivers/cpufreq/Kconfig.arm | 6 ++++--
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/cpufreq/Kconfig b/drivers/cpufreq/Kconfig
index e26fdcd14165..6947fdfca57c 100644
--- a/drivers/cpufreq/Kconfig
+++ b/drivers/cpufreq/Kconfig
@@ -181,7 +181,8 @@ config CPU_FREQ_GOV_CONSERVATIVE
config GENERIC_CPUFREQ_CPU0
tristate "Generic CPU0 cpufreq driver"
- depends on HAVE_CLK && REGULATOR && PM_OPP && OF
+ depends on HAVE_CLK && REGULATOR && OF
+ select PM_OPP
help
This adds a generic cpufreq driver for CPU0 frequency management.
It supports both uniprocessor (UP) and symmetric multiprocessor (SMP)
diff --git a/drivers/cpufreq/Kconfig.arm b/drivers/cpufreq/Kconfig.arm
index 0a141109fd29..456ba5e1781c 100644
--- a/drivers/cpufreq/Kconfig.arm
+++ b/drivers/cpufreq/Kconfig.arm
@@ -4,7 +4,8 @@
config ARM_BIG_LITTLE_CPUFREQ
tristate "Generic ARM big LITTLE CPUfreq driver"
- depends on ARM && ARM_CPU_TOPOLOGY && PM_OPP && HAVE_CLK
+ depends on ARM && ARM_CPU_TOPOLOGY && HAVE_CLK
+ select PM_OPP
help
This enables the Generic CPUfreq driver for ARM big.LITTLE platforms.
@@ -54,7 +55,8 @@ config ARM_EXYNOS5250_CPUFREQ
config ARM_EXYNOS5440_CPUFREQ
bool "SAMSUNG EXYNOS5440"
depends on SOC_EXYNOS5440
- depends on HAVE_CLK && PM_OPP && OF
+ depends on HAVE_CLK && OF
+ select PM_OPP
default y
help
This adds the CPUFreq driver for Samsung EXYNOS5440
--
1.8.5.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] cpufreq: Select PM_OPP rather than depending on it
2013-12-11 22:12 ` [PATCH 2/2] cpufreq: Select PM_OPP rather than depending on it Mark Brown
@ 2013-12-16 9:20 ` Viresh Kumar
0 siblings, 0 replies; 5+ messages in thread
From: Viresh Kumar @ 2013-12-16 9:20 UTC (permalink / raw)
To: Mark Brown
Cc: Rob Herring, Rafael J. Wysocki, cpufreq@vger.kernel.org,
linux-pm@vger.kernel.org, Lists linaro-kernel, Mark Brown
On 12 December 2013 03:42, Mark Brown <broonie@kernel.org> wrote:
> From: Mark Brown <broonie@linaro.org>
>
> PM_OPP is a helper library used by several of the existing cpufreq drivers.
> Some of the drivers select this symbol while others depend on it and rely
> on the architecture to enable it. Make this behaviour more consistent and
> obvious by having all the drivers select the symbol. This will also allow
> better build coverage of the affected drivers.
>
> Signed-off-by: Mark Brown <broonie@linaro.org>
> ---
> drivers/cpufreq/Kconfig | 3 ++-
> drivers/cpufreq/Kconfig.arm | 6 ++++--
> 2 files changed, 6 insertions(+), 3 deletions(-)
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] cpufreq: Make ARM big.LITTLE switcher depend on ARM
2013-12-11 22:12 [PATCH 1/2] cpufreq: Make ARM big.LITTLE switcher depend on ARM Mark Brown
2013-12-11 22:12 ` [PATCH 2/2] cpufreq: Select PM_OPP rather than depending on it Mark Brown
@ 2013-12-16 9:20 ` Viresh Kumar
2013-12-22 1:06 ` Rafael J. Wysocki
2 siblings, 0 replies; 5+ messages in thread
From: Viresh Kumar @ 2013-12-16 9:20 UTC (permalink / raw)
To: Mark Brown
Cc: Rob Herring, Rafael J. Wysocki, cpufreq@vger.kernel.org,
linux-pm@vger.kernel.org, Lists linaro-kernel, Mark Brown
On 12 December 2013 03:42, Mark Brown <broonie@kernel.org> wrote:
> From: Mark Brown <broonie@linaro.org>
>
> The patch currently under review to enable ARM cpufreq drivers for ARM64
> which is useful due to the large amount of shared IP between ARM and ARM64
> SoCs. However the big.LITTLE switcher relies on an architecture interface
> to build which is not present on ARM64. Add a dependency until that is
> resolved.
>
> Signed-off-by: Mark Brown <broonie@linaro.org>
> ---
> drivers/cpufreq/Kconfig.arm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] cpufreq: Make ARM big.LITTLE switcher depend on ARM
2013-12-11 22:12 [PATCH 1/2] cpufreq: Make ARM big.LITTLE switcher depend on ARM Mark Brown
2013-12-11 22:12 ` [PATCH 2/2] cpufreq: Select PM_OPP rather than depending on it Mark Brown
2013-12-16 9:20 ` [PATCH 1/2] cpufreq: Make ARM big.LITTLE switcher depend on ARM Viresh Kumar
@ 2013-12-22 1:06 ` Rafael J. Wysocki
2 siblings, 0 replies; 5+ messages in thread
From: Rafael J. Wysocki @ 2013-12-22 1:06 UTC (permalink / raw)
To: Mark Brown
Cc: Rob Herring, Viresh Kumar, cpufreq, linux-pm, linaro-kernel,
Mark Brown
On Wednesday, December 11, 2013 10:12:26 PM Mark Brown wrote:
> From: Mark Brown <broonie@linaro.org>
>
> The patch currently under review to enable ARM cpufreq drivers for ARM64
> which is useful due to the large amount of shared IP between ARM and ARM64
> SoCs. However the big.LITTLE switcher relies on an architecture interface
> to build which is not present on ARM64. Add a dependency until that is
> resolved.
>
> Signed-off-by: Mark Brown <broonie@linaro.org>
Both [1-2/2] queued up for 3.14, thanks!
> ---
> drivers/cpufreq/Kconfig.arm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/cpufreq/Kconfig.arm b/drivers/cpufreq/Kconfig.arm
> index ce52ed949249..0a141109fd29 100644
> --- a/drivers/cpufreq/Kconfig.arm
> +++ b/drivers/cpufreq/Kconfig.arm
> @@ -4,7 +4,7 @@
>
> config ARM_BIG_LITTLE_CPUFREQ
> tristate "Generic ARM big LITTLE CPUfreq driver"
> - depends on ARM_CPU_TOPOLOGY && PM_OPP && HAVE_CLK
> + depends on ARM && ARM_CPU_TOPOLOGY && PM_OPP && HAVE_CLK
> help
> This enables the Generic CPUfreq driver for ARM big.LITTLE platforms.
>
>
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-12-22 1:06 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-11 22:12 [PATCH 1/2] cpufreq: Make ARM big.LITTLE switcher depend on ARM Mark Brown
2013-12-11 22:12 ` [PATCH 2/2] cpufreq: Select PM_OPP rather than depending on it Mark Brown
2013-12-16 9:20 ` Viresh Kumar
2013-12-16 9:20 ` [PATCH 1/2] cpufreq: Make ARM big.LITTLE switcher depend on ARM Viresh Kumar
2013-12-22 1:06 ` Rafael J. Wysocki
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).