> > +static inline void timer_ack(void) > > +{ > > + write_c0_compare(cpj); > > +} > > I still don't understand why you need this function at all, and the 'cpj' > variable as well -- clockevents core will set the comparator to a needed > value. Also, I don't see much value in moving that function... Well, it's explicitly made inline and it has been moved closer to the calling function. Vitaly