Staging: ipack/bridges/tpci200: Store the irq holder in slot_irq.
authorJens Taprogge <jens.taprogge@taprogge.org>
Tue, 11 Sep 2012 11:35:13 +0000 (13:35 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 11 Sep 2012 19:18:12 +0000 (12:18 -0700)
This way we do no longer need to keep a dangling pointer to struct
ipack_device in tpci200_slot after the device has been removed.

Signed-off-by: Jens Taprogge <jens.taprogge@taprogge.org>
Signed-off-by: Samuel Iglesias Gonsálvez <siglesias@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ipack/bridges/tpci200.c
drivers/staging/ipack/bridges/tpci200.h

index 5aff0d07e564d5918764a3d5352901d6da67cc18..9d886b7fdfc603dcc1b13b885751e9c496b41303 100644 (file)
@@ -89,6 +89,7 @@ static irqreturn_t tpci200_interrupt(int irq, void *dev_id)
        unsigned short status_reg, reg_value;
        unsigned short unhandled_ints = 0;
        irqreturn_t ret = IRQ_NONE;
+       struct slot_irq *slot_irq;
 
        /* Read status register */
        status_reg = readw(&tpci200->info->interface_regs->status);
@@ -97,15 +98,17 @@ static irqreturn_t tpci200_interrupt(int irq, void *dev_id)
                unhandled_ints = status_reg & TPCI200_SLOT_INT_MASK;
                /* callback to the IRQ handler for the corresponding slot */
                for (i = 0; i < TPCI200_NB_SLOT; i++) {
-                       if ((tpci200->slots[i].irq != NULL) &&
+                       slot_irq = tpci200->slots[i].irq;
+
+                       if ((slot_irq != NULL) &&
                            (status_reg & ((TPCI200_A_INT0 | TPCI200_A_INT1) << (2*i)))) {
 
-                               ret = tpci200->slots[i].irq->handler(tpci200->slots[i].irq->arg);
+                               ret = slot_irq->handler(slot_irq->arg);
 
                                /* Dummy reads */
-                               readw(tpci200->slots[i].dev->io_space.address +
+                               readw(slot_irq->holder->io_space.address +
                                      0xC0);
-                               readw(tpci200->slots[i].dev->io_space.address +
+                               readw(slot_irq->holder->io_space.address +
                                      0xC2);
 
                                unhandled_ints &= ~(((TPCI200_A_INT0 | TPCI200_A_INT1) << (2*i)));
@@ -115,8 +118,10 @@ static irqreturn_t tpci200_interrupt(int irq, void *dev_id)
        /* Interrupts not handled are disabled */
        if (unhandled_ints) {
                for (i = 0; i < TPCI200_NB_SLOT; i++) {
+                       slot_irq = tpci200->slots[i].irq;
+
                        if (unhandled_ints & ((TPCI200_INT0_EN | TPCI200_INT1_EN) << (2*i))) {
-                               dev_info(&tpci200->slots[i].dev->dev,
+                               dev_info(&slot_irq->holder->dev,
                                         "No registered ISR for slot [%d:%d]!. IRQ will be disabled.\n",
                                         tpci200->number, i);
                                reg_value = readw(
@@ -487,6 +492,7 @@ static int tpci200_request_irq(struct ipack_device *dev, int vector,
        slot_irq->vector = vector;
        slot_irq->handler = handler;
        slot_irq->arg = arg;
+       slot_irq->holder = dev;
 
        tpci200->slots[dev->slot].irq = slot_irq;
        res = __tpci200_request_irq(tpci200, dev);
@@ -701,8 +707,7 @@ static int tpci200_pci_probe(struct pci_dev *pdev,
         * The TPCI200 has assigned his own two IRQ by PCI bus driver
         */
        for (i = 0; i < TPCI200_NB_SLOT; i++)
-               tpci200->slots[i].dev =
-                       ipack_device_register(tpci200->info->ipack_bus, i, i);
+               ipack_device_register(tpci200->info->ipack_bus, i, i);
        return 0;
 
 out_err_bus_register:
index fc90a94f31aa2c3d5b12b7dfc5eb049657622c11..75a5dccca992344444571e981fe958a036680421 100644 (file)
@@ -121,6 +121,7 @@ struct tpci200_regs {
  *
  */
 struct slot_irq {
+       struct ipack_device *holder;
        int             vector;
        int             (*handler)(void *);
        void            *arg;
@@ -136,7 +137,6 @@ struct slot_irq {
  *
  */
 struct tpci200_slot {
-       struct ipack_device     *dev;
        struct slot_irq         *irq;
        struct ipack_addr_space io_phys;
        struct ipack_addr_space id_phys;