Re: [PATCH 3/5] Deforest the function pointer jungle in the time code.

[Atsushi Nemoto <anemo@mba.ocn.ne.jp>]

On Mon, 18 Jun 2007 11:38:28 +0200, "Franck Bui-Huu" <vagabon.xyz@gmail.com> 
wrote:
> And it should write it's own clocksource support which would use
> different timer.

I suppose so.

> It shoud result in something like this:
> 
> unsigned __init get_freq(int cpu)
> {
>       return 27UL * ((1000000UL * n)/(m * pow2p));
> }
> 
> void __init plat_timer_init()
> {
>       struct cp0_hpt_info info;
> 
>       info.get_freq = get_freq;
>       info.irq = PNX8550_INT_TIMER1;
>       setup_cp0_hpt(&info, CLKEVT_ONLY);
> 
>       setup_my_clocksource_using_a_different_timer();
> }
> 
> Note that 'CLKEVT_ONLY' flag currently doesn't exist.
> 
> What do you think ?

For now, I think bloating generic setup_cp0_hpt() for this particular
chip is not good.  The pnx8550 can have customized copy of cp0_hpt
routines.  But it's just a thought...

---
Atsushi Nemoto