Changes in uspace/lib/drv/generic/driver.c [8a637a4:8820544] in mainline

File:

• uspace/lib/drv/generic/driver.c (modified) (7 diffs)

uspace/lib/drv/generic/driver.c

@@ -53 +53 @@
 #include <devman.h>
 
+#include <ipc/driver.h>
+
 #include "dev_iface.h"
 #include "ddf/driver.h"
@@ -128 +130 @@
         
         char *dev_name = NULL;
-        int rc = async_data_write_accept((void **) &dev_name, true, 0, 0, 0, 0);
-        if (rc != EOK) {
-                async_answer_0(iid, rc);
-                return;
-        }
-        
+        async_data_write_accept((void **) &dev_name, true, 0, 0, 0, 0);
         dev->name = dev_name;
         
@@ -281 +278 @@
 }
 
-static void driver_connection_devman(ipc_callid_t iid, ipc_call_t *icall,
-    void *arg)
+static void driver_connection_devman(ipc_callid_t iid, ipc_call_t *icall)
 {
         /* Accept connection */
@@ -440 +436 @@
 }
 
-static void driver_connection_driver(ipc_callid_t iid, ipc_call_t *icall,
-    void *arg)
+static void driver_connection_driver(ipc_callid_t iid, ipc_call_t *icall)
 {
         driver_connection_gen(iid, icall, true);
 }
 
-static void driver_connection_client(ipc_callid_t iid, ipc_call_t *icall,
-    void *arg)
+static void driver_connection_client(ipc_callid_t iid, ipc_call_t *icall)
 {
         driver_connection_gen(iid, icall, false);
+}
+
+/** Function for handling connections to device driver. */
+static void driver_connection(ipc_callid_t iid, ipc_call_t *icall, void *arg)
+{
+        sysarg_t conn_type;
+
+        if (iid == 0) {
+                /* Callback connection from devman */
+                /* XXX Use separate handler for this type of connection */
+                conn_type = DRIVER_DEVMAN;
+        } else {
+                conn_type = IPC_GET_ARG1(*icall);
+        }
+
+        /* Select interface */
+        switch (conn_type) {
+        case DRIVER_DEVMAN:
+                /* Handle request from device manager */
+                driver_connection_devman(iid, icall);
+                break;
+        case DRIVER_DRIVER:
+                /* Handle request from drivers of child devices */
+                driver_connection_driver(iid, icall);
+                break;
+        case DRIVER_CLIENT:
+                /* Handle request from client applications */
+                driver_connection_client(iid, icall);
+                break;
+        default:
+                /* No such interface */
+                async_answer_0(iid, ENOENT);
+        }
 }
 
@@ -593 +620 @@
  * The session will be automatically closed when @a dev is destroyed.
  *
- * @param dev Device
- *
- * @return New session or NULL if session could not be created
- *
- */
-async_sess_t *ddf_dev_parent_sess_create(ddf_dev_t *dev)
+ * @param dev   Device
+ * @param mgmt  Exchange management style
+ * @return      New session or NULL if session could not be created
+ */
+async_sess_t *ddf_dev_parent_sess_create(ddf_dev_t *dev, exch_mgmt_t mgmt)
 {
         assert(dev->parent_sess == NULL);
-        dev->parent_sess = devman_parent_device_connect(dev->handle,
+        dev->parent_sess = devman_parent_device_connect(mgmt, dev->handle,
             IPC_FLAG_BLOCKING);
 
@@ -890 +916 @@
  * This allows handling connections the non-devman way.
  */
-void ddf_fun_set_conn_handler(ddf_fun_t *fun, async_port_handler_t conn)
+void ddf_fun_set_conn_handler(ddf_fun_t *fun, async_client_conn_t conn)
 {
         assert(fun->ops == NULL);
@@ -929 +955 @@
          * incoming connections.
          */
-        port_id_t port;
-        int rc = async_create_port(INTERFACE_DDF_DRIVER, driver_connection_driver,
-            NULL, &port);
-        if (rc != EOK)
-                return rc;
-        
-        rc = async_create_port(INTERFACE_DDF_DEVMAN, driver_connection_devman,
-            NULL, &port);
-        if (rc != EOK)
-                return rc;
-        
-        async_set_fallback_port_handler(driver_connection_client, NULL);
-        
-        rc = devman_driver_register(driver->name);
+        async_set_client_connection(driver_connection);
+        int rc = devman_driver_register(driver->name);
         if (rc != EOK) {
                 printf("Error: Failed to register driver with device manager "
-                    "(%s).\n", (rc == EEXIST) ? "driver already started" :
+                    "(%s).\n", (rc == EEXISTS) ? "driver already started" :
                     str_error(rc));