Skip to content

clockevents: Add helpers to verify state of a clockevent device

Viresh Kumar lkml 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v2 Latest2 patches 0 repliesNew

Changes in v2 from the cover letter

- Don't move 'enum clock_event_state' to core's internal file.
- Move accessor functions to clockchips.h and mark them inline.
- New patch, 2/2.
- s/\<evt\>/dev in accessor functions

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2clockevents: Add helpers to verify state of a clockevent deviceinclude/linux/clockchips.h+26 -00 replies
2/2clockevents: Use helpers to verify state of a clockevent devicekernel/time/clockevents.c kernel/time/tick-broadcast.c kernel/time/tick-common.c +1 more+17 -170 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [PATCH V2 0/2] clockevents: Add helpers to verify state of a clockevent device

Hi Thomas,

Some clockevent drivers, once migrated to use per-state callbacks, would need to verify 'state' of the clockevent device in their callbacks or interrupt handler.

(Details of these drivers are posted by Thomas earlier:
https://lkml.org/lkml/2015/5/20/454)

In order to discourage drivers from using CLK_EVT_STATE_* symbols directly and to make it easy grep for abusers of these symbols, this series introduces few accessor functions. The second patch updates core code to use these functions.

V1->V2:
- Don't move 'enum clock_event_state' to core's internal file.
- Move accessor functions to clockchips.h and mark them inline.
- New patch, 2/2.
- s/\<evt\>/dev in accessor functions

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

--
viresh

Viresh Kumar (2):
clockevents: Add helpers to verify state of a clockevent device
clockevents: Use helpers to verify state of a clockevent device

 include/linux/clockchips.h   | 26 ++++++++++++++++++++++++++
 kernel/time/clockevents.c    | 24 ++++++++++++------------
 kernel/time/tick-broadcast.c |  6 +++---
 kernel/time/tick-common.c    |  2 +-
 kernel/time/tick-oneshot.c   |  2 +-
 5 files changed, 43 insertions(+), 17 deletions(-)
Signature · 1 line
2.4.0