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

Groups > linux.kernel > #1261671

Re: [PATCH 20/22] clocksource/drivers/exynos_mct: Fix Kconfig and add COMPILE_TEST option

Show all headers | View raw

On Tue, Nov 3, 2015 at 2:02 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> I suspect this will come up again in the future. The problem is
> really that drivers/clocksource has different rules from almost
> everything else, by requiring the platform to 'select' the driver.
>
> The second version that Krzysztof posted is how we handle this in
> other driver subsystems, and I would generally prefer it to do this
> consistently for everything, but John Stultz has in the past argued
> strongly for using 'select' in all clocksource drivers. The reason
> is that for each platform we know in advance which driver we want,
> and there is never a need for the user to have to select the right
> one.

And just to clarify, I don't necessarily think "select" is the right
method, as creating an option that defaults to Y if the right
architecture/platform support is present is fine too.

I just don't want users to have to search deeply through menuconfig to
find a clocksource driver checkbox when they have already selected a
platform and provided enough information for us to know which
clocksource driver is needed.  So my argument its really all about
avoiding user-prompts in kconfig for clocksources.  It is conceptually
easier to do this w/ select, but you can also do it via default y and
proper dependencies.

thanks
-john
--
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/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 01/22] clocksource/drivers/pistachio: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:00 +0100
  [PATCH 02/22] clocksource/drivers/mediatek: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:00 +0100
  [PATCH 12/22] clocksource/drivers/tegra2: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:00 +0100
  [PATCH 20/22] clocksource/drivers/exynos_mct: Fix Kconfig and add COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:00 +0100
    Re: [PATCH 20/22] clocksource/drivers/exynos_mct: Fix Kconfig and add  COMPILE_TEST option Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-11-03 01:40 +0100
      Re: [PATCH 20/22] clocksource/drivers/exynos_mct: Fix Kconfig and add  COMPILE_TEST option Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-11-03 02:00 +0100
        Re: [PATCH 20/22] clocksource/drivers/exynos_mct: Fix Kconfig and add  COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-03 09:50 +0100
          Re: [PATCH 20/22] clocksource/drivers/exynos_mct: Fix Kconfig and add COMPILE_TEST option Arnd Bergmann <arnd@arndb.de> - 2015-11-03 11:10 +0100
            Re: [PATCH 20/22] clocksource/drivers/exynos_mct: Fix Kconfig and add  COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-03 12:10 +0100
            Re: [PATCH 20/22] clocksource/drivers/exynos_mct: Fix Kconfig and add  COMPILE_TEST option Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-11-03 13:10 +0100
              Re: [PATCH 20/22] clocksource/drivers/exynos_mct: Fix Kconfig and add  COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-03 13:10 +0100
            Re: [PATCH 20/22] clocksource/drivers/exynos_mct: Fix Kconfig and add  COMPILE_TEST option John Stultz <john.stultz@linaro.org> - 2015-11-03 17:10 +0100
    Re: [PATCH 20/22] clocksource/drivers/exynos_mct: Fix Kconfig and add  COMPILE_TEST option Chanwoo Choi <cw00.choi@samsung.com> - 2015-11-03 02:00 +0100
  [PATCH 13/22] clocksource/drivers/vt8500: Remove unneeded header Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:00 +0100
  [PATCH 06/22] clocksource/drivers/meson6: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:00 +0100
  [PATCH 19/22] clocksource/drivers/prcmu: Fix Kconfig and add COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:00 +0100
    Re: [PATCH 19/22] clocksource/drivers/prcmu: Fix Kconfig and add  COMPILE_TEST option Linus Walleij <linus.walleij@linaro.org> - 2015-11-02 15:50 +0100
      Re: [PATCH 19/22] clocksource/drivers/prcmu: Fix Kconfig and add  COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 16:00 +0100
  [PATCH 08/22] clocksource/drivers/digicolor: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:10 +0100
  [PATCH 07/22] clocksource/drivers/orion: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:10 +0100
  [PATCH 15/22] clocksource/drivers/cadence_ttc: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:10 +0100
  [PATCH 17/22] clocksource/drivers/lpc32xx: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:10 +0100
  [PATCH 11/22] clocksource/drivers/sun5i: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:10 +0100
  [PATCH 22/22] clocksource/drivers/fsl-ftm: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:10 +0100
  [PATCH 16/22] clocksource/drivers/asm9260: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:10 +0100
  [PATCH 18/22] clocksource/drivers/nomadik_mtu: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:10 +0100
  [PATCH 04/22] clocksource/drivers/rockchip: Add COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:10 +0100
  [PATCH 03/22] clocksource/drivers/rockchip: Make the driver more compatible Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:10 +0100
    Re: [PATCH 03/22] clocksource/drivers/rockchip: Make the driver more compatible Arnd Bergmann <arnd@arndb.de> - 2015-11-02 16:40 +0100
      Re: [PATCH 03/22] clocksource/drivers/rockchip: Make the driver more  compatible Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 17:40 +0100
        Re: [PATCH 03/22] clocksource/drivers/rockchip: Make the driver more compatible Arnd Bergmann <arnd@arndb.de> - 2015-11-02 22:50 +0100
  [PATCH 14/22] clocksource/drivers/vt8500: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:10 +0100
  [PATCH 09/22] clocksource/drivers/dw_apb: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:10 +0100
  [PATCH 21/22] clocksource/drivers/samsung-pwm: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-02 14:10 +0100

csiph-web