diff options
author | Roman Zippel <zippel@linux-m68k.org> | 2006-10-01 02:28:26 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-01 03:39:26 -0400 |
commit | 97eebe138caaf78354b1fad233e63bafdcc4fd54 (patch) | |
tree | 15c006598c953b373af1c164ba344ef7105a7251 | |
parent | 8f807f8d2137ba728d22820103131038639b68a9 (diff) |
[PATCH] ntp: remove time_tolerance
time_tolerance isn't changed at all in the kernel, so simply remove it, this
simplifies the next patch, as it avoids a number of conversions.
Signed-off-by: Roman Zippel <zippel@linux-m68k.org>
Cc: john stultz <johnstul@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | include/linux/timex.h | 1 | ||||
-rw-r--r-- | kernel/time/ntp.c | 9 |
2 files changed, 4 insertions, 6 deletions
diff --git a/include/linux/timex.h b/include/linux/timex.h index b5f297e17668..7715b4c0caf9 100644 --- a/include/linux/timex.h +++ b/include/linux/timex.h | |||
@@ -208,7 +208,6 @@ extern int time_state; /* clock status */ | |||
208 | extern int time_status; /* clock synchronization status bits */ | 208 | extern int time_status; /* clock synchronization status bits */ |
209 | extern long time_offset; /* time adjustment (us) */ | 209 | extern long time_offset; /* time adjustment (us) */ |
210 | extern long time_constant; /* pll time constant */ | 210 | extern long time_constant; /* pll time constant */ |
211 | extern long time_tolerance; /* frequency tolerance (ppm) */ | ||
212 | extern long time_precision; /* clock precision (us) */ | 211 | extern long time_precision; /* clock precision (us) */ |
213 | extern long time_maxerror; /* maximum error */ | 212 | extern long time_maxerror; /* maximum error */ |
214 | extern long time_esterror; /* estimated error */ | 213 | extern long time_esterror; /* estimated error */ |
diff --git a/kernel/time/ntp.c b/kernel/time/ntp.c index 65223b7ed810..af7563f5d4e2 100644 --- a/kernel/time/ntp.c +++ b/kernel/time/ntp.c | |||
@@ -34,7 +34,6 @@ int time_state = TIME_OK; /* clock synchronization status */ | |||
34 | int time_status = STA_UNSYNC; /* clock status bits */ | 34 | int time_status = STA_UNSYNC; /* clock status bits */ |
35 | long time_offset; /* time adjustment (ns) */ | 35 | long time_offset; /* time adjustment (ns) */ |
36 | long time_constant = 2; /* pll time constant */ | 36 | long time_constant = 2; /* pll time constant */ |
37 | long time_tolerance = MAXFREQ; /* frequency tolerance (ppm) */ | ||
38 | long time_precision = 1; /* clock precision (us) */ | 37 | long time_precision = 1; /* clock precision (us) */ |
39 | long time_maxerror = NTP_PHASE_LIMIT; /* maximum error (us) */ | 38 | long time_maxerror = NTP_PHASE_LIMIT; /* maximum error (us) */ |
40 | long time_esterror = NTP_PHASE_LIMIT; /* estimated error (us) */ | 39 | long time_esterror = NTP_PHASE_LIMIT; /* estimated error (us) */ |
@@ -87,7 +86,7 @@ void second_overflow(void) | |||
87 | long time_adj; | 86 | long time_adj; |
88 | 87 | ||
89 | /* Bump the maxerror field */ | 88 | /* Bump the maxerror field */ |
90 | time_maxerror += time_tolerance >> SHIFT_USEC; | 89 | time_maxerror += MAXFREQ >> SHIFT_USEC; |
91 | if (time_maxerror > NTP_PHASE_LIMIT) { | 90 | if (time_maxerror > NTP_PHASE_LIMIT) { |
92 | time_maxerror = NTP_PHASE_LIMIT; | 91 | time_maxerror = NTP_PHASE_LIMIT; |
93 | time_status |= STA_UNSYNC; | 92 | time_status |= STA_UNSYNC; |
@@ -313,8 +312,8 @@ int do_adjtimex(struct timex *txc) | |||
313 | } else /* calibration interval too long (p. 12) */ | 312 | } else /* calibration interval too long (p. 12) */ |
314 | result = TIME_ERROR; | 313 | result = TIME_ERROR; |
315 | } | 314 | } |
316 | time_freq = min(time_freq, time_tolerance); | 315 | time_freq = min(time_freq, MAXFREQ); |
317 | time_freq = max(time_freq, -time_tolerance); | 316 | time_freq = max(time_freq, -MAXFREQ); |
318 | time_offset = (time_offset * NSEC_PER_USEC / HZ) << SHIFT_UPDATE; | 317 | time_offset = (time_offset * NSEC_PER_USEC / HZ) << SHIFT_UPDATE; |
319 | } /* STA_PLL */ | 318 | } /* STA_PLL */ |
320 | } /* txc->modes & ADJ_OFFSET */ | 319 | } /* txc->modes & ADJ_OFFSET */ |
@@ -337,7 +336,7 @@ leave: if ((time_status & (STA_UNSYNC|STA_CLOCKERR)) != 0) | |||
337 | txc->status = time_status; | 336 | txc->status = time_status; |
338 | txc->constant = time_constant; | 337 | txc->constant = time_constant; |
339 | txc->precision = time_precision; | 338 | txc->precision = time_precision; |
340 | txc->tolerance = time_tolerance; | 339 | txc->tolerance = MAXFREQ; |
341 | txc->tick = tick_usec; | 340 | txc->tick = tick_usec; |
342 | 341 | ||
343 | /* PPS is not implemented, so these are zero */ | 342 | /* PPS is not implemented, so these are zero */ |