]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
clocksource/drivers/imx-gpt: Remove unnecessary irq protection
authorClément Péron <peron.clem@gmail.com>
Mon, 5 Nov 2018 14:15:11 +0000 (15:15 +0100)
committerDaniel Lezcano <daniel.lezcano@linaro.org>
Tue, 18 Dec 2018 21:22:23 +0000 (22:22 +0100)
shutdown and oneshot are already protected against irq interruptions

Signed-off-by: Clément Péron <peron.clem@gmail.com>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
drivers/clocksource/timer-imx-gpt.c

index a3d6ccbf4a167c539ed600517915248902a128b7..706c0d0ff56cc9fec9af13c6a38d2fa6a3bfe516 100644 (file)
@@ -202,15 +202,8 @@ static int v2_set_next_event(unsigned long evt,
 static int mxc_shutdown(struct clock_event_device *ced)
 {
        struct imx_timer *imxtm = to_imx_timer(ced);
-       unsigned long flags;
        u32 tcn;
 
-       /*
-        * The timer interrupt generation is disabled at least
-        * for enough time to call mxc_set_next_event()
-        */
-       local_irq_save(flags);
-
        /* Disable interrupt in GPT module */
        imxtm->gpt->gpt_irq_disable(imxtm);
 
@@ -225,21 +218,12 @@ static int mxc_shutdown(struct clock_event_device *ced)
        printk(KERN_INFO "%s: changing mode\n", __func__);
 #endif /* DEBUG */
 
-       local_irq_restore(flags);
-
        return 0;
 }
 
 static int mxc_set_oneshot(struct clock_event_device *ced)
 {
        struct imx_timer *imxtm = to_imx_timer(ced);
-       unsigned long flags;
-
-       /*
-        * The timer interrupt generation is disabled at least
-        * for enough time to call mxc_set_next_event()
-        */
-       local_irq_save(flags);
 
        /* Disable interrupt in GPT module */
        imxtm->gpt->gpt_irq_disable(imxtm);
@@ -264,7 +248,6 @@ static int mxc_set_oneshot(struct clock_event_device *ced)
         * mode switching
         */
        imxtm->gpt->gpt_irq_enable(imxtm);
-       local_irq_restore(flags);
 
        return 0;
 }