Changeset 4874c2d in mainline for kernel/genarch

Timestamp:

2006-10-17T20:32:35Z (19 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.

Location:

kernel/genarch/src/kbd

Files:

• ns16550.c (modified) (4 diffs)
• z8530.c (modified) (4 diffs)

kernel/genarch/src/kbd/ns16550.c

@@ -62 +62 @@
 static irq_t ns16550_irq;
 
-static ipc_notif_cfg_t saved_notif_cfg;
-
 /*
  * These codes read from ns16550 data register are silently ignored.
@@ -88 +86 @@
                 (void) ns16550_rbr_read(&ns16550);
 
-        if (ns16550_irq.notif_cfg.answerbox) {
-                saved_notif_cfg = ns16550_irq.notif_cfg;
-                ns16550_irq.notif_cfg.answerbox = NULL;
-                ns16550_irq.notif_cfg.code = NULL;
-                ns16550_irq.notif_cfg.method = 0;
-                ns16550_irq.notif_cfg.counter = 0;
-        }
+        ns16550_irq.notif_cfg.notify = false;
 }
 
@@ -100 +92 @@
 void ns16550_release(void)
 {
-        if (saved_notif_cfg.answerbox)
-                ns16550_irq.notif_cfg = saved_notif_cfg;
+        if (ns16550_irq.notif_cfg.answerbox)
+                ns16550_irq.notif_cfg.notify = true;
 }
 
@@ -184 +176 @@
 
         if (ns16550_lsr_read(&ns16550) & LSR_DATA_READY) {
-                if (ns16550_irq.notif_cfg.answerbox) {
+                if (ns16550_irq.notif_cfg.notify && ns16550_irq.notif_cfg.answerbox) {
                         /*
                          * Send IPC notification.

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