MIPS: cevt-gt641xx: Migrate to new 'set-state' interface
authorViresh Kumar <viresh.kumar@linaro.org>
Mon, 6 Jul 2015 11:11:57 +0000 (16:41 +0530)
committerRalf Baechle <ralf@linux-mips.org>
Thu, 3 Sep 2015 10:07:51 +0000 (12:07 +0200)
Migrate cevt-gt641xx driver to the new 'set-state' interface provided by
clockevents core, the earlier 'set-mode' interface is marked obsolete
now.

This also enables us to implement callbacks for new states of clockevent
devices, for example: ONESHOT_STOPPED.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: linux-mips@linux-mips.org
Cc: linaro-kernel@lists.linaro.org
Cc: Thomas Gleixner <tglx@linutronix.de>
Patchwork: https://patchwork.linux-mips.org/patch/10604/
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
arch/mips/kernel/cevt-gt641xx.c

index f069460..6604005 100644 (file)
@@ -64,8 +64,7 @@ static int gt641xx_timer0_set_next_event(unsigned long delta,
        return 0;
 }
 
-static void gt641xx_timer0_set_mode(enum clock_event_mode mode,
-                                   struct clock_event_device *evt)
+static int gt641xx_timer0_shutdown(struct clock_event_device *evt)
 {
        u32 ctrl;
 
@@ -73,21 +72,39 @@ static void gt641xx_timer0_set_mode(enum clock_event_mode mode,
 
        ctrl = GT_READ(GT_TC_CONTROL_OFS);
        ctrl &= ~(GT_TC_CONTROL_ENTC0_MSK | GT_TC_CONTROL_SELTC0_MSK);
+       GT_WRITE(GT_TC_CONTROL_OFS, ctrl);
+
+       raw_spin_unlock(&gt641xx_timer_lock);
+       return 0;
+}
+
+static int gt641xx_timer0_set_oneshot(struct clock_event_device *evt)
+{
+       u32 ctrl;
+
+       raw_spin_lock(&gt641xx_timer_lock);
+
+       ctrl = GT_READ(GT_TC_CONTROL_OFS);
+       ctrl &= ~GT_TC_CONTROL_SELTC0_MSK;
+       ctrl |= GT_TC_CONTROL_ENTC0_MSK;
+       GT_WRITE(GT_TC_CONTROL_OFS, ctrl);
+
+       raw_spin_unlock(&gt641xx_timer_lock);
+       return 0;
+}
 
-       switch (mode) {
-       case CLOCK_EVT_MODE_PERIODIC:
-               ctrl |= GT_TC_CONTROL_ENTC0_MSK | GT_TC_CONTROL_SELTC0_MSK;
-               break;
-       case CLOCK_EVT_MODE_ONESHOT:
-               ctrl |= GT_TC_CONTROL_ENTC0_MSK;
-               break;
-       default:
-               break;
-       }
+static int gt641xx_timer0_set_periodic(struct clock_event_device *evt)
+{
+       u32 ctrl;
+
+       raw_spin_lock(&gt641xx_timer_lock);
 
+       ctrl = GT_READ(GT_TC_CONTROL_OFS);
+       ctrl |= GT_TC_CONTROL_ENTC0_MSK | GT_TC_CONTROL_SELTC0_MSK;
        GT_WRITE(GT_TC_CONTROL_OFS, ctrl);
 
        raw_spin_unlock(&gt641xx_timer_lock);
+       return 0;
 }
 
 static void gt641xx_timer0_event_handler(struct clock_event_device *dev)
@@ -95,12 +112,16 @@ static void gt641xx_timer0_event_handler(struct clock_event_device *dev)
 }
 
 static struct clock_event_device gt641xx_timer0_clockevent = {
-       .name           = "gt641xx-timer0",
-       .features       = CLOCK_EVT_FEAT_PERIODIC | CLOCK_EVT_FEAT_ONESHOT,
-       .irq            = GT641XX_TIMER0_IRQ,
-       .set_next_event = gt641xx_timer0_set_next_event,
-       .set_mode       = gt641xx_timer0_set_mode,
-       .event_handler  = gt641xx_timer0_event_handler,
+       .name                   = "gt641xx-timer0",
+       .features               = CLOCK_EVT_FEAT_PERIODIC |
+                                 CLOCK_EVT_FEAT_ONESHOT,
+       .irq                    = GT641XX_TIMER0_IRQ,
+       .set_next_event         = gt641xx_timer0_set_next_event,
+       .set_state_shutdown     = gt641xx_timer0_shutdown,
+       .set_state_periodic     = gt641xx_timer0_set_periodic,
+       .set_state_oneshot      = gt641xx_timer0_set_oneshot,
+       .tick_resume            = gt641xx_timer0_shutdown,
+       .event_handler          = gt641xx_timer0_event_handler,
 };
 
 static irqreturn_t gt641xx_timer0_interrupt(int irq, void *dev_id)