> > Something probably have converted tabs to 8 spaces...
No, it's just the lines of original code moved.
> > [...]
> >
> >> +static struct clock_event_device pnx8xxx_clockevent = {
> >> + .name = "pnx8xxx_clockevent",
> >> + .features = CLOCK_EVT_FEAT_ONESHOT,
> >
> > Aren't PNX8550 timers actually periodic in nature?
>
> All I recall is they were odd ;-)
>
> The hardware nature of timers and how to declare them to the Linux timer
> code is not always the same. CLOCK_EVT_FEAT_ONESHOT be used if the
> time to the next shot can be programmed.
Absolutely :)
> > Enabling timers before they are actually set up? :-|
>
> Are the additional timers used at all?
Heh, that also was the original code which was just moved from one
place in the file to the other.
Sergei, I decided not to mix whitespace cleanups and the clocksource
changes. I'll come up with the whitespace/tab fixups soon arranging it
as a separate patch.
Vitaly
