[CPUFREQ] Fix exposure of ARM_EXYNOS4210_CPUFREQ
Russell King - ARM Linux
linux at arm.linux.org.uk
Thu Feb 16 11:02:50 EST 2012
On Thu, Feb 16, 2012 at 09:17:30PM +0900, Kukjin Kim wrote:
> On 02/13/12 18:49, Russell King - ARM Linux wrote:
>> From: Russell King<rmk+kernel at arm.linux.org.uk>
>>
>> exynos4210-cpufreq.c is not buildable on non-exynos builds, so it's
>> pointless allowing this option to be exposed. Fix this by adding a
>> dependency on ARCH_EXYNOS.
>>
>> drivers/cpufreq/exynos4210-cpufreq.c:20:29: error: mach/regs-clock.h: No such file or directory
>> drivers/cpufreq/exynos4210-cpufreq.c:21:26: error: mach/cpufreq.h: No such file or directory
>>
>> Signed-off-by: Russell King<rmk+kernel at arm.linux.org.uk>
>> Cc: Dave Jones<davej at redhat.com>
>> Cc: cpufreq at vger.kernel.org
>> Cc: Kukjin Kim<kgene.kim at samsung.com>
>> ---
>> drivers/cpufreq/Kconfig.arm | 1 +
>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/cpufreq/Kconfig.arm b/drivers/cpufreq/Kconfig.arm
>> index e0664fe..c8bde43 100644
>> --- a/drivers/cpufreq/Kconfig.arm
>> +++ b/drivers/cpufreq/Kconfig.arm
>> @@ -34,6 +34,7 @@ config ARM_EXYNOS_CPUFREQ
>>
>> config ARM_EXYNOS4210_CPUFREQ
>> bool "Samsung EXYNOS4210"
>> + depends on ARCH_EXYNOS
>> help
>> This adds the CPUFreq driver for Samsung EXYNOS4210
>> SoC (S5PV310 or S5PC210).
>>
>
> Yes, you're right. Should be added it.
> BTW, how about following?
>
> diff --git a/drivers/cpufreq/Kconfig.arm b/drivers/cpufreq/Kconfig.arm
> index e0664fe..a5e0487 100644
> --- a/drivers/cpufreq/Kconfig.arm
> +++ b/drivers/cpufreq/Kconfig.arm
> @@ -33,7 +33,7 @@ config ARM_EXYNOS_CPUFREQ
> If in doubt, say N.
>
> config ARM_EXYNOS4210_CPUFREQ
> - bool "Samsung EXYNOS4210"
> + bool
> help
> This adds the CPUFreq driver for Samsung EXYNOS4210
> SoC (S5PV310 or S5PC210).
>
> I think, when selecting "ARM_EXYNOS_CPUFREQ", "ARM_EXYNOS4210_CPUFREQ"
> will be selected on EXYNOS4210 with above changes.
I don't think so. If you select this symbol you'll get a complaint
that CPU_FREQ is not selected. What's probably much better is:
config ARM_EXYNOS4210_CPUFREQ
def_bool ARCH_EXYNOS
because that ensures that if the CPU_FREQ dependencies change you don't
have to update numerous places for that too.
More information about the linux-arm-kernel
mailing list