Changes in uspace/drv/time/cmos-rtc/cmos-rtc.c [4122410:4f87a85a] in mainline

File:

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

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

@@ -588 +588 @@
         errno_t rc;
         bool need_cleanup = false;
+        bool bound = false;
 
         ddf_msg(LVL_DEBUG, "rtc_dev_add %s (handle = %d)",
@@ -624 +625 @@
         }
 
+        bound = true;
         rtc->fun = fun;
 
-        ddf_fun_add_to_category(fun, "clock");
+        rc = ddf_fun_add_to_category(fun, "clock");
+        if (rc != EOK) {
+                ddf_msg(LVL_ERROR, "Failed adding service to clock category.");
+                goto error;
+        }
 
         ddf_msg(LVL_NOTE, "Device %s successfully initialized",
@@ -634 +640 @@
 
 error:
+        if (bound)
+                ddf_fun_unbind(fun);
         if (fun)
                 ddf_fun_destroy(fun);
@@ -754 +762 @@
 
         rc = ddf_fun_online(fun);
-        if (rc == EOK)
-                ddf_fun_add_to_category(fun, "clock");
+        if (rc == EOK) {
+                // XXX This should be probably handled by the framework
+                rc = ddf_fun_add_to_category(fun, "clock");
+        }
 
         return rc;