summaryrefslogtreecommitdiffstats
path: root/sys/isa/atrtc.c
diff options
context:
space:
mode:
Diffstat (limited to 'sys/isa/atrtc.c')
-rw-r--r--sys/isa/atrtc.c20
1 files changed, 8 insertions, 12 deletions
diff --git a/sys/isa/atrtc.c b/sys/isa/atrtc.c
index ff37d5d..93d486c 100644
--- a/sys/isa/atrtc.c
+++ b/sys/isa/atrtc.c
@@ -692,14 +692,6 @@ set_timer_freq(u_int freq, int intr_freq)
mtx_unlock_spin(&clock_lock);
}
-/*
- * i8254_restore is called from apm_default_resume() to reload
- * the countdown register.
- * this should not be necessary but there are broken laptops that
- * do not restore the countdown register on resume.
- * when it happnes, it messes up the hardclock interval and system clock,
- * which leads to the infamous "calcru: negative time" problem.
- */
static void
i8254_restore(void)
{
@@ -715,15 +707,19 @@ static void
rtc_restore(void)
{
- /* Reenable RTC updates and interrupts. */
- /* XXX locking is needed for RTC access? */
+ /* Restore all of the RTC's "status" (actually, control) registers. */
+ /* XXX locking is needed for RTC access. */
+ writertc(RTC_STATUSB, RTCSB_24HR);
writertc(RTC_STATUSA, rtc_statusa);
- writertc(RTC_STATUSB, RTCSB_HALT | RTCSB_24HR);
writertc(RTC_STATUSB, rtc_statusb);
}
/*
- * Restore all the timers atomically.
+ * Restore all the timers non-atomically (XXX: should be atomically).
+ *
+ * This function is called from pmtimer_resume() to restore all the timers.
+ * This should not be necessary, but there are broken laptops that do not
+ * restore all the timers on resume.
*/
void
timer_restore(void)
OpenPOWER on IntegriCloud