summaryrefslogtreecommitdiffstats
path: root/kernel/time/ntp.c
diff options
context:
space:
mode:
authorIngo Molnar <mingo@elte.hu>2009-02-22 13:22:23 +0100
committerIngo Molnar <mingo@elte.hu>2009-02-25 18:38:12 +0100
commit478b7aab1682246a3d1e76e27a0aecb2f0013379 (patch)
treea40492dc5668a07c6511ae446204408e02a1e980 /kernel/time/ntp.c
parentf939890b6687e05c42361655fb6610fa08f5a601 (diff)
downloadop-kernel-dev-478b7aab1682246a3d1e76e27a0aecb2f0013379.zip
op-kernel-dev-478b7aab1682246a3d1e76e27a0aecb2f0013379.tar.gz
time: ntp: simplify ntp_update_offset_fll()
Impact: cleanup, no functionality changed Change ntp_update_offset_fll() to delta logic instead of absolute value logic. This eliminates 'freq_adj' from the function. Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/time/ntp.c')
-rw-r--r--kernel/time/ntp.c11
1 files changed, 5 insertions, 6 deletions
diff --git a/kernel/time/ntp.c b/kernel/time/ntp.c
index ee437e1..5202dde 100644
--- a/kernel/time/ntp.c
+++ b/kernel/time/ntp.c
@@ -103,20 +103,19 @@ static void ntp_update_frequency(void)
tick_length_base = new_base;
}
-static inline s64 ntp_update_offset_fll(s64 freq_adj, s64 offset64, long secs)
+static inline s64 ntp_update_offset_fll(s64 offset64, long secs)
{
time_status &= ~STA_MODE;
if (secs < MINSEC)
- return freq_adj;
+ return 0;
if (!(time_status & STA_FLL) && (secs <= MAXSEC))
- return freq_adj;
+ return 0;
- freq_adj += div_s64(offset64 << (NTP_SCALE_SHIFT - SHIFT_FLL), secs);
time_status |= STA_MODE;
- return freq_adj;
+ return div_s64(offset64 << (NTP_SCALE_SHIFT - SHIFT_FLL), secs);
}
static void ntp_update_offset(long offset)
@@ -152,7 +151,7 @@ static void ntp_update_offset(long offset)
freq_adj = (offset64 * secs) <<
(NTP_SCALE_SHIFT - 2 * (SHIFT_PLL + 2 + time_constant));
- freq_adj = ntp_update_offset_fll(freq_adj, offset64, secs);
+ freq_adj += ntp_update_offset_fll(offset64, secs);
freq_adj = min(freq_adj + time_freq, MAXFREQ_SCALED);
OpenPOWER on IntegriCloud