[patch 17/35] net: sun3lance: Remove redundant checks in interrupt handler

[Thomas Gleixner]

From: Sebastian Andrzej Siewior <bige...@linutronix.de>

lance_interrupt() contains two pointless checks:

 - A check whether the 'dev_id' argument is NULL. 'dev_id' is the pointer
   which was handed in to request_irq() and the interrupt handler will
   always be invoked with that pointer as 'dev_id' argument by the core
   code.

 - A check for interrupt reentrancy. The core code already guarantees
   non-reentrancy of interrupt handlers.

Remove these check.

Signed-off-by: Sebastian Andrzej Siewior <bige...@linutronix.de>
Signed-off-by: Thomas Gleixner <t...@linutronix.de>
Cc: "David S. Miller" <da...@davemloft.net>
Cc: Jakub Kicinski <k...@kernel.org>
Cc: net...@vger.kernel.org
---
 drivers/net/ethernet/amd/sun3lance.c |   11 -----------
 1 file changed, 11 deletions(-)

--- a/drivers/net/ethernet/amd/sun3lance.c
+++ b/drivers/net/ethernet/amd/sun3lance.c
@@ -657,16 +657,6 @@ static irqreturn_t lance_interrupt( int
        struct net_device *dev = dev_id;
        struct lance_private *lp = netdev_priv(dev);
        int csr0;
-       static int in_interrupt;
-
-       if (dev == NULL) {
-               DPRINTK( 1, ( "lance_interrupt(): invalid dev_id\n" ));
-               return IRQ_NONE;
-       }
-
-       if (in_interrupt)
-               DPRINTK( 2, ( "%s: Re-entering the interrupt handler.\n", 
dev->name ));
-       in_interrupt = 1;
 
  still_more:
        flush_cache_all();
@@ -774,7 +764,6 @@ static irqreturn_t lance_interrupt( int
 
        DPRINTK( 2, ( "%s: exiting interrupt, csr0=%#04x.\n",
                                  dev->name, DREG ));
-       in_interrupt = 0;
        return IRQ_HANDLED;
 }