Merge branch 'timers-core-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip

Pull timer updates from Thomas Gleixner:
 "A rather smalish set of updates for timers and timekeeping:

   - Two core fixes to prevent potential undefinded behaviour about
     which gcc is complaining rightfully.

   - A fix to prevent stopping the tick on an (soon) offline CPU so it
     can complete the shutdown procedure.

   - Wait for clocks to stabilize before making decisions, so a not yet
     validated clock is not rejected.

   - The usual pile of fixes to the various clocksource drivers.

   - Core code typo and include fixlets"

* 'timers-core-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
  timekeeping: Include the correct header for errno definitions
  clocksource/drivers/ti-32k: Prevent ftrace recursion
  clocksource/mips-gic-timer: Stop checking cpu_has_counter
  clocksource/mips-gic-timer: Print an error if IRQ setup fails
  tick/nohz: Prevent stopping the tick on an offline CPU
  clocksource/drivers/oxnas: Add OX820 compatible
  clocksource/drivers/timer-atmel-pit: Simplify IRQ handler
  clocksource/drivers/timer-atmel-pit: Remove uselesss WARN_ON_ONCE
  clocksource/drivers/timer-atmel-pit: Drop at91sam926x_pit_common_init
  clocksource/drivers/moxart: Replace panic by pr_err
  clocksource/drivers/moxart: Replace setup_irq by request_irq
  clocksource/drivers/moxart: Add Aspeed support
  clocksource/drivers/moxart: Use struct to hold state
  clocksource/drivers/moxart: Refactor enable/disable
  time: Avoid undefined behaviour in ktime_add_safe()
  time: Avoid undefined behaviour in timespec64_add_safe()
  timekeeping: Prints the amounts of time spent during suspend
  clocksource: Defer override invalidation unless clock is unstable
  hrtimer: Spelling fixes

1 files changed, 37 insertions, 59 deletions

diff --git a/drivers/clocksource/timer-atmel-pit.c b/drivers/clocksource/timer-atmel-pit.c
index 7f0f5b26d8c5..6555821bbdae 100644
--- a/drivers/clocksource/timer-atmel-pit.c
+++ b/drivers/clocksource/timer-atmel-pit.c
@@ -149,24 +149,13 @@ static irqreturn_t at91sam926x_pit_interrupt(int irq, void *dev_id)
 {
         struct pit_data *data = dev_id;
 
-        /*
-         * irqs should be disabled here, but as the irq is shared they are only
-         * guaranteed to be off if the timer irq is registered first.
-         */
-        WARN_ON_ONCE(!irqs_disabled());
-
         /* 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;
         }
@@ -177,11 +166,41 @@ static irqreturn_t at91sam926x_pit_interrupt(int irq, void *dev_id)
 /*
  * Set up both clocksource and clockevent support.
  */
-static int __init at91sam926x_pit_common_init(struct pit_data *data)
+static int __init at91sam926x_pit_dt_init(struct device_node *node)
 {
-        unsigned long   pit_rate;
-        unsigned        bits;
-        int             ret;
+        unsigned long   pit_rate;
+        unsigned        bits;
+        int             ret;
+        struct pit_data *data;
+
+        data = kzalloc(sizeof(*data), GFP_KERNEL);
+        if (!data)
+                return -ENOMEM;
+
+        data->base = of_iomap(node, 0);
+        if (!data->base) {
+                pr_err("Could not map PIT address\n");
+                return -ENXIO;
+        }
+
+        data->mck = of_clk_get(node, 0);
+        if (IS_ERR(data->mck)) {
+                pr_err("Unable to get mck clk\n");
+                return PTR_ERR(data->mck);
+        }
+
+        ret = clk_prepare_enable(data->mck);
+        if (ret) {
+                pr_err("Unable to enable mck\n");
+                return ret;
+        }
+
+        /* Get the interrupts property */
+        data->irq = irq_of_parse_and_map(node, 0);
+        if (!data->irq) {
+                pr_err("Unable to get IRQ from DT\n");
+                return -EINVAL;
+        }
 
         /*
          * Use our actual MCK to figure out how many MCK/16 ticks per
@@ -236,46 +255,5 @@ static int __init at91sam926x_pit_common_init(struct pit_data *data)
 
         return 0;
 }
-
-static int __init at91sam926x_pit_dt_init(struct device_node *node)
-{
-        struct pit_data *data;
-        int ret;
-
-        data = kzalloc(sizeof(*data), GFP_KERNEL);
-        if (!data)
-                return -ENOMEM;
-
-        data->base = of_iomap(node, 0);
-        if (!data->base) {
-                pr_err("Could not map PIT address\n");
-                return -ENXIO;
-        }
-
-        data->mck = of_clk_get(node, 0);
-        if (IS_ERR(data->mck))
-                /* Fallback on clkdev for !CCF-based boards */
-                data->mck = clk_get(NULL, "mck");
-
-        if (IS_ERR(data->mck)) {
-                pr_err("Unable to get mck clk\n");
-                return PTR_ERR(data->mck);
-        }
-
-        ret = clk_prepare_enable(data->mck);
-        if (ret) {
-                pr_err("Unable to enable mck\n");
-                return ret;
-        }
-
-        /* Get the interrupts property */
-        data->irq = irq_of_parse_and_map(node, 0);
-        if (!data->irq) {
-                pr_err("Unable to get IRQ from DT\n");
-                return -EINVAL;
-        }
-
-        return at91sam926x_pit_common_init(data);
-}
 CLOCKSOURCE_OF_DECLARE(at91sam926x_pit, "atmel,at91sam9260-pit",
                        at91sam926x_pit_dt_init);