[PATCH] unicore32: Remove ARCH_HAS_CPUFREQ config option

[PATCH] unicore32: Remove ARCH_HAS_CPUFREQ config option

[Stephen Boyd]

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 unicore32 platforms.

Cc: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
---
 arch/unicore32/Kconfig | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/arch/unicore32/Kconfig b/arch/unicore32/Kconfig
index aafad6fa1667..928237a7b9ca 100644
--- a/arch/unicore32/Kconfig
+++ b/arch/unicore32/Kconfig
@@ -51,9 +51,6 @@ config ARCH_HAS_ILOG2_U32
 config ARCH_HAS_ILOG2_U64
 	bool
 
-config ARCH_HAS_CPUFREQ
-	bool
-
 config GENERIC_HWEIGHT
 	def_bool y
 
@@ -87,7 +84,6 @@ config ARCH_PUV3
 	select GENERIC_CLOCKEVENTS
 	select HAVE_CLK
 	select ARCH_REQUIRE_GPIOLIB
-	select ARCH_HAS_CPUFREQ
 
 # CONFIGs for ARCH_PUV3
 
@@ -198,9 +194,7 @@ menu "Power management options"
 
 source "kernel/power/Kconfig"
 
-if ARCH_HAS_CPUFREQ
 source "drivers/cpufreq/Kconfig"
-endif
 
 config ARCH_SUSPEND_POSSIBLE
 	def_bool y if !ARCH_FPGA
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation

Re: [PATCH] unicore32: Remove ARCH_HAS_CPUFREQ config option

[Viresh Kumar]

On 4 June 2014 00:00, Stephen Boyd <sboyd@codeaurora.org> 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 unicore32 platforms.
>
> Cc: Viresh Kumar <viresh.kumar@linaro.org>
> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
> ---
>  arch/unicore32/Kconfig | 6 ------
>  1 file changed, 6 deletions(-)

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

回复： [PATCH] unicore32: Remove ARCH_HAS_CPUFREQ config option

[Xuetao Guan]

----- Stephen Boyd <sboyd@codeaurora.org> 写道：
> 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 unicore32 platforms.
> 
> Cc: Viresh Kumar <viresh.kumar@linaro.org>
> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>

Thanks, I'll push it in my repo.

Signed-off-by: Xuetao Guan <gxt@mprc.pku.edu.cn>

> ---
>  arch/unicore32/Kconfig | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/arch/unicore32/Kconfig b/arch/unicore32/Kconfig
> index aafad6fa1667..928237a7b9ca 100644
> --- a/arch/unicore32/Kconfig
> +++ b/arch/unicore32/Kconfig
> @@ -51,9 +51,6 @@ config ARCH_HAS_ILOG2_U32
>  config ARCH_HAS_ILOG2_U64
>  	bool
>  
> -config ARCH_HAS_CPUFREQ
> -	bool
> -
>  config GENERIC_HWEIGHT
>  	def_bool y
>  
> @@ -87,7 +84,6 @@ config ARCH_PUV3
>  	select GENERIC_CLOCKEVENTS
>  	select HAVE_CLK
>  	select ARCH_REQUIRE_GPIOLIB
> -	select ARCH_HAS_CPUFREQ
>  
>  # CONFIGs for ARCH_PUV3
>  
> @@ -198,9 +194,7 @@ menu "Power management options"
>  
>  source "kernel/power/Kconfig"
>  
> -if ARCH_HAS_CPUFREQ
>  source "drivers/cpufreq/Kconfig"
> -endif
>  
>  config ARCH_SUSPEND_POSSIBLE
>  	def_bool y if !ARCH_FPGA
> -- 
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> hosted by The Linux Foundation
>