On Sun, 13 Feb 2011, Guan Xuetao wrote: > +static struct clock_event_device ckevt_puv3_osmr0 = { > + .name = "osmr0", > + .features = CLOCK_EVT_FEAT_ONESHOT, > +#ifdef CONFIG_ARCH_FPGA > + .shift = 18, /* correct shift val: 16, but warn_on_slowpath */ > +#else > + .shift = 30, > +#endif Please get rid of this. > +void __init time_init(void) > +{ > + OST_OIER = 0; /* disable any timer interrupts */ > + OST_OSSR = 0; /* clear status on all timers */ > + > + ckevt_puv3_osmr0.mult = > + div_sc(CLOCK_TICK_RATE, NSEC_PER_SEC, ckevt_puv3_osmr0.shift); And use clockevents_calc_mult_shift() instead of this. Thanks, tglx -- 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