Re: [PATCH v2 04/18] clocksource: Add ARM System timer driver

From: Paul Bolle
Date: Tue Mar 03 2015 - 14:43:19 EST


Maxime Coquelin schreef op ma 02-03-2015 om 17:53 [+0100]:
> Do you agree if I define it like this:
>
> config ARMV7M_SYSTICK
> bool "Clocksource driver for ARMv7-M System timer"
> depends on OF && (CPU_V7M || COMPILE_TEST)
> select CLKSRC_OF
> select CLKSRC_MMIO
> help
> This options enables clocksource support for the ARMv7-M system
> timer unit.

I don't really have strong feelings on whatever way you choose to fix
the, well, minor problem I pointed out.

Having said that, if a Kconfig entry without a prompt (and therefor,
without help) actually does what you want it to do, why bother adding a
prompt and a one line help text?


Paul Bolle

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