Changeset 56fd7cf in mainline for uspace/drv/infrastructure/rootmac/rootmac.c

Timestamp:

2012-08-17T11:37:03Z (12 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

1d5a540

Parents:

be2a38ad

Message:

Make ddf_dev_t and ddf_fun_t opaque. This further tighthens the DDF interface.

File:

• uspace/drv/infrastructure/rootmac/rootmac.c (modified) (5 diffs)

uspace/drv/infrastructure/rootmac/rootmac.c

@@ -44 +44 @@
 #define NAME  "rootmac"
 
-/** Obtain function soft-state from DDF function node */
-#define ROOTMAC_FUN(fnode) \
-        ((rootmac_fun_t *) (fnode)->driver_data)
-
 typedef struct {
         hw_resource_list_t hw_resources;
@@ -80 +76 @@
 static ddf_dev_ops_t rootmac_fun_ops;
 
+/** Obtain function soft-state from DDF function node */
+static rootmac_fun_t *rootmac_fun(ddf_fun_t *fnode)
+{
+        return ddf_fun_data_get(fnode);
+}
+
 static bool rootmac_add_fun(ddf_dev_t *dev, const char *name,
-    const char *str_match_id, rootmac_fun_t *fun)
+    const char *str_match_id, rootmac_fun_t *fun_proto)
 {
         ddf_msg(LVL_DEBUG, "Adding new function '%s'.", name);
         
         ddf_fun_t *fnode = NULL;
-        match_id_t *match_id = NULL;
+        int rc;
         
         /* Create new device. */
@@ -93 +95 @@
                 goto failure;
         
-        fnode->driver_data = fun;
+        rootmac_fun_t *fun = ddf_fun_data_alloc(fnode, sizeof(rootmac_fun_t));
+        *fun = *fun_proto;
         
-        /* Initialize match id list */
-        match_id = create_match_id();
-        if (match_id == NULL)
+        /* Add match ID */
+        rc = ddf_fun_add_match_id(fnode, str_match_id, 100);
+        if (rc != EOK)
                 goto failure;
         
-        match_id->id = str_match_id;
-        match_id->score = 100;
-        add_match_id(&fnode->match_ids, match_id);
-        
         /* Set provided operations to the device. */
-        fnode->ops = &rootmac_fun_ops;
+        ddf_fun_set_ops(fnode, &rootmac_fun_ops);
         
         /* Register function. */
@@ -116 +115 @@
         
 failure:
-        if (match_id != NULL)
-                match_id->id = NULL;
-        
         if (fnode != NULL)
                 ddf_fun_destroy(fnode);
@@ -162 +158 @@
 static hw_resource_list_t *rootmac_get_resources(ddf_fun_t *fnode)
 {
-        rootmac_fun_t *fun = ROOTMAC_FUN(fnode);
+        rootmac_fun_t *fun = rootmac_fun(fnode);
         assert(fun != NULL);