linux-mips
[Top] [All Lists]

Re: [PATCH] pnx8550: fix system timer support

To: vitalywool@gmail.com
Subject: Re: [PATCH] pnx8550: fix system timer support
From: Atsushi Nemoto <anemo@mba.ocn.ne.jp>
Date: Thu, 21 Dec 2006 23:54:09 +0900 (JST)
Cc: ralf@linux-mips.org, linux-mips@linux-mips.org, danieljlaird@hotmail.com
In-reply-to: <20061221173439.fc76c832.vitalywool@gmail.com>
Original-recipient: rfc822;linux-mips@linux-mips.org
References: <20061221173439.fc76c832.vitalywool@gmail.com>
Sender: linux-mips-bounce@linux-mips.org
On Thu, 21 Dec 2006 17:34:39 +0300, Vitaly Wool <vitalywool@gmail.com> wrote:
> the patch inlined below restores proper time accounting for
> PNX8550-based boards. It also gets rid of #ifdef in the generic code
> which becomes unnecessary then.
...
> Signed-off-by: Vitaly Wool <vwool@ru.mvista.com>

This patch obsoletes my patch posted 2 days ago.  Daniel, please
ignore mine and try this.

> --- a/arch/mips/philips/pnx8550/common/time.c
> +++ b/arch/mips/philips/pnx8550/common/time.c
> @@ -29,12 +29,27 @@
>  #include <asm/hardirq.h>
>  #include <asm/div64.h>
>  #include <asm/debug.h>
> +#include <asm/time.h>
>  
>  #include <int.h>
>  #include <cm.h>
>  
>  extern unsigned int mips_hpt_frequency;

Minor comment.  The asm/time.h is already included, isn't it?  And
mips_hpt_frequency is declared in asm/time.h so the "external" line
can be removed.

---
Atsushi Nemoto

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