Changes in uspace/drv/bus/isa/isa.c [267f235:e882e3a] in mainline

File:

• uspace/drv/bus/isa/isa.c (modified) (20 diffs)

uspace/drv/bus/isa/isa.c

@@ -73 +73 @@
 #define CHILD_FUN_CONF_PATH "/drv/isa/isa.dev"
 
+/** Obtain soft-state from device node */
+#define ISA_BUS(dev) ((isa_bus_t *) ((dev)->driver_data))
+
+/** Obtain soft-state from function node */
+#define ISA_FUN(fun) ((isa_fun_t *) ((fun)->driver_data))
+
 #define ISA_MAX_HW_RES 5
 
@@ -89 +95 @@
 } isa_fun_t;
 
-/** Obtain soft-state from device node */
-static isa_bus_t *isa_bus(ddf_dev_t *dev)
-{
-        return ddf_dev_data_get(dev);
-}
-
-/** Obtain soft-state from function node */
-static isa_fun_t *isa_fun(ddf_fun_t *fun)
-{
-        return ddf_fun_data_get(fun);
-}
-
 static hw_resource_list_t *isa_get_fun_resources(ddf_fun_t *fnode)
 {
-        isa_fun_t *fun = isa_fun(fnode);
+        isa_fun_t *fun = ISA_FUN(fnode);
         assert(fun != NULL);
 
@@ -113 +107 @@
         /* This is an old ugly way, copied from pci driver */
         assert(fnode);
-        isa_fun_t *fun = isa_fun(fnode);
+        isa_fun_t *isa_fun = fnode->driver_data;
 
         sysarg_t apic;
@@ -129 +123 @@
                 return false;
 
-        const hw_resource_list_t *res = &fun->hw_resources;
+        assert(isa_fun);
+        const hw_resource_list_t *res = &isa_fun->hw_resources;
         assert(res);
         for (size_t i = 0; i < res->count; ++i) {
@@ -155 +150 @@
 {
         assert(fnode);
-        isa_fun_t *fun = isa_fun(fnode);
-        const hw_resource_list_t *res = &fun->hw_resources;
+        isa_fun_t *isa_fun = fnode->driver_data;
+        const hw_resource_list_t *res = &isa_fun->hw_resources;
         assert(res);
         
@@ -206 +201 @@
 
         isa_fun_t *fun = ddf_fun_data_alloc(fnode, sizeof(isa_fun_t));
-        if (fun == NULL) {
-                ddf_fun_destroy(fnode);
+        if (fun == NULL)
                 return NULL;
-        }
 
         fibril_mutex_initialize(&fun->mutex);
@@ -344 +337 @@
 
                 ddf_msg(LVL_NOTE, "Added irq 0x%x to function %s", irq,
-                    ddf_fun_get_name(fun->fnode));
+                    fun->fnode->name);
         }
 }
@@ -360 +353 @@
                         fun->hw_resources.count++;
                         ddf_msg(LVL_NOTE, "Added dma 0x%x to function %s", dma,
-                            ddf_fun_get_name(fun->fnode));
+                            fun->fnode->name);
                         
                         return;
@@ -371 +364 @@
                         fun->hw_resources.count++;
                         ddf_msg(LVL_NOTE, "Added dma 0x%x to function %s", dma,
-                            ddf_fun_get_name(fun->fnode));
+                            fun->fnode->name);
                         
                         return;
@@ -377 +370 @@
                 
                 ddf_msg(LVL_WARN, "Skipped dma 0x%x for function %s", dma,
-                    ddf_fun_get_name(fun->fnode));
+                    fun->fnode->name);
         }
 }
@@ -396 +389 @@
                 ddf_msg(LVL_NOTE, "Added io range (addr=0x%x, size=0x%x) to "
                     "function %s", (unsigned int) addr, (unsigned int) len,
-                    ddf_fun_get_name(fun->fnode));
+                    fun->fnode->name);
         }
 }
@@ -468 +461 @@
         if (val == end) {
                 ddf_msg(LVL_ERROR, "Cannot read match score for function "
-                    "%s.", ddf_fun_get_name(fun->fnode));
+                    "%s.", fun->fnode->name);
                 return;
         }
@@ -476 +469 @@
         if (id == NULL) {
                 ddf_msg(LVL_ERROR, "Cannot read match ID for function %s.",
-                    ddf_fun_get_name(fun->fnode));
+                    fun->fnode->name);
                 return;
         }
 
         ddf_msg(LVL_DEBUG, "Adding match id '%s' with score %d to "
-            "function %s", id, score, ddf_fun_get_name(fun->fnode));
+            "function %s", id, score, fun->fnode->name);
 
         rc = ddf_fun_add_match_id(fun->fnode, id, score);
@@ -559 +552 @@
 
         isa_fun_t *fun = isa_fun_create(isa, fun_name);
-        free(fun_name);
         if (fun == NULL) {
+                free(fun_name);
                 return NULL;
         }
@@ -584 +577 @@
 
         /* Set device operations to the device. */
-        ddf_fun_set_ops(fun->fnode, &isa_fun_ops);
-
-        ddf_msg(LVL_DEBUG, "Binding function %s.", ddf_fun_get_name(fun->fnode));
+        fun->fnode->ops = &isa_fun_ops;
+
+        ddf_msg(LVL_DEBUG, "Binding function %s.", fun->fnode->name);
 
         /* XXX Handle error */
@@ -619 +612 @@
 
         ddf_msg(LVL_DEBUG, "isa_dev_add, device handle = %d",
-            (int) ddf_dev_get_handle(dev));
+            (int) dev->handle);
 
         isa = ddf_dev_data_alloc(dev, sizeof(isa_bus_t));
@@ -657 +650 @@
 static int isa_dev_remove(ddf_dev_t *dev)
 {
-        isa_bus_t *isa = isa_bus(dev);
+        isa_bus_t *isa = ISA_BUS(dev);
         int rc;
 
@@ -669 +662 @@
                 if (rc != EOK) {
                         fibril_mutex_unlock(&isa->mutex);
-                        ddf_msg(LVL_ERROR, "Failed offlining %s", ddf_fun_get_name(fun->fnode));
+                        ddf_msg(LVL_ERROR, "Failed offlining %s", fun->fnode->name);
                         return rc;
                 }
@@ -676 +669 @@
                 if (rc != EOK) {
                         fibril_mutex_unlock(&isa->mutex);
-                        ddf_msg(LVL_ERROR, "Failed unbinding %s", ddf_fun_get_name(fun->fnode));
+                        ddf_msg(LVL_ERROR, "Failed unbinding %s", fun->fnode->name);
                         return rc;
                 }
@@ -712 +705 @@
 static void isa_init()
 {
-        ddf_log_init(NAME);
+        ddf_log_init(NAME, LVL_ERROR);
         isa_fun_ops.interfaces[HW_RES_DEV_IFACE] = &isa_fun_hw_res_ops;
 }