On Wed, 24 Oct 2007 23:00:37 +0400, Sergei Shtylyov <firstname.lastname@example.org>
> > + if (tx4927_ccfgptr->ccfg & TX4927_CCFG_TINTDIS)
> > + txx9_clockevent_init(TX4927_TMR_REG(0) & 0xfffffffffULL,
> > + TXX9_IRQ_BASE + 17,
> > + 50000000);
> Wait, you're not registering TXx9 clocksource anyway, so it's only taking
> space indeed. Maybe we should register clock source/event regardless of the
> value of CCFG.TINTDIS, and let them be selected according to rating?
Yes, I did not intentionally. If new board developers wanted to use
txx9_clocksource (for some mysterious reason), he can register it.
For existing tx49 platform, no good reason to use txx9_clocksource.
The standard mips_clocksource can be used regardless of TINTDIS, and
mips_clocksource is _always_ has higher precision (and lightweight)
And if you only registered txx9 clock source/event and did not use
them, those timer modules cannot be used for other purpose (for
example, pulse generator) anymore. It looks overkill for me.