Messages in this thread | | | Date | Mon, 12 Aug 2013 14:55:31 +0200 | From | Daniel Lezcano <> | Subject | Re: [PATCH] clocksource: add an "menu" item to Kconfig |
| |
On 08/07/2013 06:58 PM, John Stultz wrote: > On 08/07/2013 09:30 AM, Sebastian Andrzej Siewior wrote: >> On 08/07/2013 06:21 PM, John Stultz wrote: >>> 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? >> There was one visible Option if I remember correctly. I added two and >> was wondering once I found them. I do have a PCI card which provides a >> clock source and clock events device. So I assumed select <m|y> would >> be doable. > > PCI card, eh? (Heh. In writing my previous mail, I think I even wrote > "These are clocksources soldered to the motherboard, not pci cards!", > but deleted it as it didn't seem useful. :) > > So I think that's actually a first for clocksource drivers. > > So, if there are genuinely cross-architecture pluggable clocksources, > that aren't part of a platform definition, I would be ok with making an > exception. But I'd want to make sure that we kept a high-bar for adding > clocksource options, so that folks don't just clutter up the Kconfig menu. > > That said, Daniel (now cc'ed - sorry for not cc'ing you earlier!) is now > in charge of the clocksource directory, so my objections and wishes > aren't as important as his.
Sebastian,
instead of adding a menu item in the Kconfig, is it possible to have this Kconfig PCI board support to select the right driver ?
Thanks -- Daniel
-- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |