[PATCH] ARM: ux500: don't select CPUFREQ_DT

[PATCH] ARM: ux500: don't select CPUFREQ_DT

[Arnd Bergmann]

The new 'select CPUFREQ_DT' statement causes a warning in some
configurations:

warning: (ARCH_U8500) selects CPUFREQ_DT which has unmet direct dependencies (CPU_FREQ && HAVE_CLK && OF && (!CPU_THERMAL || THERMAL))

No other platform does this, so I think for consistency it's
best if we remove it again from ux500 and instead add the driver
to the defconfig.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 arch/arm/mach-ux500/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/mach-ux500/Kconfig b/arch/arm/mach-ux500/Kconfig
index 2b56e4af5a27..4740ac393297 100644
--- a/arch/arm/mach-ux500/Kconfig
+++ b/arch/arm/mach-ux500/Kconfig
@@ -9,7 +9,6 @@ menuconfig ARCH_U8500
 	select ARM_GIC
 	select CACHE_L2X0
 	select CLKSRC_NOMADIK_MTU
-	select CPUFREQ_DT
 	select GPIOLIB
 	select HAVE_ARM_SCU if SMP
 	select HAVE_ARM_TWD if SMP
-- 
2.9.0

Re: [PATCH] ARM: ux500: don't select CPUFREQ_DT

[Linus Walleij]

On Wed, Aug 23, 2017 at 3:27 PM, Arnd Bergmann <arnd@arndb.de> wrote:

> The new 'select CPUFREQ_DT' statement causes a warning in some
> configurations:
>
> warning: (ARCH_U8500) selects CPUFREQ_DT which has unmet direct dependencies (CPU_FREQ && HAVE_CLK && OF && (!CPU_THERMAL || THERMAL))
>
> No other platform does this, so I think for consistency it's
> best if we remove it again from ux500 and instead add the driver
> to the defconfig.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

OK then. I should update the defconfig anyway.
Acked-by: Linus Walleij <linus.walleij@linaro.org>

Rafael, can you stick this on top of the other patches on
your cpufreq branch?

Yours,
Linus Walleij

Re: [PATCH] ARM: ux500: don't select CPUFREQ_DT

[Rafael J. Wysocki]

On Thursday, August 24, 2017 9:47:52 PM CEST Linus Walleij wrote:
> On Wed, Aug 23, 2017 at 3:27 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> 
> > The new 'select CPUFREQ_DT' statement causes a warning in some
> > configurations:
> >
> > warning: (ARCH_U8500) selects CPUFREQ_DT which has unmet direct dependencies (CPU_FREQ && HAVE_CLK && OF && (!CPU_THERMAL || THERMAL))
> >
> > No other platform does this, so I think for consistency it's
> > best if we remove it again from ux500 and instead add the driver
> > to the defconfig.
> >
> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> 
> OK then. I should update the defconfig anyway.
> Acked-by: Linus Walleij <linus.walleij@linaro.org>
> 
> Rafael, can you stick this on top of the other patches on
> your cpufreq branch?

Done locally, but will show up in linux-next next week.

Though honestly, that not being in linux-pm Patchwork doesn't make things
particularly easy for me ...

Thanks,
Rafael