Changeset 4874c2d in mainline for kernel/genarch/src/kbd/z8530.c

Timestamp:

2006-10-17T20:32:35Z (18 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

b14e35f2

Parents:

8513ad7

Message:

Do not copy notification config out of the IRQ structure.
Add the notify member, that toggles notifications on and off instead.
This is good for preventing other tasks from registering the notification
while the notification config is stored outside the IRQ structure.
It should also help to implement the cleanup code.

File:

• kernel/genarch/src/kbd/z8530.c (modified) (4 diffs)

kernel/genarch/src/kbd/z8530.c

@@ -63 +63 @@
 static irq_t z8530_irq;         /**< z8530's IRQ. */ 
 
-static ipc_notif_cfg_t saved_notif_cfg;
-
 static void z8530_suspend(chardev_t *);
 static void z8530_resume(chardev_t *);
@@ -92 +90 @@
         z8530_write_a(&z8530, WR9, WR9_MIE);            /* Master Interrupt Enable. */
         
-        if (z8530_irq.notif_cfg.answerbox) {
-                saved_notif_cfg = z8530_irq.notif_cfg;
-                z8530_irq.notif_cfg.answerbox = NULL;
-                z8530_irq.notif_cfg.code = NULL;
-                z8530_irq.notif_cfg.method = 0;
-                z8530_irq.notif_cfg.counter = 0;
-        }
+        z8530_irq.notif_cfg.notify = false;
 }
 
@@ -104 +96 @@
 void z8530_release(void)
 {
-        if (saved_notif_cfg.answerbox)
-                z8530_irq.notif_cfg = saved_notif_cfg;
+        if (z8530_irq.notif_cfg.answerbox)
+                z8530_irq.notif_cfg.notify = true;
 }
 
@@ -205 +197 @@
          * interrupt traps. Process the interrupt directly.
          */
-        if (irq->notif_cfg.answerbox)
+        if (irq->notif_cfg.notify && irq->notif_cfg.answerbox)
                 ipc_irq_send_notif(irq);
         else