[Top] [All Lists]

Re: [PATCH][respin] pnx8550: fix system timer support

Subject: Re: [PATCH][respin] pnx8550: fix system timer support
From: Daniel Laird <>
Date: Tue, 2 Jan 2007 06:05:55 -0800 (PST)
In-reply-to: <>
Original-recipient: rfc822;
References: <> <> <>

Vitaly Wool-4 wrote:
> On 12/28/06, Atsushi Nemoto <> wrote:
>> On Thu, 28 Dec 2006 17:14:05 +0300, Vitaly Wool <>
>> wrote:
>> > --- linux-mips.git.orig/arch/mips/philips/pnx8550/common/time.c
>> > +++ linux-mips.git/arch/mips/philips/pnx8550/common/time.c
>> > @@ -29,11 +29,22 @@
>> >  #include <asm/hardirq.h>
>> >  #include <asm/div64.h>
>> >  #include <asm/debug.h>
>> > +#include <asm/time.h>
>> As I said before, asm/time.h is already included just before there.
>> Why double inclusion?
> Oh shoot, thanks, this hunk is bogus.
> Vitaly
I have now tried this new patch and am still not having much success and am
still not understanding the patch very well.
First things first, if I do use the line = hpt_read; 
It does not compile as this symbol is not in a header file and is a static
struct in arch/mips/kernel/time.c
I can make it not static and extern it from pnx8550/common/time.c is this
how I should do it?

Secondly I look at the logic for the arch/mips/kernel/time.c in the
time_init code
I think we want to follow the else branch (/* We know counter frequency.  Or
we can get it.  */)
In this case it then checks to see if mips_hpt_read is undefined which in
our case it is.
It then defines the mips_hpt_read to be c0_hpt_read

It then also overrides mips_timer_ack to be c0_timer_ack which i think is
wrong as we have already overridden this function in
arch/mips/philips/pnx8550/common/time.c.  (Is this behaviour correct?)

I used the patch and ran the kernel,  It does not work very well, Long Hang
Memory: ......
Very slow behaviour after this.

I tried the following:
else {
    /* We know counter frequency.  Or we can get it.  */
    if (!mips_hpt_read) 
        /* No external high precision timer -- use R4k.  */
        mips_hpt_read = c0_hpt_read;

        if (!mips_timer_state) {
            /* No external timer interrupt -- use R4k.  */
            mips_hpt_init = c0_hpt_timer_init;
+           if(!mips_timer_ack)
                mips_timer_ack = c0_timer_ack;
This means it uses the mips_timer_ack function defined in
(mips_timer_ack = timer_ack;)

If I use this patch then the kernel still hangs for a long time at 
Memory: ...... (of the order of 12-14 secs)
It then seems to run at full speed and to a prompt.

In summary:
How do I override properly?
Should mips_timer_ack = c0_timer_ack; be being done even though the board
specific mips_timer_ack = timer_ack; has been done?
Finally there is still the long hang which still looks top be related to
waiting for counter to cycle could ?

Any help with all three would be appreciated but 1 and 2 appear to be more
generic questions whilst 3 is PNX8550 specific (any help  would be
appreciated there as well).


View this message in context:
Sent from the linux-mips main mailing list archive at

<Prev in Thread] Current Thread [Next in Thread>