To ensure that the timer interrupt is properly enabled/disabled across
the whole CPU cluster use enable/disable_irq() instead of
local_irq_disable().

Signed-off-by: Soren Brinkmann <soren.brinkm...@xilinx.com>
---
 drivers/clocksource/cadence_ttc_timer.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/clocksource/cadence_ttc_timer.c 
b/drivers/clocksource/cadence_ttc_timer.c
index a92350b55d32..246d018d1e63 100644
--- a/drivers/clocksource/cadence_ttc_timer.c
+++ b/drivers/clocksource/cadence_ttc_timer.c
@@ -322,18 +322,16 @@ static int ttc_rate_change_clockevent_cb(struct 
notifier_block *nb,
        switch (event) {
        case POST_RATE_CHANGE:
        {
-               unsigned long flags;
-
                /*
                 * clockevents_update_freq should be called with IRQ disabled on
                 * the CPU the timer provides events for. The timer we use is
                 * common to both CPUs, not sure if we need to run on both
                 * cores.
                 */
-               local_irq_save(flags);
+               disable_irq(ttcce->ce.irq);
                clockevents_update_freq(&ttcce->ce,
                                ndata->new_rate / PRESCALE);
-               local_irq_restore(flags);
+               enable_irq(ttcce->ce.irq);
 
                /* update cached frequency */
                ttc->freq = ndata->new_rate;
-- 
1.8.4.4

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to