[PATCH 0/2] clockevents: Hide CLOCK_EVT_STATE_* from rest of the kernel

From: Viresh Kumar
Date: Wed May 20 2015 - 05:11:19 EST


Hi Thomas,

Ingo suggested [1] to keep CLOCK_EVT_STATE_* symbols somewhere in
kernel/time/. We couldn't do it as bL_switcher code was using it
earlier. But that's fixed now. And so the first patch moves these
symbols to tick-internal.h.

Some of the drivers [2] need to verify state of the clockevent device
from their callbacks or interrupt handlers.

Because these symbols (defined by 'enum clock_event_state') will now be
internal to the core, we need some helpers to verify state of a
clockevent device.

One way out was to maintain the state in drivers as well, but that would
be unnecessary burden on them. And so the second patch introduces
helpers for these states.

Rebased-over: tip/timers/core (dependency on 8fff52fd5093 ("clockevents:
Introduce CLOCK_EVT_STATE_ONESHOT_STOPPED state"))

--
viresh

[1] https://lists.linaro.org/pipermail/linaro-kernel/2015-February/020292.html
[2] http://pastebin.com/374X18mv

Viresh Kumar (2):
clockevents: Move 'enum clock_event_state' to tick-internal.h
clockevents: Add helpers to verify state of a clockevent device

include/linux/clockchips.h | 34 ++++++++++++----------------------
kernel/time/clockevents.c | 31 +++++++++++++++++++++++++++++++
kernel/time/tick-internal.h | 21 +++++++++++++++++++++
3 files changed, 64 insertions(+), 22 deletions(-)

--
2.4.0

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