ipoctal: request_irq after configuration
authorFederico Vaga <federico.vaga@cern.ch>
Thu, 3 Jul 2014 08:53:58 +0000 (10:53 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 10 Jul 2014 00:02:15 +0000 (17:02 -0700)
The request for an IRQ handler must be done after whole configuration. This
was not the case for this driver which request the IRQ in the middle of
the configuration. Sometimes, it happens that something is not completely
configured, we recieve an interrupt thus we stumble into troubles in the
IRQ handler.

Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
Acked-by: Samuel Iglesias Gonsalvez <siglesias@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/ipack/devices/ipoctal.c

index 69687f156999cd49d83bd1ccd9fa5d9ac345685a..e41bef048c23ca4100b48811e5acbffbd2caa6e0 100644 (file)
@@ -323,13 +323,6 @@ static int ipoctal_inst_slot(struct ipoctal *ipoctal, unsigned int bus_nr,
                         &block_regs[i].w.imr);
        }
 
-       /*
-        * IP-OCTAL has different addresses to copy its IRQ vector.
-        * Depending of the carrier these addresses are accesible or not.
-        * More info in the datasheet.
-        */
-       ipoctal->dev->bus->ops->request_irq(ipoctal->dev,
-                                      ipoctal_irq_handler, ipoctal);
        /* Dummy write */
        iowrite8(1, ipoctal->mem8_space + 1);
 
@@ -390,6 +383,14 @@ static int ipoctal_inst_slot(struct ipoctal *ipoctal, unsigned int bus_nr,
                dev_set_drvdata(tty_dev, channel);
        }
 
+       /*
+        * IP-OCTAL has different addresses to copy its IRQ vector.
+        * Depending of the carrier these addresses are accesible or not.
+        * More info in the datasheet.
+        */
+       ipoctal->dev->bus->ops->request_irq(ipoctal->dev,
+                                      ipoctal_irq_handler, ipoctal);
+
        return 0;
 }