V4L/DVB (13740): [Mantis] Schedule the work instead of handling the task directly
authorManu Abraham <abraham.manu@gmail.com>
Fri, 4 Dec 2009 08:09:04 +0000 (05:09 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sun, 17 Jan 2010 13:55:34 +0000 (11:55 -0200)
Signed-off-by: Manu Abraham <manu@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/dvb/mantis/mantis_common.h
drivers/media/dvb/mantis/mantis_evm.c
drivers/media/dvb/mantis/mantis_link.h
drivers/media/dvb/mantis/mantis_pci.c

index f9891ebcfb0ddc4a35ea2e4f295b4131a3aec5a3..7f1cfad74883977cde61d0157aaa2a366288c41b 100644 (file)
@@ -151,4 +151,8 @@ extern int mantis_dvb_exit(struct mantis_pci *mantis);
 extern void mantis_dma_xfer(unsigned long data);
 extern void gpio_set_bits(struct mantis_pci *mantis, u32 bitpos, u8 value);
 
+extern struct dvb_device *mantis_ca_init(struct mantis_pci *mantis);
+extern void mantis_ca_exit(struct mantis_pci *mantis);
+
+
 #endif //__MANTIS_COMMON_H
index c35ddd868f892600d8cd2c4bb7f0b59878e1179f..745148399981f21a6d17a4296b2f62ba3c35a39e 100644 (file)
@@ -22,9 +22,9 @@
 #include "mantis_link.h"
 #include "mantis_hif.h"
 
-void mantis_hifevm_tasklet(unsigned long data)
+static void mantis_hifevm_work(struct work_struct *work)
 {
-       struct mantis_ca *ca = (struct mantis_ca *) data;
+       struct mantis_ca *ca = container_of(work, struct mantis_ca, hif_evm_work);
        struct mantis_pci *mantis = ca->ca_priv;
 
        u32 gpif_stat, gpif_mask;
@@ -38,15 +38,13 @@ void mantis_hifevm_tasklet(unsigned long data)
                if (gpif_stat & MANTIS_CARD_PLUGIN) {
                        dprintk(verbose, MANTIS_DEBUG, 1, "Event Mgr: Adapter(%d) Slot(0): CAM Plugin", mantis->num);
                        mmwrite(0xdada0000, MANTIS_CARD_RESET);
-                       // Plugin call here
-                       gpif_stat = 0; // crude !
+                       mantis_event_cam_plugin(ca);
                }
        } else {
                if (gpif_stat & MANTIS_CARD_PLUGOUT) {
                        dprintk(verbose, MANTIS_DEBUG, 1, "Event Mgr: Adapter(%d) Slot(0): CAM Unplug", mantis->num);
                        mmwrite(0xdada0000, MANTIS_CARD_RESET);
-                       // Unplug call here
-                       gpif_stat = 0; // crude !
+                       mantis_event_cam_unplug(ca);
                }
        }
 
@@ -91,9 +89,9 @@ int mantis_evmgr_init(struct mantis_ca *ca)
        struct mantis_pci *mantis = ca->ca_priv;
 
        dprintk(verbose, MANTIS_DEBUG, 1, "Initializing Mantis Host I/F Event manager");
-       tasklet_init(&ca->hif_evm_tasklet, mantis_hifevm_tasklet, (unsigned long) ca);
-
+       INIT_WORK(&ca->hif_evm_work, mantis_hifevm_work);
        mantis_pcmcia_init(ca);
+       schedule_work(&ca->hif_evm_work);
 
        return 0;
 }
@@ -103,7 +101,6 @@ void mantis_evmgr_exit(struct mantis_ca *ca)
        struct mantis_pci *mantis = ca->ca_priv;
 
        dprintk(verbose, MANTIS_DEBUG, 1, "Mantis Host I/F Event manager exiting");
-       tasklet_kill(&ca->hif_evm_tasklet);
-
+       flush_scheduled_work();
        mantis_pcmcia_exit(ca);
 }
index a45dd7185e06bc96450645781e180bc6ff90a919..33b39b781d2eb7d619f1d2e7c70378e9d7355c89 100644 (file)
@@ -21,6 +21,8 @@
 #ifndef __MANTIS_LINK_H
 #define __MANTIS_LINK_H
 
+#include <linux/workqueue.h>
+
 enum mantis_sbuf_status {
        MANTIS_SBUF_DATA_AVAIL          = 1,
        MANTIS_SBUF_DATA_EMPTY          = 2,
@@ -40,7 +42,7 @@ enum mantis_slot_state {
 struct mantis_ca {
        struct mantis_slot              slot;
 
-       struct tasklet_struct           hif_evm_tasklet;
+       struct work_struct              hif_evm_work;
 
        u32                             hif_event;
        wait_queue_head_t               hif_opdone_wq;
index 814c6ace3d3765f1ea4c5166ee07d6e2d37ee6bd..6eecd54a44570684b9c3166678df193a712c49ac 100644 (file)
@@ -77,7 +77,7 @@ static irqreturn_t mantis_pci_irq(int irq, void *dev_id)
        }
        if (stat & MANTIS_INT_IRQ0) {
                dprintk(verbose, MANTIS_DEBUG, 0, "* INT IRQ-0 *");
-               tasklet_schedule(&ca->hif_evm_tasklet);
+               schedule_work(&ca->hif_evm_work);
        }
        if (stat & MANTIS_INT_IRQ1) {
                dprintk(verbose, MANTIS_DEBUG, 0, "* INT IRQ-1 *");