On Wed, Oct 8, 2014 at 6:44 PM, Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote: > On Wed, 8 Oct 2014, Ley Foon Tan wrote: >> +static void nios2_timer_set_mode(enum clock_event_mode mode, >> + struct clock_event_device *evt) >> +{ >> + unsigned long period; >> + struct nios2_clockevent_dev *nios2_ced = to_nios2_clkevent(evt); >> + struct nios2_timer *timer = &nios2_ced->timer; >> + >> + switch (mode) { >> + case CLOCK_EVT_MODE_PERIODIC: >> + period = DIV_ROUND_UP(timer->freq, HZ); >> + nios2_timer_config(timer, period, CLOCK_EVT_MODE_PERIODIC); >> + break; >> + case CLOCK_EVT_MODE_ONESHOT: >> + nios2_timer_config(timer, ULONG_MAX, CLOCK_EVT_MODE_ONESHOT); > > What's the point of this? This should simply stop the timer. Okay, will change this. Regards Ley Foon -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html