Re: [PATCH] clocksource: add an "menu" item to Kconfig

[John Stultz]

On 08/07/2013 03:56 AM, Sebastian Andrzej Siewior wrote:
> This patch adds a menu item to clocksource' Kconfig file. Without it the
> changeable items look somehow lost in the main drivers' menu

So I'd like to push back on this a bit and ask why we need user-visible 
clocksource config options?

I'm much prefer the platform options select the needed clocksources, and 
not leave it up to the user (who really doesn't want to go digging 
around to find the clocksource sub-menu under drivers, and scan a list 
of 100 options that aren't possible on their hardware).

For an example of what I consider bad behavior here, see the RTC driver 
directory - at a certain point they just become hashes: "Oh no.. you 
selected rtc-387aae6fd, on your hardware, you need rtc-adfb8e5".

It really just seems like it forces more unnecessary options on the user 
and more ways to build a kernel that won't boot.

I do get you're trying to fix up the bad situation, caused by 
user-prompts slipping into the Kconfig.

But I might prefer a patch closer to whats below.

thanks
-john


diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig
index b7b9b04..b3050db 100644
--- a/drivers/clocksource/Kconfig
+++ b/drivers/clocksource/Kconfig
@@ -57,14 +57,14 @@ config CLKSRC_NOMADIK_MTU_SCHED_CLOCK
 	  Use the Multi Timer Unit as the sched_clock.
 
 config CLKSRC_DBX500_PRCMU
-	bool "Clocksource PRCMU Timer"
+	bool
 	depends on UX500_SOC_DB8500
 	default y
 	help
 	  Use the always on PRCMU Timer as clocksource
 
 config CLKSRC_DBX500_PRCMU_SCHED_CLOCK
-	bool "Clocksource PRCMU Timer sched_clock"
+	bool
 	depends on (CLKSRC_DBX500_PRCMU && !CLKSRC_NOMADIK_MTU_SCHED_CLOCK)
 	default y
 	help


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/