Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1718346 > unrolled thread

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

Started byArnd Bergmann <arnd@arndb.de>
First post2017-08-23 15:30 +0200
Last post2017-08-25 01:40 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] ARM: ux500: don't select CPUFREQ_DT Arnd Bergmann <arnd@arndb.de> - 2017-08-23 15:30 +0200
    Re: [PATCH] ARM: ux500: don't select CPUFREQ_DT Linus Walleij <linus.walleij@linaro.org> - 2017-08-24 21:50 +0200
      Re: [PATCH] ARM: ux500: don't select CPUFREQ_DT "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-08-25 01:40 +0200

#1718346 — [PATCH] ARM: ux500: don't select CPUFREQ_DT

FromArnd Bergmann <arnd@arndb.de>
Date2017-08-23 15:30 +0200
Subject[PATCH] ARM: ux500: don't select CPUFREQ_DT
Message-ID<uhDTk-14P-11@gated-at.bofh.it>
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

[toc] | [next] | [standalone]


#1719528

FromLinus Walleij <linus.walleij@linaro.org>
Date2017-08-24 21:50 +0200
Message-ID<ui6iB-2kS-7@gated-at.bofh.it>
In reply to#1718346
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

[toc] | [prev] | [next] | [standalone]


#1719647

From"Rafael J. Wysocki" <rjw@rjwysocki.net>
Date2017-08-25 01:40 +0200
Message-ID<ui9Tb-4MH-9@gated-at.bofh.it>
In reply to#1719528
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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web