From: Viresh Kumar <[email protected]>

Migrate sh_mtu2 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.

Cc: Magnus Damm <[email protected]>
Cc: Laurent Pinchart <[email protected]>
Cc: Paul Mundt <[email protected]>
Signed-off-by: Viresh Kumar <[email protected]>
Signed-off-by: Daniel Lezcano <[email protected]>
Acked-by: Laurent Pinchart <[email protected]>
---
 drivers/clocksource/sh_mtu2.c | 42 ++++++++++++++++--------------------------
 1 file changed, 16 insertions(+), 26 deletions(-)

diff --git a/drivers/clocksource/sh_mtu2.c b/drivers/clocksource/sh_mtu2.c
index 3d88698..f1985da 100644
--- a/drivers/clocksource/sh_mtu2.c
+++ b/drivers/clocksource/sh_mtu2.c
@@ -276,36 +276,25 @@ static struct sh_mtu2_channel *ced_to_sh_mtu2(struct 
clock_event_device *ced)
        return container_of(ced, struct sh_mtu2_channel, ced);
 }
 
-static void sh_mtu2_clock_event_mode(enum clock_event_mode mode,
-                                   struct clock_event_device *ced)
+static int sh_mtu2_clock_event_shutdown(struct clock_event_device *ced)
 {
        struct sh_mtu2_channel *ch = ced_to_sh_mtu2(ced);
-       int disabled = 0;
 
-       /* deal with old setting first */
-       switch (ced->mode) {
-       case CLOCK_EVT_MODE_PERIODIC:
+       sh_mtu2_disable(ch);
+       return 0;
+}
+
+static int sh_mtu2_clock_event_set_periodic(struct clock_event_device *ced)
+{
+       struct sh_mtu2_channel *ch = ced_to_sh_mtu2(ced);
+
+       if (clockevent_state_periodic(ced))
                sh_mtu2_disable(ch);
-               disabled = 1;
-               break;
-       default:
-               break;
-       }
 
-       switch (mode) {
-       case CLOCK_EVT_MODE_PERIODIC:
-               dev_info(&ch->mtu->pdev->dev,
-                        "ch%u: used for periodic clock events\n", ch->index);
-               sh_mtu2_enable(ch);
-               break;
-       case CLOCK_EVT_MODE_UNUSED:
-               if (!disabled)
-                       sh_mtu2_disable(ch);
-               break;
-       case CLOCK_EVT_MODE_SHUTDOWN:
-       default:
-               break;
-       }
+       dev_info(&ch->mtu->pdev->dev, "ch%u: used for periodic clock events\n",
+                ch->index);
+       sh_mtu2_enable(ch);
+       return 0;
 }
 
 static void sh_mtu2_clock_event_suspend(struct clock_event_device *ced)
@@ -327,7 +316,8 @@ static void sh_mtu2_register_clockevent(struct 
sh_mtu2_channel *ch,
        ced->features = CLOCK_EVT_FEAT_PERIODIC;
        ced->rating = 200;
        ced->cpumask = cpu_possible_mask;
-       ced->set_mode = sh_mtu2_clock_event_mode;
+       ced->set_state_shutdown = sh_mtu2_clock_event_shutdown;
+       ced->set_state_periodic = sh_mtu2_clock_event_set_periodic;
        ced->suspend = sh_mtu2_clock_event_suspend;
        ced->resume = sh_mtu2_clock_event_resume;
 
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to