staging: fsl-dpaa2/eth: Use generic irq handler
authorIoana Radulescu <ruxandra.radulescu@nxp.com>
Fri, 23 Mar 2018 13:44:05 +0000 (08:44 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 23 Mar 2018 15:01:56 +0000 (16:01 +0100)
For the link state interrupt, we used a dummy non-threaded
irq handler, which had the same implementation as the generic
irq_default_primary_handler() function.

Give up on using our own irq handler and let the kernel use
the generic one instead.

Signed-off-by: Ioana Radulescu <ruxandra.radulescu@nxp.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c

index ac1a7504266457b1b1d6c7363db81c66eb26b531..beb59593a6b71da58ee67f11be760d74f41ed569 100644 (file)
@@ -2317,11 +2317,6 @@ static int poll_link_state(void *arg)
        return 0;
 }
 
-static irqreturn_t dpni_irq0_handler(int irq_num, void *arg)
-{
-       return IRQ_WAKE_THREAD;
-}
-
 static irqreturn_t dpni_irq0_handler_thread(int irq_num, void *arg)
 {
        u32 status = ~0;
@@ -2356,8 +2351,7 @@ static int setup_irqs(struct fsl_mc_device *ls_dev)
 
        irq = ls_dev->irqs[0];
        err = devm_request_threaded_irq(&ls_dev->dev, irq->msi_desc->irq,
-                                       dpni_irq0_handler,
-                                       dpni_irq0_handler_thread,
+                                       NULL, dpni_irq0_handler_thread,
                                        IRQF_NO_SUSPEND | IRQF_ONESHOT,
                                        dev_name(&ls_dev->dev), &ls_dev->dev);
        if (err < 0) {