Changeset 79ae36dd in mainline for uspace/drv/ns8250/ns8250.c

Timestamp:

2011-06-08T19:01:55Z (13 years ago)

Author:

Martin Decky <martin@…>

Branches:

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

Children:

0eff68e

Parents:

764d71e

Message:

new async framework with integrated exchange tracking

• strict isolation between low-level IPC and high-level async framework with integrated exchange tracking
  • each IPC connection is represented by an async_sess_t structure
  • each IPC exchange is represented by an async_exch_t structure
  • exchange management is either based on atomic messages (EXCHANGE_ATOMIC), locking (EXCHANGE_SERIALIZE) or connection cloning (EXCHANGE_CLONE)
• async_obsolete: temporary compatibility layer to keep old async clients working (several pieces of code are currently broken, but only non-essential functionality)
• IPC_M_PHONE_HANGUP is now method no. 0 (for elegant boolean evaluation)
• IPC_M_DEBUG_ALL has been renamed to IPC_M_DEBUG
• IPC_M_PING has been removed (VFS protocol now has VFS_IN_PING)
• console routines in libc have been rewritten for better abstraction
• additional use for libc-private header files (FILE structure opaque to the client)
• various cstyle changes (typos, indentation, missing externs in header files, improved comments, etc.)

File:

• uspace/drv/ns8250/ns8250.c (modified) (2 diffs)

uspace/drv/ns8250/ns8250.c

@@ -263 +263 @@
 static void ns8250_dev_cleanup(ns8250_t *ns)
 {
-        if (ns->dev->parent_phone > 0) {
-                async_hangup(ns->dev->parent_phone);
-                ns->dev->parent_phone = 0;
+        if (ns->dev->parent_sess) {
+                async_hangup(ns->dev->parent_sess);
+                ns->dev->parent_sess = NULL;
         }
 }
@@ -337 +337 @@
         
         /* Connect to the parent's driver. */
-        ns->dev->parent_phone = devman_parent_device_connect(ns->dev->handle,
-            IPC_FLAG_BLOCKING);
-        if (ns->dev->parent_phone < 0) {
+        ns->dev->parent_sess = devman_parent_device_connect(EXCHANGE_SERIALIZE,
+            ns->dev->handle, IPC_FLAG_BLOCKING);
+        if (!ns->dev->parent_sess) {
                 ddf_msg(LVL_ERROR, "Failed to connect to parent driver of "
                     "device %s.", ns->dev->name);
-                ret = ns->dev->parent_phone;
+                ret = ENOENT;
                 goto failed;
         }
         
         /* Get hw resources. */
-        ret = hw_res_get_resource_list(ns->dev->parent_phone, &hw_resources);
+        ret = hw_res_get_resource_list(ns->dev->parent_sess, &hw_resources);
         if (ret != EOK) {
                 ddf_msg(LVL_ERROR, "Failed to get HW resources for device "