* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
@ 2014-06-06 10:36 Mark Brown
2014-06-06 11:14 ` Simon Horman
` (4 more replies)
0 siblings, 5 replies; 17+ messages in thread
From: Mark Brown @ 2014-06-06 10:36 UTC (permalink / raw)
To: linux-arm-kernel
From: Mark Brown <broonie@linaro.org>
Since the OPP layer is a kernel library which has been converted to be
directly selectable by its callers rather than user selectable and
requiring architectures to enable it explicitly the ARCH_HAS_OPP symbol
has become redundant and can be removed. Do so.
Signed-off-by: Mark Brown <broonie@linaro.org>
---
Documentation/power/opp.txt | 3 ---
arch/arm/mach-exynos/Kconfig | 1 -
arch/arm/mach-highbank/Kconfig | 1 -
arch/arm/mach-imx/Kconfig | 1 -
arch/arm/mach-omap2/Kconfig | 1 -
arch/arm/mach-shmobile/Kconfig | 2 --
arch/arm/mach-vexpress/Kconfig | 1 -
arch/arm/mach-zynq/Kconfig | 1 -
drivers/devfreq/Kconfig | 1 -
kernel/power/Kconfig | 3 ---
10 files changed, 15 deletions(-)
diff --git a/Documentation/power/opp.txt b/Documentation/power/opp.txt
index a9adad8..c6279c2 100644
--- a/Documentation/power/opp.txt
+++ b/Documentation/power/opp.txt
@@ -51,9 +51,6 @@ Typical usage of the OPP library is as follows:
SoC framework -> modifies on required cases certain OPPs -> OPP layer
-> queries to search/retrieve information ->
-Architectures that provide a SoC framework for OPP should select ARCH_HAS_OPP
-to make the OPP layer available.
-
OPP layer expects each domain to be represented by a unique device pointer. SoC
framework registers a set of initial OPPs per device with the OPP layer. This
list is expected to be an optimally small number typically around 5 per device.
diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
index d58995c9..3f78c45 100644
--- a/arch/arm/mach-exynos/Kconfig
+++ b/arch/arm/mach-exynos/Kconfig
@@ -103,7 +103,6 @@ config SOC_EXYNOS5440
default y
depends on ARCH_EXYNOS5
select ARCH_DMA_ADDR_T_64BIT if ARM_LPAE
- select ARCH_HAS_OPP
select HAVE_ARM_ARCH_TIMER
select AUTO_ZRELADDR
select MIGHT_HAVE_PCI
diff --git a/arch/arm/mach-highbank/Kconfig b/arch/arm/mach-highbank/Kconfig
index 830b76e..bef970f 100644
--- a/arch/arm/mach-highbank/Kconfig
+++ b/arch/arm/mach-highbank/Kconfig
@@ -3,7 +3,6 @@ config ARCH_HIGHBANK
select ARCH_DMA_ADDR_T_64BIT if ARM_LPAE
select ARCH_HAS_CPUFREQ
select ARCH_HAS_HOLES_MEMORYMODEL
- select ARCH_HAS_OPP
select ARCH_SUPPORTS_BIG_ENDIAN
select ARM_AMBA
select ARM_ERRATA_764369 if SMP
diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
index 4cce93d..95690e4 100644
--- a/arch/arm/mach-imx/Kconfig
+++ b/arch/arm/mach-imx/Kconfig
@@ -1,7 +1,6 @@
config ARCH_MXC
bool "Freescale i.MX family" if ARCH_MULTI_V4_V5 || ARCH_MULTI_V6_V7
select ARCH_HAS_CPUFREQ
- select ARCH_HAS_OPP
select ARCH_REQUIRE_GPIOLIB
select ARM_CPU_SUSPEND if PM
select CLKSRC_MMIO
diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index 0ba4826..524b027 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -12,7 +12,6 @@ config ARCH_OMAP3
bool "TI OMAP3"
depends on ARCH_MULTI_V7
select ARCH_OMAP2PLUS
- select ARCH_HAS_OPP
select ARM_CPU_SUSPEND if PM
select OMAP_INTERCONNECT
select PM_OPP if PM
diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
index dbd954e..b51d142 100644
--- a/arch/arm/mach-shmobile/Kconfig
+++ b/arch/arm/mach-shmobile/Kconfig
@@ -86,7 +86,6 @@ config ARCH_R8A73A4
select SH_CLK_CPG
select RENESAS_IRQC
select ARCH_HAS_CPUFREQ
- select ARCH_HAS_OPP
select SYS_SUPPORTS_SH_CMT
select SYS_SUPPORTS_SH_TMU
@@ -265,7 +264,6 @@ config MACH_KZM9G
bool "KZM-A9-GT board"
depends on ARCH_SH73A0
select ARCH_HAS_CPUFREQ
- select ARCH_HAS_OPP
select ARCH_REQUIRE_GPIOLIB
select REGULATOR_FIXED_VOLTAGE if REGULATOR
select SND_SOC_AK4642 if SND_SIMPLE_CARD
diff --git a/arch/arm/mach-vexpress/Kconfig b/arch/arm/mach-vexpress/Kconfig
index 90249cf..71629fe 100644
--- a/arch/arm/mach-vexpress/Kconfig
+++ b/arch/arm/mach-vexpress/Kconfig
@@ -66,7 +66,6 @@ config ARCH_VEXPRESS_DCSCB
config ARCH_VEXPRESS_SPC
bool "Versatile Express Serial Power Controller (SPC)"
select ARCH_HAS_CPUFREQ
- select ARCH_HAS_OPP
select PM_OPP
help
The TC2 (A15x2 A7x3) versatile express core tile integrates a logic
diff --git a/arch/arm/mach-zynq/Kconfig b/arch/arm/mach-zynq/Kconfig
index 573e0db..bc485f2 100644
--- a/arch/arm/mach-zynq/Kconfig
+++ b/arch/arm/mach-zynq/Kconfig
@@ -1,7 +1,6 @@
config ARCH_ZYNQ
bool "Xilinx Zynq ARM Cortex A9 Platform" if ARCH_MULTI_V7
select ARCH_HAS_CPUFREQ
- select ARCH_HAS_OPP
select ARCH_SUPPORTS_BIG_ENDIAN
select ARM_AMBA
select ARM_GIC
diff --git a/drivers/devfreq/Kconfig b/drivers/devfreq/Kconfig
index 49e74c1..3dced0a 100644
--- a/drivers/devfreq/Kconfig
+++ b/drivers/devfreq/Kconfig
@@ -68,7 +68,6 @@ comment "DEVFREQ Drivers"
config ARM_EXYNOS4_BUS_DEVFREQ
bool "ARM Exynos4210/4212/4412 Memory Bus DEVFREQ Driver"
depends on (CPU_EXYNOS4210 || SOC_EXYNOS4212 || SOC_EXYNOS4412) && !ARCH_MULTIPLATFORM
- select ARCH_HAS_OPP
select DEVFREQ_GOV_SIMPLE_ONDEMAND
select PM_OPP
help
diff --git a/kernel/power/Kconfig b/kernel/power/Kconfig
index 9a83d78..e4e4121 100644
--- a/kernel/power/Kconfig
+++ b/kernel/power/Kconfig
@@ -253,9 +253,6 @@ config APM_EMULATION
anything, try disabling/enabling this option (or disabling/enabling
APM in your BIOS).
-config ARCH_HAS_OPP
- bool
-
config PM_OPP
bool
---help---
--
2.0.0
^ permalink raw reply related [flat|nested] 17+ messages in thread* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-06 10:36 [PATCH] PM / OPP: Remove ARCH_HAS_OPP Mark Brown
@ 2014-06-06 11:14 ` Simon Horman
2014-06-06 12:14 ` Magnus Damm
2014-06-06 13:45 ` Rob Herring
` (3 subsequent siblings)
4 siblings, 1 reply; 17+ messages in thread
From: Simon Horman @ 2014-06-06 11:14 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jun 06, 2014 at 11:36:56AM +0100, Mark Brown wrote:
> From: Mark Brown <broonie@linaro.org>
>
> Since the OPP layer is a kernel library which has been converted to be
> directly selectable by its callers rather than user selectable and
> requiring architectures to enable it explicitly the ARCH_HAS_OPP symbol
> has become redundant and can be removed. Do so.
>
> Signed-off-by: Mark Brown <broonie@linaro.org>
> ---
> Documentation/power/opp.txt | 3 ---
> arch/arm/mach-exynos/Kconfig | 1 -
> arch/arm/mach-highbank/Kconfig | 1 -
> arch/arm/mach-imx/Kconfig | 1 -
> arch/arm/mach-omap2/Kconfig | 1 -
> arch/arm/mach-shmobile/Kconfig | 2 --
> arch/arm/mach-vexpress/Kconfig | 1 -
> arch/arm/mach-zynq/Kconfig | 1 -
> drivers/devfreq/Kconfig | 1 -
> kernel/power/Kconfig | 3 ---
> 10 files changed, 15 deletions(-)
shmobile portion:
Acked-by: Simon Horman <horms+renesas@verge.net.au>
> diff --git a/Documentation/power/opp.txt b/Documentation/power/opp.txt
> index a9adad8..c6279c2 100644
> --- a/Documentation/power/opp.txt
> +++ b/Documentation/power/opp.txt
> @@ -51,9 +51,6 @@ Typical usage of the OPP library is as follows:
> SoC framework -> modifies on required cases certain OPPs -> OPP layer
> -> queries to search/retrieve information ->
>
> -Architectures that provide a SoC framework for OPP should select ARCH_HAS_OPP
> -to make the OPP layer available.
> -
> OPP layer expects each domain to be represented by a unique device pointer. SoC
> framework registers a set of initial OPPs per device with the OPP layer. This
> list is expected to be an optimally small number typically around 5 per device.
> diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
> index d58995c9..3f78c45 100644
> --- a/arch/arm/mach-exynos/Kconfig
> +++ b/arch/arm/mach-exynos/Kconfig
> @@ -103,7 +103,6 @@ config SOC_EXYNOS5440
> default y
> depends on ARCH_EXYNOS5
> select ARCH_DMA_ADDR_T_64BIT if ARM_LPAE
> - select ARCH_HAS_OPP
> select HAVE_ARM_ARCH_TIMER
> select AUTO_ZRELADDR
> select MIGHT_HAVE_PCI
> diff --git a/arch/arm/mach-highbank/Kconfig b/arch/arm/mach-highbank/Kconfig
> index 830b76e..bef970f 100644
> --- a/arch/arm/mach-highbank/Kconfig
> +++ b/arch/arm/mach-highbank/Kconfig
> @@ -3,7 +3,6 @@ config ARCH_HIGHBANK
> select ARCH_DMA_ADDR_T_64BIT if ARM_LPAE
> select ARCH_HAS_CPUFREQ
> select ARCH_HAS_HOLES_MEMORYMODEL
> - select ARCH_HAS_OPP
> select ARCH_SUPPORTS_BIG_ENDIAN
> select ARM_AMBA
> select ARM_ERRATA_764369 if SMP
> diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
> index 4cce93d..95690e4 100644
> --- a/arch/arm/mach-imx/Kconfig
> +++ b/arch/arm/mach-imx/Kconfig
> @@ -1,7 +1,6 @@
> config ARCH_MXC
> bool "Freescale i.MX family" if ARCH_MULTI_V4_V5 || ARCH_MULTI_V6_V7
> select ARCH_HAS_CPUFREQ
> - select ARCH_HAS_OPP
> select ARCH_REQUIRE_GPIOLIB
> select ARM_CPU_SUSPEND if PM
> select CLKSRC_MMIO
> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
> index 0ba4826..524b027 100644
> --- a/arch/arm/mach-omap2/Kconfig
> +++ b/arch/arm/mach-omap2/Kconfig
> @@ -12,7 +12,6 @@ config ARCH_OMAP3
> bool "TI OMAP3"
> depends on ARCH_MULTI_V7
> select ARCH_OMAP2PLUS
> - select ARCH_HAS_OPP
> select ARM_CPU_SUSPEND if PM
> select OMAP_INTERCONNECT
> select PM_OPP if PM
> diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
> index dbd954e..b51d142 100644
> --- a/arch/arm/mach-shmobile/Kconfig
> +++ b/arch/arm/mach-shmobile/Kconfig
> @@ -86,7 +86,6 @@ config ARCH_R8A73A4
> select SH_CLK_CPG
> select RENESAS_IRQC
> select ARCH_HAS_CPUFREQ
> - select ARCH_HAS_OPP
> select SYS_SUPPORTS_SH_CMT
> select SYS_SUPPORTS_SH_TMU
>
> @@ -265,7 +264,6 @@ config MACH_KZM9G
> bool "KZM-A9-GT board"
> depends on ARCH_SH73A0
> select ARCH_HAS_CPUFREQ
> - select ARCH_HAS_OPP
> select ARCH_REQUIRE_GPIOLIB
> select REGULATOR_FIXED_VOLTAGE if REGULATOR
> select SND_SOC_AK4642 if SND_SIMPLE_CARD
> diff --git a/arch/arm/mach-vexpress/Kconfig b/arch/arm/mach-vexpress/Kconfig
> index 90249cf..71629fe 100644
> --- a/arch/arm/mach-vexpress/Kconfig
> +++ b/arch/arm/mach-vexpress/Kconfig
> @@ -66,7 +66,6 @@ config ARCH_VEXPRESS_DCSCB
> config ARCH_VEXPRESS_SPC
> bool "Versatile Express Serial Power Controller (SPC)"
> select ARCH_HAS_CPUFREQ
> - select ARCH_HAS_OPP
> select PM_OPP
> help
> The TC2 (A15x2 A7x3) versatile express core tile integrates a logic
> diff --git a/arch/arm/mach-zynq/Kconfig b/arch/arm/mach-zynq/Kconfig
> index 573e0db..bc485f2 100644
> --- a/arch/arm/mach-zynq/Kconfig
> +++ b/arch/arm/mach-zynq/Kconfig
> @@ -1,7 +1,6 @@
> config ARCH_ZYNQ
> bool "Xilinx Zynq ARM Cortex A9 Platform" if ARCH_MULTI_V7
> select ARCH_HAS_CPUFREQ
> - select ARCH_HAS_OPP
> select ARCH_SUPPORTS_BIG_ENDIAN
> select ARM_AMBA
> select ARM_GIC
> diff --git a/drivers/devfreq/Kconfig b/drivers/devfreq/Kconfig
> index 49e74c1..3dced0a 100644
> --- a/drivers/devfreq/Kconfig
> +++ b/drivers/devfreq/Kconfig
> @@ -68,7 +68,6 @@ comment "DEVFREQ Drivers"
> config ARM_EXYNOS4_BUS_DEVFREQ
> bool "ARM Exynos4210/4212/4412 Memory Bus DEVFREQ Driver"
> depends on (CPU_EXYNOS4210 || SOC_EXYNOS4212 || SOC_EXYNOS4412) && !ARCH_MULTIPLATFORM
> - select ARCH_HAS_OPP
> select DEVFREQ_GOV_SIMPLE_ONDEMAND
> select PM_OPP
> help
> diff --git a/kernel/power/Kconfig b/kernel/power/Kconfig
> index 9a83d78..e4e4121 100644
> --- a/kernel/power/Kconfig
> +++ b/kernel/power/Kconfig
> @@ -253,9 +253,6 @@ config APM_EMULATION
> anything, try disabling/enabling this option (or disabling/enabling
> APM in your BIOS).
>
> -config ARCH_HAS_OPP
> - bool
> -
> config PM_OPP
> bool
> ---help---
> --
> 2.0.0
>
^ permalink raw reply [flat|nested] 17+ messages in thread* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-06 11:14 ` Simon Horman
@ 2014-06-06 12:14 ` Magnus Damm
2014-06-06 12:50 ` Simon Horman
0 siblings, 1 reply; 17+ messages in thread
From: Magnus Damm @ 2014-06-06 12:14 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jun 6, 2014 at 8:14 PM, Simon Horman <horms@verge.net.au> wrote:
> On Fri, Jun 06, 2014 at 11:36:56AM +0100, Mark Brown wrote:
>> From: Mark Brown <broonie@linaro.org>
>>
>> Since the OPP layer is a kernel library which has been converted to be
>> directly selectable by its callers rather than user selectable and
>> requiring architectures to enable it explicitly the ARCH_HAS_OPP symbol
>> has become redundant and can be removed. Do so.
>>
>> Signed-off-by: Mark Brown <broonie@linaro.org>
>> ---
>> Documentation/power/opp.txt | 3 ---
>> arch/arm/mach-exynos/Kconfig | 1 -
>> arch/arm/mach-highbank/Kconfig | 1 -
>> arch/arm/mach-imx/Kconfig | 1 -
>> arch/arm/mach-omap2/Kconfig | 1 -
>> arch/arm/mach-shmobile/Kconfig | 2 --
>> arch/arm/mach-vexpress/Kconfig | 1 -
>> arch/arm/mach-zynq/Kconfig | 1 -
>> drivers/devfreq/Kconfig | 1 -
>> kernel/power/Kconfig | 3 ---
>> 10 files changed, 15 deletions(-)
>
> shmobile portion:
>
> Acked-by: Simon Horman <horms+renesas@verge.net.au>
Hi Simon, Mark,
Nice to see cleanups in this area. Reducing the number of Kconfig
symbols must be a good thing.
I'm not sure about the expected merge order for this kind of change vs
queued up stuff in the renesas git tree, but I believe the following
patch selects ARCH_HAS_OPP:
[PATCH v3] ARM: shmobile: Mark all SoCs in shmobile as CPUFreq, capable
Cheers,
/ magnus
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-06 12:14 ` Magnus Damm
@ 2014-06-06 12:50 ` Simon Horman
2014-06-06 13:01 ` Magnus Damm
2014-06-06 13:08 ` Mark Brown
0 siblings, 2 replies; 17+ messages in thread
From: Simon Horman @ 2014-06-06 12:50 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jun 06, 2014 at 09:14:01PM +0900, Magnus Damm wrote:
> On Fri, Jun 6, 2014 at 8:14 PM, Simon Horman <horms@verge.net.au> wrote:
> > On Fri, Jun 06, 2014 at 11:36:56AM +0100, Mark Brown wrote:
> >> From: Mark Brown <broonie@linaro.org>
> >>
> >> Since the OPP layer is a kernel library which has been converted to be
> >> directly selectable by its callers rather than user selectable and
> >> requiring architectures to enable it explicitly the ARCH_HAS_OPP symbol
> >> has become redundant and can be removed. Do so.
> >>
> >> Signed-off-by: Mark Brown <broonie@linaro.org>
> >> ---
> >> Documentation/power/opp.txt | 3 ---
> >> arch/arm/mach-exynos/Kconfig | 1 -
> >> arch/arm/mach-highbank/Kconfig | 1 -
> >> arch/arm/mach-imx/Kconfig | 1 -
> >> arch/arm/mach-omap2/Kconfig | 1 -
> >> arch/arm/mach-shmobile/Kconfig | 2 --
> >> arch/arm/mach-vexpress/Kconfig | 1 -
> >> arch/arm/mach-zynq/Kconfig | 1 -
> >> drivers/devfreq/Kconfig | 1 -
> >> kernel/power/Kconfig | 3 ---
> >> 10 files changed, 15 deletions(-)
> >
> > shmobile portion:
> >
> > Acked-by: Simon Horman <horms+renesas@verge.net.au>
>
> Hi Simon, Mark,
>
> Nice to see cleanups in this area. Reducing the number of Kconfig
> symbols must be a good thing.
>
> I'm not sure about the expected merge order for this kind of change vs
> queued up stuff in the renesas git tree, but I believe the following
> patch selects ARCH_HAS_OPP:
>
> [PATCH v3] ARM: shmobile: Mark all SoCs in shmobile as CPUFreq, capable
I propose that we fix that up by adding an incremental patch to
mach-shmobile via my renesas tree once the dependency (assuming there is
one) is in Linus's tree.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-06 12:50 ` Simon Horman
@ 2014-06-06 13:01 ` Magnus Damm
2014-06-06 13:08 ` Mark Brown
1 sibling, 0 replies; 17+ messages in thread
From: Magnus Damm @ 2014-06-06 13:01 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jun 6, 2014 at 9:50 PM, Simon Horman <horms@verge.net.au> wrote:
> On Fri, Jun 06, 2014 at 09:14:01PM +0900, Magnus Damm wrote:
>> On Fri, Jun 6, 2014 at 8:14 PM, Simon Horman <horms@verge.net.au> wrote:
>> > On Fri, Jun 06, 2014 at 11:36:56AM +0100, Mark Brown wrote:
>> >> From: Mark Brown <broonie@linaro.org>
>> >>
>> >> Since the OPP layer is a kernel library which has been converted to be
>> >> directly selectable by its callers rather than user selectable and
>> >> requiring architectures to enable it explicitly the ARCH_HAS_OPP symbol
>> >> has become redundant and can be removed. Do so.
>> >>
>> >> Signed-off-by: Mark Brown <broonie@linaro.org>
>> >> ---
>> >> Documentation/power/opp.txt | 3 ---
>> >> arch/arm/mach-exynos/Kconfig | 1 -
>> >> arch/arm/mach-highbank/Kconfig | 1 -
>> >> arch/arm/mach-imx/Kconfig | 1 -
>> >> arch/arm/mach-omap2/Kconfig | 1 -
>> >> arch/arm/mach-shmobile/Kconfig | 2 --
>> >> arch/arm/mach-vexpress/Kconfig | 1 -
>> >> arch/arm/mach-zynq/Kconfig | 1 -
>> >> drivers/devfreq/Kconfig | 1 -
>> >> kernel/power/Kconfig | 3 ---
>> >> 10 files changed, 15 deletions(-)
>> >
>> > shmobile portion:
>> >
>> > Acked-by: Simon Horman <horms+renesas@verge.net.au>
>>
>> Hi Simon, Mark,
>>
>> Nice to see cleanups in this area. Reducing the number of Kconfig
>> symbols must be a good thing.
>>
>> I'm not sure about the expected merge order for this kind of change vs
>> queued up stuff in the renesas git tree, but I believe the following
>> patch selects ARCH_HAS_OPP:
>>
>> [PATCH v3] ARM: shmobile: Mark all SoCs in shmobile as CPUFreq, capable
>
> I propose that we fix that up by adding an incremental patch to
> mach-shmobile via my renesas tree once the dependency (assuming there is
> one) is in Linus's tree.
Sounds good. Thanks!
/ magnus
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-06 12:50 ` Simon Horman
2014-06-06 13:01 ` Magnus Damm
@ 2014-06-06 13:08 ` Mark Brown
2014-06-06 21:15 ` Rafael J. Wysocki
1 sibling, 1 reply; 17+ messages in thread
From: Mark Brown @ 2014-06-06 13:08 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jun 06, 2014 at 09:50:06PM +0900, Simon Horman wrote:
> On Fri, Jun 06, 2014 at 09:14:01PM +0900, Magnus Damm wrote:
> > I'm not sure about the expected merge order for this kind of change vs
> > queued up stuff in the renesas git tree, but I believe the following
> > patch selects ARCH_HAS_OPP:
> > [PATCH v3] ARM: shmobile: Mark all SoCs in shmobile as CPUFreq, capable
> I propose that we fix that up by adding an incremental patch to
> mach-shmobile via my renesas tree once the dependency (assuming there is
> one) is in Linus's tree.
Yes, the conversion to make ARCH_HAS_OPP unused is in Raphael's tree for
the merge window. I'd expect this patch to end up going after the merge
window (or at the end I guess).
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140606/e1c776b1/attachment.sig>
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-06 13:08 ` Mark Brown
@ 2014-06-06 21:15 ` Rafael J. Wysocki
2014-06-09 19:49 ` Mark Brown
0 siblings, 1 reply; 17+ messages in thread
From: Rafael J. Wysocki @ 2014-06-06 21:15 UTC (permalink / raw)
To: linux-arm-kernel
On Friday, June 06, 2014 02:08:50 PM Mark Brown wrote:
>
> --cU9XODsizZBnwgll
> Content-Type: text/plain; charset=us-ascii
> Content-Disposition: inline
>
> On Fri, Jun 06, 2014 at 09:50:06PM +0900, Simon Horman wrote:
> > On Fri, Jun 06, 2014 at 09:14:01PM +0900, Magnus Damm wrote:
>
> > > I'm not sure about the expected merge order for this kind of change vs
> > > queued up stuff in the renesas git tree, but I believe the following
> > > patch selects ARCH_HAS_OPP:
>
> > > [PATCH v3] ARM: shmobile: Mark all SoCs in shmobile as CPUFreq, capable
>
> > I propose that we fix that up by adding an incremental patch to
> > mach-shmobile via my renesas tree once the dependency (assuming there is
> > one) is in Linus's tree.
>
> Yes, the conversion to make ARCH_HAS_OPP unused is in Raphael's tree for
> the merge window.
Perhaps already merged?
Rafael
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-06 21:15 ` Rafael J. Wysocki
@ 2014-06-09 19:49 ` Mark Brown
2014-06-09 22:22 ` Rafael J. Wysocki
0 siblings, 1 reply; 17+ messages in thread
From: Mark Brown @ 2014-06-09 19:49 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jun 06, 2014 at 11:15:06PM +0200, Rafael J. Wysocki wrote:
> On Friday, June 06, 2014 02:08:50 PM Mark Brown wrote:
> > Yes, the conversion to make ARCH_HAS_OPP unused is in Raphael's tree for
> > the merge window.
> Perhaps already merged?
Yes, looks like it is.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140609/f9241515/attachment.sig>
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-09 19:49 ` Mark Brown
@ 2014-06-09 22:22 ` Rafael J. Wysocki
2014-06-09 22:51 ` Mark Brown
0 siblings, 1 reply; 17+ messages in thread
From: Rafael J. Wysocki @ 2014-06-09 22:22 UTC (permalink / raw)
To: linux-arm-kernel
On Monday, June 09, 2014 08:49:17 PM Mark Brown wrote:
> On Fri, Jun 06, 2014 at 11:15:06PM +0200, Rafael J. Wysocki wrote:
> > On Friday, June 06, 2014 02:08:50 PM Mark Brown wrote:
>
> > > Yes, the conversion to make ARCH_HAS_OPP unused is in Raphael's tree for
> > > the merge window.
>
> > Perhaps already merged?
>
> Yes, looks like it is.
But it doesn't apply for me on top of the current Linus' tree.
Can you please rebase?
Rafael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140610/1f4a686e/attachment.sig>
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-09 22:22 ` Rafael J. Wysocki
@ 2014-06-09 22:51 ` Mark Brown
2014-06-10 0:15 ` Rafael J. Wysocki
0 siblings, 1 reply; 17+ messages in thread
From: Mark Brown @ 2014-06-09 22:51 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Jun 10, 2014 at 12:22:07AM +0200, Rafael J. Wysocki wrote:
> On Monday, June 09, 2014 08:49:17 PM Mark Brown wrote:
> > Yes, looks like it is.
> But it doesn't apply for me on top of the current Linus' tree.
> Can you please rebase?
It probably depends on other people's trees too, I just killed all the
usages in -next - I gather the SH trees added some new ones for example.
I'd expect it'll apply after the merge window, if not that's the time to
look at it.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140609/8e1c04eb/attachment-0001.sig>
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-09 22:51 ` Mark Brown
@ 2014-06-10 0:15 ` Rafael J. Wysocki
0 siblings, 0 replies; 17+ messages in thread
From: Rafael J. Wysocki @ 2014-06-10 0:15 UTC (permalink / raw)
To: linux-arm-kernel
On Monday, June 09, 2014 11:51:42 PM Mark Brown wrote:
>
> --fYgRtaZIy+F1uhp1
> Content-Type: text/plain; charset=us-ascii
> Content-Disposition: inline
>
> On Tue, Jun 10, 2014 at 12:22:07AM +0200, Rafael J. Wysocki wrote:
> > On Monday, June 09, 2014 08:49:17 PM Mark Brown wrote:
>
> > > Yes, looks like it is.
>
> > But it doesn't apply for me on top of the current Linus' tree.
>
> > Can you please rebase?
>
> It probably depends on other people's trees too, I just killed all the
> usages in -next - I gather the SH trees added some new ones for example.
> I'd expect it'll apply after the merge window, if not that's the time to
> look at it.
OK
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-06 10:36 [PATCH] PM / OPP: Remove ARCH_HAS_OPP Mark Brown
2014-06-06 11:14 ` Simon Horman
@ 2014-06-06 13:45 ` Rob Herring
2014-06-06 14:33 ` Nishanth Menon
` (2 subsequent siblings)
4 siblings, 0 replies; 17+ messages in thread
From: Rob Herring @ 2014-06-06 13:45 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jun 6, 2014 at 5:36 AM, Mark Brown <broonie@kernel.org> wrote:
> From: Mark Brown <broonie@linaro.org>
>
> Since the OPP layer is a kernel library which has been converted to be
> directly selectable by its callers rather than user selectable and
> requiring architectures to enable it explicitly the ARCH_HAS_OPP symbol
> has become redundant and can be removed. Do so.
>
> Signed-off-by: Mark Brown <broonie@linaro.org>
Acked-by: Rob Herring <robh@kernel.org>
> ---
> Documentation/power/opp.txt | 3 ---
> arch/arm/mach-exynos/Kconfig | 1 -
> arch/arm/mach-highbank/Kconfig | 1 -
> arch/arm/mach-imx/Kconfig | 1 -
> arch/arm/mach-omap2/Kconfig | 1 -
> arch/arm/mach-shmobile/Kconfig | 2 --
> arch/arm/mach-vexpress/Kconfig | 1 -
> arch/arm/mach-zynq/Kconfig | 1 -
> drivers/devfreq/Kconfig | 1 -
> kernel/power/Kconfig | 3 ---
> 10 files changed, 15 deletions(-)
>
> diff --git a/Documentation/power/opp.txt b/Documentation/power/opp.txt
> index a9adad8..c6279c2 100644
> --- a/Documentation/power/opp.txt
> +++ b/Documentation/power/opp.txt
> @@ -51,9 +51,6 @@ Typical usage of the OPP library is as follows:
> SoC framework -> modifies on required cases certain OPPs -> OPP layer
> -> queries to search/retrieve information ->
>
> -Architectures that provide a SoC framework for OPP should select ARCH_HAS_OPP
> -to make the OPP layer available.
> -
> OPP layer expects each domain to be represented by a unique device pointer. SoC
> framework registers a set of initial OPPs per device with the OPP layer. This
> list is expected to be an optimally small number typically around 5 per device.
> diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
> index d58995c9..3f78c45 100644
> --- a/arch/arm/mach-exynos/Kconfig
> +++ b/arch/arm/mach-exynos/Kconfig
> @@ -103,7 +103,6 @@ config SOC_EXYNOS5440
> default y
> depends on ARCH_EXYNOS5
> select ARCH_DMA_ADDR_T_64BIT if ARM_LPAE
> - select ARCH_HAS_OPP
> select HAVE_ARM_ARCH_TIMER
> select AUTO_ZRELADDR
> select MIGHT_HAVE_PCI
> diff --git a/arch/arm/mach-highbank/Kconfig b/arch/arm/mach-highbank/Kconfig
> index 830b76e..bef970f 100644
> --- a/arch/arm/mach-highbank/Kconfig
> +++ b/arch/arm/mach-highbank/Kconfig
> @@ -3,7 +3,6 @@ config ARCH_HIGHBANK
> select ARCH_DMA_ADDR_T_64BIT if ARM_LPAE
> select ARCH_HAS_CPUFREQ
> select ARCH_HAS_HOLES_MEMORYMODEL
> - select ARCH_HAS_OPP
> select ARCH_SUPPORTS_BIG_ENDIAN
> select ARM_AMBA
> select ARM_ERRATA_764369 if SMP
> diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
> index 4cce93d..95690e4 100644
> --- a/arch/arm/mach-imx/Kconfig
> +++ b/arch/arm/mach-imx/Kconfig
> @@ -1,7 +1,6 @@
> config ARCH_MXC
> bool "Freescale i.MX family" if ARCH_MULTI_V4_V5 || ARCH_MULTI_V6_V7
> select ARCH_HAS_CPUFREQ
> - select ARCH_HAS_OPP
> select ARCH_REQUIRE_GPIOLIB
> select ARM_CPU_SUSPEND if PM
> select CLKSRC_MMIO
> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
> index 0ba4826..524b027 100644
> --- a/arch/arm/mach-omap2/Kconfig
> +++ b/arch/arm/mach-omap2/Kconfig
> @@ -12,7 +12,6 @@ config ARCH_OMAP3
> bool "TI OMAP3"
> depends on ARCH_MULTI_V7
> select ARCH_OMAP2PLUS
> - select ARCH_HAS_OPP
> select ARM_CPU_SUSPEND if PM
> select OMAP_INTERCONNECT
> select PM_OPP if PM
> diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
> index dbd954e..b51d142 100644
> --- a/arch/arm/mach-shmobile/Kconfig
> +++ b/arch/arm/mach-shmobile/Kconfig
> @@ -86,7 +86,6 @@ config ARCH_R8A73A4
> select SH_CLK_CPG
> select RENESAS_IRQC
> select ARCH_HAS_CPUFREQ
> - select ARCH_HAS_OPP
> select SYS_SUPPORTS_SH_CMT
> select SYS_SUPPORTS_SH_TMU
>
> @@ -265,7 +264,6 @@ config MACH_KZM9G
> bool "KZM-A9-GT board"
> depends on ARCH_SH73A0
> select ARCH_HAS_CPUFREQ
> - select ARCH_HAS_OPP
> select ARCH_REQUIRE_GPIOLIB
> select REGULATOR_FIXED_VOLTAGE if REGULATOR
> select SND_SOC_AK4642 if SND_SIMPLE_CARD
> diff --git a/arch/arm/mach-vexpress/Kconfig b/arch/arm/mach-vexpress/Kconfig
> index 90249cf..71629fe 100644
> --- a/arch/arm/mach-vexpress/Kconfig
> +++ b/arch/arm/mach-vexpress/Kconfig
> @@ -66,7 +66,6 @@ config ARCH_VEXPRESS_DCSCB
> config ARCH_VEXPRESS_SPC
> bool "Versatile Express Serial Power Controller (SPC)"
> select ARCH_HAS_CPUFREQ
> - select ARCH_HAS_OPP
> select PM_OPP
> help
> The TC2 (A15x2 A7x3) versatile express core tile integrates a logic
> diff --git a/arch/arm/mach-zynq/Kconfig b/arch/arm/mach-zynq/Kconfig
> index 573e0db..bc485f2 100644
> --- a/arch/arm/mach-zynq/Kconfig
> +++ b/arch/arm/mach-zynq/Kconfig
> @@ -1,7 +1,6 @@
> config ARCH_ZYNQ
> bool "Xilinx Zynq ARM Cortex A9 Platform" if ARCH_MULTI_V7
> select ARCH_HAS_CPUFREQ
> - select ARCH_HAS_OPP
> select ARCH_SUPPORTS_BIG_ENDIAN
> select ARM_AMBA
> select ARM_GIC
> diff --git a/drivers/devfreq/Kconfig b/drivers/devfreq/Kconfig
> index 49e74c1..3dced0a 100644
> --- a/drivers/devfreq/Kconfig
> +++ b/drivers/devfreq/Kconfig
> @@ -68,7 +68,6 @@ comment "DEVFREQ Drivers"
> config ARM_EXYNOS4_BUS_DEVFREQ
> bool "ARM Exynos4210/4212/4412 Memory Bus DEVFREQ Driver"
> depends on (CPU_EXYNOS4210 || SOC_EXYNOS4212 || SOC_EXYNOS4412) && !ARCH_MULTIPLATFORM
> - select ARCH_HAS_OPP
> select DEVFREQ_GOV_SIMPLE_ONDEMAND
> select PM_OPP
> help
> diff --git a/kernel/power/Kconfig b/kernel/power/Kconfig
> index 9a83d78..e4e4121 100644
> --- a/kernel/power/Kconfig
> +++ b/kernel/power/Kconfig
> @@ -253,9 +253,6 @@ config APM_EMULATION
> anything, try disabling/enabling this option (or disabling/enabling
> APM in your BIOS).
>
> -config ARCH_HAS_OPP
> - bool
> -
> config PM_OPP
> bool
> ---help---
> --
> 2.0.0
>
^ permalink raw reply [flat|nested] 17+ messages in thread* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-06 10:36 [PATCH] PM / OPP: Remove ARCH_HAS_OPP Mark Brown
2014-06-06 11:14 ` Simon Horman
2014-06-06 13:45 ` Rob Herring
@ 2014-06-06 14:33 ` Nishanth Menon
2014-06-09 1:56 ` Shawn Guo
2014-06-09 4:49 ` Viresh Kumar
4 siblings, 0 replies; 17+ messages in thread
From: Nishanth Menon @ 2014-06-06 14:33 UTC (permalink / raw)
To: linux-arm-kernel
On 06/06/2014 05:36 AM, Mark Brown wrote:
[...]
> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
> index 0ba4826..524b027 100644
> --- a/arch/arm/mach-omap2/Kconfig
> +++ b/arch/arm/mach-omap2/Kconfig
> @@ -12,7 +12,6 @@ config ARCH_OMAP3
> bool "TI OMAP3"
> depends on ARCH_MULTI_V7
> select ARCH_OMAP2PLUS
> - select ARCH_HAS_OPP
> select ARM_CPU_SUSPEND if PM
> select OMAP_INTERCONNECT
> select PM_OPP if PM
For OMAP portion:
Acked-by: Nishanth Menon <nm@ti.com>
--
Regards,
Nishanth Menon
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-06 10:36 [PATCH] PM / OPP: Remove ARCH_HAS_OPP Mark Brown
` (2 preceding siblings ...)
2014-06-06 14:33 ` Nishanth Menon
@ 2014-06-09 1:56 ` Shawn Guo
2014-06-09 4:49 ` Viresh Kumar
4 siblings, 0 replies; 17+ messages in thread
From: Shawn Guo @ 2014-06-09 1:56 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jun 06, 2014 at 11:36:56AM +0100, Mark Brown wrote:
> From: Mark Brown <broonie@linaro.org>
>
> Since the OPP layer is a kernel library which has been converted to be
> directly selectable by its callers rather than user selectable and
> requiring architectures to enable it explicitly the ARCH_HAS_OPP symbol
> has become redundant and can be removed. Do so.
>
> Signed-off-by: Mark Brown <broonie@linaro.org>
> ---
...
> arch/arm/mach-imx/Kconfig | 1 -
Acked-by: Shawn Guo <shawn.guo@freescale.com>
^ permalink raw reply [flat|nested] 17+ messages in thread* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-06-06 10:36 [PATCH] PM / OPP: Remove ARCH_HAS_OPP Mark Brown
` (3 preceding siblings ...)
2014-06-09 1:56 ` Shawn Guo
@ 2014-06-09 4:49 ` Viresh Kumar
4 siblings, 0 replies; 17+ messages in thread
From: Viresh Kumar @ 2014-06-09 4:49 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jun 6, 2014 at 4:06 PM, Mark Brown <broonie@kernel.org> wrote:
> From: Mark Brown <broonie@linaro.org>
>
> Since the OPP layer is a kernel library which has been converted to be
> directly selectable by its callers rather than user selectable and
> requiring architectures to enable it explicitly the ARCH_HAS_OPP symbol
> has become redundant and can be removed. Do so.
>
> Signed-off-by: Mark Brown <broonie@linaro.org>
> ---
> Documentation/power/opp.txt | 3 ---
> arch/arm/mach-exynos/Kconfig | 1 -
> arch/arm/mach-highbank/Kconfig | 1 -
> arch/arm/mach-imx/Kconfig | 1 -
> arch/arm/mach-omap2/Kconfig | 1 -
> arch/arm/mach-shmobile/Kconfig | 2 --
> arch/arm/mach-vexpress/Kconfig | 1 -
> arch/arm/mach-zynq/Kconfig | 1 -
> drivers/devfreq/Kconfig | 1 -
> kernel/power/Kconfig | 3 ---
> 10 files changed, 15 deletions(-)
Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
@ 2014-07-22 14:43 Mark Brown
2014-07-22 23:51 ` Rafael J. Wysocki
0 siblings, 1 reply; 17+ messages in thread
From: Mark Brown @ 2014-07-22 14:43 UTC (permalink / raw)
To: linux-arm-kernel
From: Mark Brown <broonie@linaro.org>
Since the OPP layer is a kernel library which has been converted to be
directly selectable by its callers rather than user selectable and
requiring architectures to enable it explicitly the ARCH_HAS_OPP symbol
has become redundant and can be removed. Do so.
Signed-off-by: Mark Brown <broonie@linaro.org>
Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
Acked-by: Nishanth Menon <nm@ti.com>
Acked-by: Rob Herring <robh@kernel.org>
Acked-by: Shawn Guo <shawn.guo@freescale.com>
Acked-by: Simon Horman <horms+renesas@verge.net.au>
---
Raphael, IIRC you said that you'd applied this but it seems not to have
appeared in -next. There's a few more references crept in, I'll try to
get them removed at source.
Documentation/power/opp.txt | 3 ---
arch/arm/mach-exynos/Kconfig | 1 -
arch/arm/mach-highbank/Kconfig | 1 -
arch/arm/mach-imx/Kconfig | 1 -
arch/arm/mach-omap2/Kconfig | 1 -
arch/arm/mach-shmobile/Kconfig | 2 --
arch/arm/mach-vexpress/Kconfig | 1 -
arch/arm/mach-zynq/Kconfig | 1 -
drivers/devfreq/Kconfig | 1 -
kernel/power/Kconfig | 3 ---
10 files changed, 15 deletions(-)
diff --git a/Documentation/power/opp.txt b/Documentation/power/opp.txt
index a9adad828cdc..c6279c2be47c 100644
--- a/Documentation/power/opp.txt
+++ b/Documentation/power/opp.txt
@@ -51,9 +51,6 @@ Typical usage of the OPP library is as follows:
SoC framework -> modifies on required cases certain OPPs -> OPP layer
-> queries to search/retrieve information ->
-Architectures that provide a SoC framework for OPP should select ARCH_HAS_OPP
-to make the OPP layer available.
-
OPP layer expects each domain to be represented by a unique device pointer. SoC
framework registers a set of initial OPPs per device with the OPP layer. This
list is expected to be an optimally small number typically around 5 per device.
diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
index 8f9b66c4ac78..f7889f6a1353 100644
--- a/arch/arm/mach-exynos/Kconfig
+++ b/arch/arm/mach-exynos/Kconfig
@@ -100,7 +100,6 @@ config SOC_EXYNOS5440
default y
depends on ARCH_EXYNOS5
select ARCH_DMA_ADDR_T_64BIT if ARM_LPAE
- select ARCH_HAS_OPP
select HAVE_ARM_ARCH_TIMER
select AUTO_ZRELADDR
select MIGHT_HAVE_PCI
diff --git a/arch/arm/mach-highbank/Kconfig b/arch/arm/mach-highbank/Kconfig
index a5960e2ac090..31aa866c3317 100644
--- a/arch/arm/mach-highbank/Kconfig
+++ b/arch/arm/mach-highbank/Kconfig
@@ -2,7 +2,6 @@ config ARCH_HIGHBANK
bool "Calxeda ECX-1000/2000 (Highbank/Midway)" if ARCH_MULTI_V7
select ARCH_DMA_ADDR_T_64BIT if ARM_LPAE
select ARCH_HAS_HOLES_MEMORYMODEL
- select ARCH_HAS_OPP
select ARCH_SUPPORTS_BIG_ENDIAN
select ARM_AMBA
select ARM_ERRATA_764369 if SMP
diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
index 2bc7b97861b4..9de84a215abd 100644
--- a/arch/arm/mach-imx/Kconfig
+++ b/arch/arm/mach-imx/Kconfig
@@ -1,6 +1,5 @@
menuconfig ARCH_MXC
bool "Freescale i.MX family" if ARCH_MULTI_V4_V5 || ARCH_MULTI_V6_V7
- select ARCH_HAS_OPP
select ARCH_REQUIRE_GPIOLIB
select ARM_CPU_SUSPEND if PM
select CLKSRC_MMIO
diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index 1c1ed737f7ab..e7189dcc9309 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -15,7 +15,6 @@ config ARCH_OMAP3
bool "TI OMAP3"
depends on ARCH_MULTI_V7
select ARCH_OMAP2PLUS
- select ARCH_HAS_OPP
select ARM_CPU_SUSPEND if PM
select OMAP_INTERCONNECT
select PM_OPP if PM
diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
index 384221d72d22..5814754c1240 100644
--- a/arch/arm/mach-shmobile/Kconfig
+++ b/arch/arm/mach-shmobile/Kconfig
@@ -92,7 +92,6 @@ config ARCH_R8A73A4
select CPU_V7
select SH_CLK_CPG
select RENESAS_IRQC
- select ARCH_HAS_OPP
select SYS_SUPPORTS_SH_CMT
select SYS_SUPPORTS_SH_TMU
@@ -244,7 +243,6 @@ config MACH_KOELSCH
config MACH_KZM9G
bool "KZM-A9-GT board"
depends on ARCH_SH73A0
- select ARCH_HAS_OPP
select ARCH_REQUIRE_GPIOLIB
select REGULATOR_FIXED_VOLTAGE if REGULATOR
select SND_SOC_AK4642 if SND_SIMPLE_CARD
diff --git a/arch/arm/mach-vexpress/Kconfig b/arch/arm/mach-vexpress/Kconfig
index d8b9330f896a..1af70329b88d 100644
--- a/arch/arm/mach-vexpress/Kconfig
+++ b/arch/arm/mach-vexpress/Kconfig
@@ -64,7 +64,6 @@ config ARCH_VEXPRESS_DCSCB
config ARCH_VEXPRESS_SPC
bool "Versatile Express Serial Power Controller (SPC)"
- select ARCH_HAS_OPP
select PM_OPP
help
The TC2 (A15x2 A7x3) versatile express core tile integrates a logic
diff --git a/arch/arm/mach-zynq/Kconfig b/arch/arm/mach-zynq/Kconfig
index 0c164f81e72d..aaa5162c1509 100644
--- a/arch/arm/mach-zynq/Kconfig
+++ b/arch/arm/mach-zynq/Kconfig
@@ -1,6 +1,5 @@
config ARCH_ZYNQ
bool "Xilinx Zynq ARM Cortex A9 Platform" if ARCH_MULTI_V7
- select ARCH_HAS_OPP
select ARCH_SUPPORTS_BIG_ENDIAN
select ARM_AMBA
select ARM_GIC
diff --git a/drivers/devfreq/Kconfig b/drivers/devfreq/Kconfig
index 49e74c1fc639..3dced0a9eae3 100644
--- a/drivers/devfreq/Kconfig
+++ b/drivers/devfreq/Kconfig
@@ -68,7 +68,6 @@ comment "DEVFREQ Drivers"
config ARM_EXYNOS4_BUS_DEVFREQ
bool "ARM Exynos4210/4212/4412 Memory Bus DEVFREQ Driver"
depends on (CPU_EXYNOS4210 || SOC_EXYNOS4212 || SOC_EXYNOS4412) && !ARCH_MULTIPLATFORM
- select ARCH_HAS_OPP
select DEVFREQ_GOV_SIMPLE_ONDEMAND
select PM_OPP
help
diff --git a/kernel/power/Kconfig b/kernel/power/Kconfig
index 9a83d780facd..e4e4121fa327 100644
--- a/kernel/power/Kconfig
+++ b/kernel/power/Kconfig
@@ -253,9 +253,6 @@ config APM_EMULATION
anything, try disabling/enabling this option (or disabling/enabling
APM in your BIOS).
-config ARCH_HAS_OPP
- bool
-
config PM_OPP
bool
---help---
--
2.0.1
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH] PM / OPP: Remove ARCH_HAS_OPP
2014-07-22 14:43 Mark Brown
@ 2014-07-22 23:51 ` Rafael J. Wysocki
0 siblings, 0 replies; 17+ messages in thread
From: Rafael J. Wysocki @ 2014-07-22 23:51 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday, July 22, 2014 03:43:12 PM Mark Brown wrote:
> From: Mark Brown <broonie@linaro.org>
>
> Since the OPP layer is a kernel library which has been converted to be
> directly selectable by its callers rather than user selectable and
> requiring architectures to enable it explicitly the ARCH_HAS_OPP symbol
> has become redundant and can be removed. Do so.
>
> Signed-off-by: Mark Brown <broonie@linaro.org>
> Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
> Acked-by: Nishanth Menon <nm@ti.com>
> Acked-by: Rob Herring <robh@kernel.org>
> Acked-by: Shawn Guo <shawn.guo@freescale.com>
> Acked-by: Simon Horman <horms+renesas@verge.net.au>
> ---
>
> Raphael, IIRC you said that you'd applied this but it seems not to have
> appeared in -next. There's a few more references crept in, I'll try to
> get them removed at source.
I've queued this up for 3.17, thanks!
> Documentation/power/opp.txt | 3 ---
> arch/arm/mach-exynos/Kconfig | 1 -
> arch/arm/mach-highbank/Kconfig | 1 -
> arch/arm/mach-imx/Kconfig | 1 -
> arch/arm/mach-omap2/Kconfig | 1 -
> arch/arm/mach-shmobile/Kconfig | 2 --
> arch/arm/mach-vexpress/Kconfig | 1 -
> arch/arm/mach-zynq/Kconfig | 1 -
> drivers/devfreq/Kconfig | 1 -
> kernel/power/Kconfig | 3 ---
> 10 files changed, 15 deletions(-)
>
> diff --git a/Documentation/power/opp.txt b/Documentation/power/opp.txt
> index a9adad828cdc..c6279c2be47c 100644
> --- a/Documentation/power/opp.txt
> +++ b/Documentation/power/opp.txt
> @@ -51,9 +51,6 @@ Typical usage of the OPP library is as follows:
> SoC framework -> modifies on required cases certain OPPs -> OPP layer
> -> queries to search/retrieve information ->
>
> -Architectures that provide a SoC framework for OPP should select ARCH_HAS_OPP
> -to make the OPP layer available.
> -
> OPP layer expects each domain to be represented by a unique device pointer. SoC
> framework registers a set of initial OPPs per device with the OPP layer. This
> list is expected to be an optimally small number typically around 5 per device.
> diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
> index 8f9b66c4ac78..f7889f6a1353 100644
> --- a/arch/arm/mach-exynos/Kconfig
> +++ b/arch/arm/mach-exynos/Kconfig
> @@ -100,7 +100,6 @@ config SOC_EXYNOS5440
> default y
> depends on ARCH_EXYNOS5
> select ARCH_DMA_ADDR_T_64BIT if ARM_LPAE
> - select ARCH_HAS_OPP
> select HAVE_ARM_ARCH_TIMER
> select AUTO_ZRELADDR
> select MIGHT_HAVE_PCI
> diff --git a/arch/arm/mach-highbank/Kconfig b/arch/arm/mach-highbank/Kconfig
> index a5960e2ac090..31aa866c3317 100644
> --- a/arch/arm/mach-highbank/Kconfig
> +++ b/arch/arm/mach-highbank/Kconfig
> @@ -2,7 +2,6 @@ config ARCH_HIGHBANK
> bool "Calxeda ECX-1000/2000 (Highbank/Midway)" if ARCH_MULTI_V7
> select ARCH_DMA_ADDR_T_64BIT if ARM_LPAE
> select ARCH_HAS_HOLES_MEMORYMODEL
> - select ARCH_HAS_OPP
> select ARCH_SUPPORTS_BIG_ENDIAN
> select ARM_AMBA
> select ARM_ERRATA_764369 if SMP
> diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
> index 2bc7b97861b4..9de84a215abd 100644
> --- a/arch/arm/mach-imx/Kconfig
> +++ b/arch/arm/mach-imx/Kconfig
> @@ -1,6 +1,5 @@
> menuconfig ARCH_MXC
> bool "Freescale i.MX family" if ARCH_MULTI_V4_V5 || ARCH_MULTI_V6_V7
> - select ARCH_HAS_OPP
> select ARCH_REQUIRE_GPIOLIB
> select ARM_CPU_SUSPEND if PM
> select CLKSRC_MMIO
> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
> index 1c1ed737f7ab..e7189dcc9309 100644
> --- a/arch/arm/mach-omap2/Kconfig
> +++ b/arch/arm/mach-omap2/Kconfig
> @@ -15,7 +15,6 @@ config ARCH_OMAP3
> bool "TI OMAP3"
> depends on ARCH_MULTI_V7
> select ARCH_OMAP2PLUS
> - select ARCH_HAS_OPP
> select ARM_CPU_SUSPEND if PM
> select OMAP_INTERCONNECT
> select PM_OPP if PM
> diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
> index 384221d72d22..5814754c1240 100644
> --- a/arch/arm/mach-shmobile/Kconfig
> +++ b/arch/arm/mach-shmobile/Kconfig
> @@ -92,7 +92,6 @@ config ARCH_R8A73A4
> select CPU_V7
> select SH_CLK_CPG
> select RENESAS_IRQC
> - select ARCH_HAS_OPP
> select SYS_SUPPORTS_SH_CMT
> select SYS_SUPPORTS_SH_TMU
>
> @@ -244,7 +243,6 @@ config MACH_KOELSCH
> config MACH_KZM9G
> bool "KZM-A9-GT board"
> depends on ARCH_SH73A0
> - select ARCH_HAS_OPP
> select ARCH_REQUIRE_GPIOLIB
> select REGULATOR_FIXED_VOLTAGE if REGULATOR
> select SND_SOC_AK4642 if SND_SIMPLE_CARD
> diff --git a/arch/arm/mach-vexpress/Kconfig b/arch/arm/mach-vexpress/Kconfig
> index d8b9330f896a..1af70329b88d 100644
> --- a/arch/arm/mach-vexpress/Kconfig
> +++ b/arch/arm/mach-vexpress/Kconfig
> @@ -64,7 +64,6 @@ config ARCH_VEXPRESS_DCSCB
>
> config ARCH_VEXPRESS_SPC
> bool "Versatile Express Serial Power Controller (SPC)"
> - select ARCH_HAS_OPP
> select PM_OPP
> help
> The TC2 (A15x2 A7x3) versatile express core tile integrates a logic
> diff --git a/arch/arm/mach-zynq/Kconfig b/arch/arm/mach-zynq/Kconfig
> index 0c164f81e72d..aaa5162c1509 100644
> --- a/arch/arm/mach-zynq/Kconfig
> +++ b/arch/arm/mach-zynq/Kconfig
> @@ -1,6 +1,5 @@
> config ARCH_ZYNQ
> bool "Xilinx Zynq ARM Cortex A9 Platform" if ARCH_MULTI_V7
> - select ARCH_HAS_OPP
> select ARCH_SUPPORTS_BIG_ENDIAN
> select ARM_AMBA
> select ARM_GIC
> diff --git a/drivers/devfreq/Kconfig b/drivers/devfreq/Kconfig
> index 49e74c1fc639..3dced0a9eae3 100644
> --- a/drivers/devfreq/Kconfig
> +++ b/drivers/devfreq/Kconfig
> @@ -68,7 +68,6 @@ comment "DEVFREQ Drivers"
> config ARM_EXYNOS4_BUS_DEVFREQ
> bool "ARM Exynos4210/4212/4412 Memory Bus DEVFREQ Driver"
> depends on (CPU_EXYNOS4210 || SOC_EXYNOS4212 || SOC_EXYNOS4412) && !ARCH_MULTIPLATFORM
> - select ARCH_HAS_OPP
> select DEVFREQ_GOV_SIMPLE_ONDEMAND
> select PM_OPP
> help
> diff --git a/kernel/power/Kconfig b/kernel/power/Kconfig
> index 9a83d780facd..e4e4121fa327 100644
> --- a/kernel/power/Kconfig
> +++ b/kernel/power/Kconfig
> @@ -253,9 +253,6 @@ config APM_EMULATION
> anything, try disabling/enabling this option (or disabling/enabling
> APM in your BIOS).
>
> -config ARCH_HAS_OPP
> - bool
> -
> config PM_OPP
> bool
> ---help---
>
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2014-07-22 23:51 UTC | newest]
Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-06 10:36 [PATCH] PM / OPP: Remove ARCH_HAS_OPP Mark Brown
2014-06-06 11:14 ` Simon Horman
2014-06-06 12:14 ` Magnus Damm
2014-06-06 12:50 ` Simon Horman
2014-06-06 13:01 ` Magnus Damm
2014-06-06 13:08 ` Mark Brown
2014-06-06 21:15 ` Rafael J. Wysocki
2014-06-09 19:49 ` Mark Brown
2014-06-09 22:22 ` Rafael J. Wysocki
2014-06-09 22:51 ` Mark Brown
2014-06-10 0:15 ` Rafael J. Wysocki
2014-06-06 13:45 ` Rob Herring
2014-06-06 14:33 ` Nishanth Menon
2014-06-09 1:56 ` Shawn Guo
2014-06-09 4:49 ` Viresh Kumar
2014-07-22 14:43 Mark Brown
2014-07-22 23:51 ` 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).