> > +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