Changes in uspace/drv/audio/sb16/main.c [cccd60c3:f9b2cb4c] in mainline

File:

• uspace/drv/audio/sb16/main.c (modified) (9 diffs)

uspace/drv/audio/sb16/main.c

@@ -37 +37 @@
 #include <ddf/log.h>
 #include <device/hw_res_parsed.h>
+#include <devman.h>
 #include <assert.h>
 #include <stdio.h>
@@ -50 +51 @@
 static int sb_get_res(ddf_dev_t *device, addr_range_t **pp_sb_regs,
     addr_range_t **pp_mpu_regs, int *irq, int *dma8, int *dma16);
-static int sb_enable_interrupt(ddf_dev_t *device, int irq);
+static int sb_enable_interrupts(ddf_dev_t *device);
 
 static driver_ops_t sb_driver_ops = {
@@ -93 +94 @@
         irq_cmd_t irq_cmds[irq_cmd_count];
         irq_pio_range_t irq_ranges[1];
-        int irq_cap;
 
         sb16_t *soft_state = ddf_dev_data_alloc(device, sizeof(sb16_t));
@@ -123 +123 @@
         };
 
-        irq_cap = register_interrupt_handler(device, irq, irq_handler,
-            &irq_code);
-        if (irq_cap < 0) {
-                rc = irq_cap;
+        rc = register_interrupt_handler(device, irq, irq_handler, &irq_code);
+        if (rc != EOK) {
                 ddf_log_error("Failed to register irq handler: %s.",
                     str_error(rc));
@@ -134 +132 @@
         handler_regd = true;
 
-        rc = sb_enable_interrupt(device, irq);
+        rc = sb_enable_interrupts(device);
         if (rc != EOK) {
                 ddf_log_error("Failed to enable interrupts: %s.",
@@ -170 +168 @@
 error:
         if (handler_regd)
-                unregister_interrupt_handler(device, irq_cap);
+                unregister_interrupt_handler(device, irq);
         return rc;
 }
@@ -179 +177 @@
         assert(device);
 
-        async_sess_t *parent_sess = ddf_dev_parent_sess_get(device);
-        if (parent_sess == NULL)
+        async_sess_t *parent_sess = devman_parent_device_connect(
+            ddf_dev_get_handle(device), IPC_FLAG_BLOCKING);
+        if (!parent_sess)
                 return ENOMEM;
 
@@ -186 +185 @@
         hw_res_list_parsed_init(&hw_res);
         const int ret = hw_res_get_list_parsed(parent_sess, &hw_res, 0);
+        async_hangup(parent_sess);
         if (ret != EOK) {
                 return ret;
@@ -242 +242 @@
 }
 
-static int sb_enable_interrupt(ddf_dev_t *device, int irq)
-{
-        async_sess_t *parent_sess = ddf_dev_parent_sess_get(device);
-        if (parent_sess == NULL)
+int sb_enable_interrupts(ddf_dev_t *device)
+{
+        async_sess_t *parent_sess = devman_parent_device_connect(
+            ddf_dev_get_handle(device), IPC_FLAG_BLOCKING);
+        if (!parent_sess)
                 return ENOMEM;
 
-        return hw_res_enable_interrupt(parent_sess, irq);
+        bool enabled = hw_res_enable_interrupt(parent_sess);
+        async_hangup(parent_sess);
+
+        return enabled ? EOK : EIO;
 }