[PATCH 4/9] lpc2k: generic time and clockevents
Ithamar R. Adema
ithamar.adema at team-embedded.nl
Thu Mar 17 18:00:22 EDT 2011
On Thu, 2011-03-17 at 21:38 +0000, Jamie Iles wrote:
> One pedantic nitpick, but otherwise looks nice to me.
Thanks for reviewing, and pedantic nitpicks are always welcome ;)
> On Thu, Mar 17, 2011 at 04:54:19PM +0100, Ithamar R. Adema wrote:
[....]
> > +static struct clock_event_device lpc2k_clkevt = {
> > + .name = "lpc2k_clkevt",
> > + .features = CLOCK_EVT_FEAT_ONESHOT,
> > + .shift = 32,
>
> You shouldn't need to specify a shift if you use
> clockevents_calc_mult_shift() and let the core code calculate mult and
> shift for you.
Good point, I should use that. Will include this in the next version of
these patches!
Regards,
Ithamar.
More information about the linux-arm-kernel
mailing list