Changes in uspace/drv/time/cmos-rtc/cmos-rtc.c [f9b2cb4c:a582dff] in mainline

File:

• uspace/drv/time/cmos-rtc/cmos-rtc.c (modified) (14 diffs)

uspace/drv/time/cmos-rtc/cmos-rtc.c

@@ -40 +40 @@
 #include <as.h>
 #include <sysinfo.h>
+#include <libarch/ddi.h>
 #include <libarch/barrier.h>
 #include <stdio.h>
@@ -76 +77 @@
         int clients_connected;
         /** time at which the system booted */
-        struct timeval boot_time;
+        time_t boottime;
 } rtc_t;
 
@@ -97 +98 @@
 static int rtc_dev_remove(ddf_dev_t *dev);
 static void rtc_register_write(rtc_t *rtc, int reg, int data);
+static time_t uptime_get(void);
 static bool is_battery_ok(rtc_t *rtc);
 static int  rtc_fun_online(ddf_fun_t *fun);
@@ -204 +206 @@
         ddf_msg(LVL_DEBUG, "rtc_dev_initialize %s", ddf_dev_get_name(rtc->dev));
 
-        rtc->boot_time.tv_sec = 0;
-        rtc->boot_time.tv_usec = 0;
+        rtc->boottime = 0;
         rtc->clients_connected = 0;
 
@@ -213 +214 @@
         /* Connect to the parent's driver */
 
-        parent_sess = ddf_dev_parent_sess_create(rtc->dev);
+        parent_sess = ddf_dev_parent_sess_create(rtc->dev, EXCHANGE_SERIALIZE);
         if (!parent_sess) {
                 ddf_msg(LVL_ERROR, "Failed to connect to parent driver\
@@ -326 +327 @@
         fibril_mutex_lock(&rtc->mutex);
 
-        if (rtc->boot_time.tv_sec) {
+        if (rtc->boottime != 0) {
                 /* There is no need to read the current time from the
                  * device because it has already been cached.
                  */
 
-                struct timeval curtime;
-                
-                getuptime(&curtime);
-                tv_add(&curtime, &rtc->boot_time);
+                time_t cur_time = rtc->boottime + uptime_get();
+
                 fibril_mutex_unlock(&rtc->mutex);
 
-                return time_tv2tm(&curtime, t);
+                return time_local2tm(cur_time, t);
         }
 
@@ -346 +345 @@
         }
 
-        /* Microseconds are below RTC's resolution, assume 0. */
-        t->tm_usec = 0;
-
         /* now read the registers */
         do {
                 /* Suspend until the update process has finished */
-                while (rtc_update_in_progress(rtc))
-                        ;
-
-                t->tm_sec = rtc_register_read(rtc, RTC_SEC);
-                t->tm_min = rtc_register_read(rtc, RTC_MIN);
+                while (rtc_update_in_progress(rtc));
+
+                t->tm_sec  = rtc_register_read(rtc, RTC_SEC);
+                t->tm_min  = rtc_register_read(rtc, RTC_MIN);
                 t->tm_hour = rtc_register_read(rtc, RTC_HOUR);
                 t->tm_mday = rtc_register_read(rtc, RTC_DAY);
-                t->tm_mon = rtc_register_read(rtc, RTC_MON);
+                t->tm_mon  = rtc_register_read(rtc, RTC_MON);
                 t->tm_year = rtc_register_read(rtc, RTC_YEAR);
 
                 /* Now check if it is stable */
-        } while(t->tm_sec != rtc_register_read(rtc, RTC_SEC) ||
-            t->tm_min != rtc_register_read(rtc, RTC_MIN) ||
+        } while(t->tm_sec  != rtc_register_read(rtc, RTC_SEC) ||
+            t->tm_min  != rtc_register_read(rtc, RTC_MIN) ||
             t->tm_mday != rtc_register_read(rtc, RTC_DAY) ||
-            t->tm_mon != rtc_register_read(rtc, RTC_MON) ||
+            t->tm_mon  != rtc_register_read(rtc, RTC_MON) ||
             t->tm_year != rtc_register_read(rtc, RTC_YEAR));
 
@@ -372 +367 @@
         bool _12h_mode = !(rtc_register_read(rtc, RTC_STATUS_B) &
             RTC_B_24H);
+
         if (_12h_mode) {
                 /* The RTC is working in 12h mode, check if it is AM or PM */
@@ -383 +379 @@
         /* Check if the RTC is working in BCD mode */
         bcd_mode = !(rtc_register_read(rtc, RTC_STATUS_B) & RTC_B_BCD);
+
         if (bcd_mode) {
-                t->tm_sec = bcd2bin(t->tm_sec);
-                t->tm_min = bcd2bin(t->tm_min);
+                t->tm_sec  = bcd2bin(t->tm_sec);
+                t->tm_min  = bcd2bin(t->tm_min);
                 t->tm_hour = bcd2bin(t->tm_hour);
                 t->tm_mday = bcd2bin(t->tm_mday);
-                t->tm_mon = bcd2bin(t->tm_mon);
+                t->tm_mon  = bcd2bin(t->tm_mon);
                 t->tm_year = bcd2bin(t->tm_year);
         }
@@ -418 +415 @@
                 result = EINVAL;
         else {
-                struct timeval uptime;
-
-                getuptime(&uptime);
-                rtc->boot_time.tv_sec = r;
-                rtc->boot_time.tv_usec = t->tm_usec;    /* normalized */
-                tv_sub(&rtc->boot_time, &uptime);
+                rtc->boottime = r - uptime_get();
                 result = EOK;
         }
@@ -444 +436 @@
         bool bcd_mode;
         time_t norm_time;
-        struct timeval uptime;
-        struct timeval ntv;
+        time_t uptime;
         int  reg_b;
         int  reg_a;
@@ -455 +446 @@
                 return EINVAL;
 
-        ntv.tv_sec = norm_time;
-        ntv.tv_usec = t->tm_usec;
-        getuptime(&uptime);
-
-        if (tv_gteq(&uptime, &ntv)) {
+        uptime = uptime_get();
+        if (norm_time <= uptime) {
                 /* This is not acceptable */
                 return EINVAL;
@@ -471 +459 @@
         }
 
-        /* boot_time must be recomputed */
-        rtc->boot_time.tv_sec = 0;
-        rtc->boot_time.tv_usec = 0;
+        /* boottime must be recomputed */
+        rtc->boottime = 0;
 
         /* Detect the RTC epoch */
@@ -745 +732 @@
 }
 
+static time_t
+uptime_get(void)
+{
+        struct timeval tv;
+
+        getuptime(&tv);
+
+        return tv.tv_sec;
+}
+
 static int
 rtc_fun_online(ddf_fun_t *fun)