Skip to contentSkip to conversation

clocksource: ixp4xx: select TIMER_OF when needed

PATCH Applied Arnd Bergmann 3 participants 3 messages Jan 3, 2021 – Feb 22, 2021

Subsystems (4)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 clocksource: ixp4xx: select TIMER_OF when needed

From: Arnd Bergmann <[email protected]>

Compile-testing the ixp4xx timer with CONFIG_OF enabled but CONFIG_TIMER_OF disabled leads to a harmless warning:

arm-linux-gnueabi-ld: warning: orphan section __timer_of_table' from drivers/clocksource/timer-ixp4xx.o' being placed in section `__timer_of_table'

Move the select statement from the platform code into the driver so it always gets enabled in configurations that rely on it.

Fixes 40df14cc5cc0 ("clocksource/drivers/ixp4xx: Add OF initialization support")
---
 arch/arm/mach-ixp4xx/Kconfig | 1 -
 drivers/clocksource/Kconfig  | 1 +
 2 files changed, 1 insertion(+), 1 deletion(-)
arch/arm/mach-ixp4xx/Kconfig +0 -1
index f7211b57b1e7..165c184801e1 100644
@@ -13,7 +13,6 @@ config MACH_IXP4XX_OF
 	select I2C
 	select I2C_IOP3XX
 	select PCI
-	select TIMER_OF
 	select USE_OF
 	help
 	  Say 'Y' here to support Device Tree-based IXP4xx platforms.
drivers/clocksource/Kconfig +1 -0
index 14c7c4712478..66be9ea69e33 100644
@@ -79,6 +79,7 @@ config IXP4XX_TIMER
 	bool "Intel XScale IXP4xx timer driver" if COMPILE_TEST
 	depends on HAS_IOMEM
 	select CLKSRC_MMIO
+	select TIMER_OF if OF
 	help
 	  Enables support for the Intel XScale IXP4xx SoC timer.
 
Signature · 1 line
2.29.2
Reviewed-by on v1 1/1
#1 lore raw
Quoted 5 lines · Arnd Bergmann

From: Arnd Bergmann <[email protected]>

Compile-testing the ixp4xx timer with CONFIG_OF enabled but CONFIG_TIMER_OF disabled leads to a harmless warning:

Quoted 7 lines

arm-linux-gnueabi-ld: warning: orphan section __timer_of_table' from drivers/clocksource/timer-ixp4xx.o' being placed in section `__timer_of_table'

Move the select statement from the platform code into the driver so it always gets enabled in configurations that rely on it.

Fixes 40df14cc5cc0 ("clocksource/drivers/ixp4xx: Add OF initialization support")

That's a nice fix!

Yours,
Linus Walleij

merge bot Applied
#2 lore raw

[tip: timers/urgent] clocksource/drivers/ixp4xx: Select TIMER_OF when needed

In reply to Arnd Bergmann
The following commit has been merged into the timers/urgent branch of tip:

The following commit has been merged into the timers/urgent branch of tip:

Commit-ID:     7a3b8758bd6e45f7b671723b5c9fa2b69d0787ae
Gitweb:        https://git.kernel.org/tip/7a3b8758bd6e45f7b671723b5c9fa2b69d0787ae
Author:        Arnd Bergmann <[email protected]>
AuthorDate:    Sun, 03 Jan 2021 14:59:24 +01:00
Committer:     Daniel Lezcano <[email protected]>
CommitterDate: Mon, 18 Jan 2021 16:20:15 +01:00

clocksource/drivers/ixp4xx: Select TIMER_OF when needed

Compile-testing the ixp4xx timer with CONFIG_OF enabled but CONFIG_TIMER_OF disabled leads to a harmless warning:

arm-linux-gnueabi-ld: warning: orphan section __timer_of_table' from drivers/clocksource/timer-ixp4xx.o' being placed in section `__timer_of_table'

Move the select statement from the platform code into the driver so it always gets enabled in configurations that rely on it.

Fixes 40df14cc5cc0 ("clocksource/drivers/ixp4xx: Add OF initialization support")
---
 arch/arm/mach-ixp4xx/Kconfig | 1 -
 drivers/clocksource/Kconfig  | 1 +
 2 files changed, 1 insertion(+), 1 deletion(-)
arch/arm/mach-ixp4xx/Kconfig +0 -1
index f7211b5..165c184 100644
@@ -13,7 +13,6 @@ config MACH_IXP4XX_OF
 	select I2C
 	select I2C_IOP3XX
 	select PCI
-	select TIMER_OF
 	select USE_OF
 	help
 	  Say 'Y' here to support Device Tree-based IXP4xx platforms.
drivers/clocksource/Kconfig +1 -0
index 14c7c47..66be9ea 100644
@@ -79,6 +79,7 @@ config IXP4XX_TIMER
 	bool "Intel XScale IXP4xx timer driver" if COMPILE_TEST
 	depends on HAS_IOMEM
 	select CLKSRC_MMIO
+	select TIMER_OF if OF
 	help
 	  Enables support for the Intel XScale IXP4xx SoC timer.