diff mbox

ARM: ux500: don't select CPUFREQ_DT

Message ID 20170823132733.619811-1-arnd@arndb.de
State Accepted
Commit ec4259aa4497d61d7dc168e0c303971b1fc6747d
Headers show

Commit Message

Arnd Bergmann Aug. 23, 2017, 1:27 p.m. UTC
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(-)

-- 
2.9.0

Comments

Linus Walleij Aug. 24, 2017, 7:47 p.m. UTC | #1
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
Rafael J. Wysocki Aug. 24, 2017, 11:28 p.m. UTC | #2
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
diff mbox

Patch

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