Changeset 7edfde9 in mainline for uspace/lib

Timestamp:

2015-05-13T02:22:50Z (10 years ago)

Author:

Martin Decky <martin@…>

Branches:

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

Children:

0683992

Parents:

6d81e1e

Message:

fix the device driver regressions in a generic way
therefore the previous fix (revision 2329) can be reverted
the root cause of the regression was a forgotten fallback to default_ucode that disappeared in the refactoring of revision 2170 (mea culpa)

Location:

uspace/lib

Files:

• c/generic/irq.c (modified) (2 diffs)
• drv/generic/interrupt.c (modified) (1 diff)

uspace/lib/c/generic/irq.c

@@ -35 +35 @@
 #include <ipc/irq.h>
 #include <libc.h>
+#include <stdlib.h>
+#include <macros.h>
+
+static irq_cmd_t default_cmds[] = {
+        {
+                .cmd = CMD_ACCEPT
+        }
+};
+
+static const irq_code_t default_ucode = {
+        0,
+        NULL,
+        ARRAY_SIZE(default_cmds),
+        default_cmds
+};
 
 /** Subscribe to IRQ notification.
@@ -49 +64 @@
     const irq_code_t *ucode)
 {
+        if (ucode == NULL)
+                ucode = &default_ucode;
+        
         return __SYSCALL4(SYS_IPC_IRQ_SUBSCRIBE, inr, devno, method,
             (sysarg_t) ucode);

uspace/lib/drv/generic/interrupt.c

@@ -44 +44 @@
 #include "private/driver.h"
 
-static irq_cmd_t default_cmds[] = {
-        {
-                .cmd = CMD_ACCEPT
-        }
-};
-
-static const irq_code_t default_pseudocode = {
-        0,
-        NULL,
-        ARRAY_SIZE(default_cmds),
-        default_cmds
-};
-
 int register_interrupt_handler(ddf_dev_t *dev, int irq,
     interrupt_handler_t *handler, const irq_code_t *pseudocode)