[PATCH] Don't disable irqs in set_next_event and set_mode callbacks

Uwe Kleine-König u.kleine-koenig at pengutronix.de
Mon Sep 21 05:16:14 EDT 2009


On Mon, Sep 21, 2009 at 10:04:00AM +0100, Russell King - ARM Linux wrote:
> On Mon, Sep 21, 2009 at 09:39:22AM +0200, Uwe Kleine-König wrote:
> > These functions are called with irqs already off.
> > 
> > at91rm2000 had a WARN_ON_ONCE if irqs were enabled since Nov 2008 with
> > noone reporting having hit it.
> 
> It might be useful to document these clockevent interfaces.  There's
> at least a few ARM clockevent implementations which don't set the
> periodic interval when set_next_event() is called - probably because
> it wasn't realised that it was required.
I'm a bit confused now as AFAIK set_next_event isn't called in periodic
mode.

You mean they don't start the timer if mode==CLOCK_EVT_MODE_PERIODIC in
.set_mode?
It looks as if pxa is such an implementation, it does nothing in this
case.

Best regards
Uwe

-- 
Pengutronix e.K.                              | Uwe Kleine-König            |
Industrial Linux Solutions                    | http://www.pengutronix.de/  |



More information about the linux-arm-kernel mailing list