Re: [PATCH] 2.6.14 include/asm-arm/arch-ixp4xx/timex.h: fix clock frequency on NSLU2

From: Lennert Buytenhek
Date: Fri Oct 28 2005 - 03:33:01 EST


John,


On Thu, Oct 27, 2005 at 11:32:25PM -0700, John Bowler wrote:

> The problem is that the NSLU2 deviates from the Intel IXDP425
> system by using a cystal with frequency 66MHz as opposed to the
> Intel stated (not quite mandated) value of 66.666666MHz. This
> results in serious errors in the RTC (1%).
>
> The patch configs in the correct value for the NSLU2. The problem
> with the patch is that a multi-config kernel which includes NSLU2
> support will use the NSLU2 frequency. However such kernels are
> not a requirement as generic kernels are never flashed into NSLU2
> systems (so there is no need to build NSLU2 support into a generic
> kernel).

As previously stated: with this patch, compiling in support for the
NSLU2 will make the timer tick base right for the NSLU2 but wrong
for everything else.

There are two ways of dealing with this problem:
1/ The patch you sent, plus some config hacks to make it impossible
to compile in NSLU2 support together with support for another ixp4xx
machine type that has FREQ=66666666, sidestepping the issue; or
2/ Make the ixp4xx timer tick base runtime selectable altogether, for
example with the patch that I sent to linux-arm-kernel a while ago.

I'm not very much in favor of 1/, but at least it's better than not
having any kind of protection at all. If you leave it to the user,
they simply _are_ going to get it messed up.


--L
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/