diff options
author | Alexandre Belloni <alexandre.belloni@free-electrons.com> | 2016-09-09 13:13:50 +0200 |
---|---|---|
committer | Daniel Lezcano <daniel.lezcano@linaro.org> | 2016-09-12 07:28:46 +0200 |
commit | 2783e5d63a9a38b5246c75b29f0dce4cf995a1b3 (patch) | |
tree | d39adddb8528ec51567627af05ea00b977144de0 /drivers/clocksource/timer-atmel-pit.c | |
parent | clocksource/drivers/timer-atmel-pit: Remove uselesss WARN_ON_ONCE (diff) | |
download | linux-2783e5d63a9a38b5246c75b29f0dce4cf995a1b3.tar.xz linux-2783e5d63a9a38b5246c75b29f0dce4cf995a1b3.zip |
clocksource/drivers/timer-atmel-pit: Simplify IRQ handler
Because the PIT is also a proper clocksource, the timekeeping code is
already able to handle lost ticks.
Reported-by: Thomas Gleixner <tglx@linutronix.de>
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Diffstat (limited to 'drivers/clocksource/timer-atmel-pit.c')
-rw-r--r-- | drivers/clocksource/timer-atmel-pit.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/drivers/clocksource/timer-atmel-pit.c b/drivers/clocksource/timer-atmel-pit.c index 91cf04704ed1..6555821bbdae 100644 --- a/drivers/clocksource/timer-atmel-pit.c +++ b/drivers/clocksource/timer-atmel-pit.c @@ -152,15 +152,10 @@ static irqreturn_t at91sam926x_pit_interrupt(int irq, void *dev_id) /* The PIT interrupt may be disabled, and is shared */ if (clockevent_state_periodic(&data->clkevt) && (pit_read(data->base, AT91_PIT_SR) & AT91_PIT_PITS)) { - unsigned nr_ticks; - /* Get number of ticks performed before irq, and ack it */ - nr_ticks = PIT_PICNT(pit_read(data->base, AT91_PIT_PIVR)); - do { - data->cnt += data->cycle; - data->clkevt.event_handler(&data->clkevt); - nr_ticks--; - } while (nr_ticks); + data->cnt += data->cycle * PIT_PICNT(pit_read(data->base, + AT91_PIT_PIVR)); + data->clkevt.event_handler(&data->clkevt); return IRQ_HANDLED; } |