Re: [PATCH] ARM: Remove ARCH_HAS_CPUFREQ config option

[Sachin Kamat]

On Tue, Jun 3, 2014 at 3:37 AM, Stephen Boyd <sboyd@xxxxxxxxxxxxxx> wrote:
> This config exists entirely to hide the cpufreq menu from the
> kernel configuration unless a platform has selected it. Nothing
> is actually built if this config is 'Y' and it just leads to
> more patches that add a select under a platform Kconfig so that
> some other CPUfreq option can be chosen. Let's remove the
> option so that we can always enable CPUfreq drivers on ARM
> platforms.
>
> Signed-off-by: Stephen Boyd <sboyd@xxxxxxxxxxxxxx>
> ---
>  arch/arm/Kconfig               | 18 ------------------
>  arch/arm/mach-davinci/Kconfig  |  1 -
>  arch/arm/mach-highbank/Kconfig |  1 -
>  arch/arm/mach-imx/Kconfig      |  3 ---
>  arch/arm/mach-mvebu/Kconfig    |  1 -
>  arch/arm/mach-omap2/Kconfig    |  1 -
>  arch/arm/mach-shmobile/Kconfig |  2 --
>  arch/arm/mach-spear/Kconfig    |  1 -
>  arch/arm/mach-tegra/Kconfig    |  1 -
>  arch/arm/mach-ux500/Kconfig    |  1 -
>  arch/arm/mach-vexpress/Kconfig |  1 -
>  arch/arm/mach-vt8500/Kconfig   |  1 -
>  arch/arm/mach-zynq/Kconfig     |  1 -
>  13 files changed, 33 deletions(-)
>
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 5e91e40c1d22..3fdae40d5114 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -179,13 +179,6 @@ config ARCH_HAS_ILOG2_U32
>  config ARCH_HAS_ILOG2_U64
>         bool
>
> -config ARCH_HAS_CPUFREQ
> -       bool
> -       help
> -         Internal node to signify that the ARCH has CPUFREQ support
> -         and that the relevant menu configurations are displayed for
> -         it.
> -
>  config ARCH_HAS_BANDGAP
>         bool
>
> @@ -321,7 +314,6 @@ config ARCH_MULTIPLATFORM
>
>  config ARCH_INTEGRATOR
>         bool "ARM Ltd. Integrator family"
> -       select ARCH_HAS_CPUFREQ
>         select ARM_AMBA
>         select ARM_PATCH_PHYS_VIRT
>         select AUTO_ZRELADDR
> @@ -540,7 +532,6 @@ config ARCH_DOVE
>
>  config ARCH_KIRKWOOD
>         bool "Marvell Kirkwood"
> -       select ARCH_HAS_CPUFREQ
>         select ARCH_REQUIRE_GPIOLIB
>         select CPU_FEROCEON
>         select GENERIC_CLOCKEVENTS
> @@ -639,7 +630,6 @@ config ARCH_LPC32XX
>  config ARCH_PXA
>         bool "PXA2xx/PXA3xx-based"
>         depends on MMU
> -       select ARCH_HAS_CPUFREQ
>         select ARCH_MTD_XIP
>         select ARCH_REQUIRE_GPIOLIB
>         select ARM_CPU_SUSPEND if PM
> @@ -709,7 +699,6 @@ config ARCH_RPC
>
>  config ARCH_SA1100
>         bool "SA1100-based"
> -       select ARCH_HAS_CPUFREQ
>         select ARCH_MTD_XIP
>         select ARCH_REQUIRE_GPIOLIB
>         select ARCH_SPARSEMEM_ENABLE
> @@ -727,7 +716,6 @@ config ARCH_SA1100
>
>  config ARCH_S3C24XX
>         bool "Samsung S3C24XX SoCs"
> -       select ARCH_HAS_CPUFREQ
>         select ARCH_REQUIRE_GPIOLIB
>         select ATAGS
>         select CLKDEV_LOOKUP
> @@ -748,7 +736,6 @@ config ARCH_S3C24XX
>
>  config ARCH_S3C64XX
>         bool "Samsung S3C64XX"
> -       select ARCH_HAS_CPUFREQ
>         select ARCH_REQUIRE_GPIOLIB
>         select ARM_AMBA
>         select ARM_VIC
> @@ -811,7 +798,6 @@ config ARCH_S5PC100
>
>  config ARCH_S5PV210
>         bool "Samsung S5PV210/S5PC110"
> -       select ARCH_HAS_CPUFREQ
>         select ARCH_HAS_HOLES_MEMORYMODEL
>         select ARCH_SPARSEMEM_ENABLE
>         select ATAGS
> @@ -831,7 +817,6 @@ config ARCH_S5PV210
>
>  config ARCH_EXYNOS
>         bool "Samsung EXYNOS"
> -       select ARCH_HAS_CPUFREQ
>         select ARCH_HAS_HOLES_MEMORYMODEL
>         select ARCH_REQUIRE_GPIOLIB
>         select ARCH_SPARSEMEM_ENABLE

I think this patch is not based on the latest code base as ARCH_EXYNOS
symbol is now defined in arch/arm/mach-exynos/Kconfig file.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/