[Top] [All Lists]

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

To: "Thomas Bogendoerfer" <>
Subject: Re: [PATCH 3/5] Deforest the function pointer jungle in the time code.
From: "Franck Bui-Huu" <>
Date: Thu, 14 Jun 2007 15:43:47 +0200
Cc:, "Ralf Baechle" <>
Dkim-signature: a=rsa-sha1; c=relaxed/relaxed;; s=beta; h=domainkey-signature:received:received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=VjVszxTQgDUrdwuzzUwo/0Zttgaxi1T4PP/tPEuKFHKE67c+E8oEUyu2DKzuyaPSFmXG+A2dSc/lOQLvMY5hQ1HrSjiiUlaLep1143+BmIXL4EoTft7x7pZNXFxlpxWzqex8s5O2w3d6CGUtPI9EYNcNKD/+daSr9gdrmVyv8z0=
Domainkey-signature: a=rsa-sha1; c=nofws;; s=beta; h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=UYIoFSypvpNyDrtyD7VJpJnABv7ON8aeEiMEQmrVCLRiYBUtf80Yk5ZDNWo3rZHBJqp2BViiltkmk64/Ybge2dr/QFxJ+/RQbIhzs6N7Av4LY+cYxDSwMdLcF1hnxQOojaS5h7PQ09Howeh+I1iPp8lRLmd4zsfd1FEt9KSsHFk=
In-reply-to: <>
Original-recipient: rfc822;
References: <> <> <>

On 6/14/07, Thomas Bogendoerfer <> wrote:
On Thu, Jun 14, 2007 at 12:19:59PM +0200, Franck Bui-Huu wrote:
>  arch/mips/sni/a20r.c                      |    1 -
>  arch/mips/sni/ds1216.c                    |    4 +-
>  arch/mips/sni/pcimt.c                     |    3 -
>  arch/mips/sni/pcit.c                      |    3 -
>  arch/mips/sni/rm200.c                     |    2 -
>  arch/mips/sni/time.c                      |    2 +-

the SNI part is broken and can't work that way.

I don't get you there. Are you talking about patch 3/5 (you're
replying on this one) or the patch 5/5 ?

patch #3 is only clean up, so it shouldn't break anything....
patch #5 does not migrate any platforms, so it's actually broken for
all current platforms.

1. SNI used two different RTC chips (ds126 and mc146818) and it's no big
   deal support them in just one kernel with the current framework

That's the point now, if the current implementation can fit well with
all platforms.

2. One line of SNI machines (a20r) can't use the cp0 counter, so it's not
   a really good idea to calibrate it

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().


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