diff options
| author |   Giovambattista Pulcini <gpulcini@swintel.it> | 2005-04-16 15:24:24 -0700 |
|---|---|---|
| committer |   Linus Torvalds <torvalds@ppc970.osdl.org> | 2005-04-16 15:24:24 -0700 |
| commit | 54095a6ec7b11a008c07c5016b025206a051f560 (patch) | |
| tree | 7e2abe2dc483e05387c331ec41e1f5c51678dd3e /arch/ppc/platforms/chrp_time.c | |
| parent | [PATCH] ppc32: Fix mpc8xx watchdog (diff) | |
| download | linux-dev-54095a6ec7b11a008c07c5016b025206a051f560.tar.xz linux-dev-54095a6ec7b11a008c07c5016b025206a051f560.zip | |
[PATCH] ppc32: Fix a problem with NTP on !(chrp||gemini)
The following problem was found by Giovambattista Pulcini
<gpulcini@swintel.it>, who also provided a partial patch, and this has been
verified by our time guru Gabriel Paubert <paubert@iram.es>.
The problem is that in do_settimeofday() we always set time_state to
TIME_ERROR and except on two platforms, never re-set it. This meant that
ntp_gettime() and ntp_adjtime() always returned TIME_ERROR, incorrectly.
Based on Gabriel's analysis, time_state is used for leap-second processing,
and ppc shouldn't be mucking with it.
Signed-off-by: Tom Rini <trini@kernel.crashing.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch/ppc/platforms/chrp_time.c')
| -rw-r--r-- | arch/ppc/platforms/chrp_time.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/arch/ppc/platforms/chrp_time.c b/arch/ppc/platforms/chrp_time.c index e2be0c838d8a..4864a7de3daa 100644 --- a/arch/ppc/platforms/chrp_time.c +++ b/arch/ppc/platforms/chrp_time.c @@ -115,8 +115,6 @@ int __chrp chrp_set_rtc_time(unsigned long nowtime) chrp_cmos_clock_write(save_control, RTC_CONTROL); chrp_cmos_clock_write(save_freq_select, RTC_FREQ_SELECT); - if ( (time_state == TIME_ERROR) || (time_state == TIME_BAD) ) - time_state = TIME_OK; spin_unlock(&rtc_lock); return 0; } |