Changeset 92574f4 in mainline for uspace/drv/rootpc/rootpc.c

Timestamp:

2011-02-24T12:03:27Z (13 years ago)

Author:

Lubos Slovak <lubos.slovak@…>

Branches:

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

Children:

e7b7ebd5

Parents:

4837092 (diff), a80849c (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merged development (changes in DDF, etc.).

Conflicts in uspace/drv/usbkbd/main.c

File:

• uspace/drv/rootpc/rootpc.c (modified) (8 diffs)

uspace/drv/rootpc/rootpc.c

@@ -46 +46 @@
 #include <macros.h>
 
-#include <driver.h>
+#include <ddf/driver.h>
 #include <devman.h>
 #include <ipc/devman.h>
@@ -55 +55 @@
 #define NAME "rootpc"
 
-typedef struct rootpc_child_dev_data {
+/** Obtain function soft-state from DDF function node */
+#define ROOTPC_FUN(fnode) ((rootpc_fun_t *) (fnode)->driver_data)
+
+typedef struct rootpc_fun {
         hw_resource_list_t hw_resources;
-} rootpc_child_dev_data_t;
-
-static int rootpc_add_device(device_t *dev);
+} rootpc_fun_t;
+
+static int rootpc_add_device(ddf_dev_t *dev);
 static void root_pc_init(void);
 
@@ -82 +85 @@
 };
 
-static rootpc_child_dev_data_t pci_data = {
+static rootpc_fun_t pci_data = {
         .hw_resources = {
                 1,
@@ -89 +92 @@
 };
 
-static hw_resource_list_t *rootpc_get_child_resources(device_t *dev)
-{
-        rootpc_child_dev_data_t *data;
-        
-        data = (rootpc_child_dev_data_t *) dev->driver_data;
-        if (NULL == data)
-                return NULL;
-        
-        return &data->hw_resources;
-}
-
-static bool rootpc_enable_child_interrupt(device_t *dev)
+static hw_resource_list_t *rootpc_get_resources(ddf_fun_t *fnode)
+{
+        rootpc_fun_t *fun = ROOTPC_FUN(fnode);
+        
+        assert(fun != NULL);
+        return &fun->hw_resources;
+}
+
+static bool rootpc_enable_interrupt(ddf_fun_t *fun)
 {
         /* TODO */
@@ -107 +107 @@
 }
 
-static hw_res_ops_t child_hw_res_ops = {
-        &rootpc_get_child_resources,
-        &rootpc_enable_child_interrupt
+static hw_res_ops_t fun_hw_res_ops = {
+        &rootpc_get_resources,
+        &rootpc_enable_interrupt
 };
 
 /* Initialized in root_pc_init() function. */
-static device_ops_t rootpc_child_ops;
+static ddf_dev_ops_t rootpc_fun_ops;
 
 static bool
-rootpc_add_child(device_t *parent, const char *name, const char *str_match_id,
-    rootpc_child_dev_data_t *drv_data)
-{
-        printf(NAME ": adding new child device '%s'.\n", name);
-        
-        device_t *child = NULL;
+rootpc_add_fun(ddf_dev_t *dev, const char *name, const char *str_match_id,
+    rootpc_fun_t *fun)
+{
+        printf(NAME ": adding new function '%s'.\n", name);
+        
+        ddf_fun_t *fnode = NULL;
         match_id_t *match_id = NULL;
         
         /* Create new device. */
-        child = create_device();
-        if (NULL == child)
+        fnode = ddf_fun_create(dev, fun_inner, name);
+        if (fnode == NULL)
                 goto failure;
         
-        child->name = name;
-        child->driver_data = drv_data;
+        fnode->driver_data = fun;
         
         /* Initialize match id list */
         match_id = create_match_id();
-        if (NULL == match_id)
+        if (match_id == NULL)
                 goto failure;
         
         match_id->id = str_match_id;
         match_id->score = 100;
-        add_match_id(&child->match_ids, match_id);
+        add_match_id(&fnode->match_ids, match_id);
         
         /* Set provided operations to the device. */
-        child->ops = &rootpc_child_ops;
-        
-        /* Register child device. */
-        if (EOK != child_device_register(child, parent))
+        fnode->ops = &rootpc_fun_ops;
+        
+        /* Register function. */
+        if (ddf_fun_bind(fnode) != EOK) {
+                printf(NAME ": error binding function %s.\n", name);
                 goto failure;
+        }
         
         return true;
         
 failure:
-        if (NULL != match_id)
+        if (match_id != NULL)
                 match_id->id = NULL;
         
-        if (NULL != child) {
-                child->name = NULL;
-                delete_device(child);
-        }
-        
-        printf(NAME ": failed to add child device '%s'.\n", name);
+        if (fnode != NULL)
+                ddf_fun_destroy(fnode);
+        
+        printf(NAME ": failed to add function '%s'.\n", name);
         
         return false;
 }
 
-static bool rootpc_add_children(device_t *dev)
-{
-        return rootpc_add_child(dev, "pci0", "intel_pci", &pci_data);
+static bool rootpc_add_functions(ddf_dev_t *dev)
+{
+        return rootpc_add_fun(dev, "pci0", "intel_pci", &pci_data);
 }
 
@@ -175 +174 @@
  * @return              Zero on success, negative error number otherwise.
  */
-static int rootpc_add_device(device_t *dev)
+static int rootpc_add_device(ddf_dev_t *dev)
 {
         printf(NAME ": rootpc_add_device, device handle = %d\n",
             (int)dev->handle);
         
-        /* Register child devices. */
-        if (!rootpc_add_children(dev)) {
-                printf(NAME ": failed to add child devices for PC platform.\n");
+        /* Register functions. */
+        if (!rootpc_add_functions(dev)) {
+                printf(NAME ": failed to add functions for PC platform.\n");
         }
         
@@ -190 +189 @@
 static void root_pc_init(void)
 {
-        rootpc_child_ops.interfaces[HW_RES_DEV_IFACE] = &child_hw_res_ops;
+        rootpc_fun_ops.interfaces[HW_RES_DEV_IFACE] = &fun_hw_res_ops;
 }
 
@@ -197 +196 @@
         printf(NAME ": HelenOS PC platform driver\n");
         root_pc_init();
-        return driver_main(&rootpc_driver);
+        return ddf_driver_main(&rootpc_driver);
 }