Hi, On 6/14/07, Maciej W. Rozycki <macro@xxxxxxxxxxxxxx> wrote:
On Thu, 14 Jun 2007, Franck Bui-Huu wrote: > The current code doesn't automatically calibrate any hpt. It was > really hard to guess which ones need that so now if you need to > calibrate your hpt, then you have to call calibrate_hpt(). You are wrong -- calibration is currently automatic if a platform provides a HPT, but has not set up its frequency:
Well it all depends on what you call "current" code... My own fault I should have said "new" instead of "current", sorry.
Which should normally be the case unless there is no way to do calibration, when a platform can provide a hardcoded value. There is nothing to guess here.
Are you sure it's the normal case? I would say that only DEC needs that calibration: Doing the following on the _current_ tree: $ git grep -l mips_timer_state arch/mips arch/mips/dec/time.c arch/mips/kernel/time.c
I'll have a look at your patches, but I hope you have got about the most interesting configuration right, which is the DEC platform, where you can have one of these:
As I said, all platforms haven't been migrated, but DEC seems an interesting one to migrate. I'll try to do it and see if you can accept such changes.
1. No HPT at all. 2. HPT in the chipset. 3. HPT in CP0. depending on the configuration as determined at the run time, with no predefined frequency in the cases #2 and #3.
Good to know. Thanks -- Franck