sky2: remove unneede workaround
authorStephen Hemminger <shemminger@vyatta.com>
Tue, 3 Feb 2009 11:27:27 +0000 (11:27 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 3 Feb 2009 23:08:34 +0000 (15:08 -0800)
This workaround is not needed. It was inherited from sk98lin driver but only
applies to an early development version of the chip that is not supported
by sky2.  The workaround required an unnecessary pci read which hurts performance

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/sky2.c

index 994703cc0db32c87229f7754f1a246c1cc44d9ea..db925085c185761cb4abfe795478c42ceb73e196 100644 (file)
@@ -2687,13 +2687,6 @@ static int sky2_poll(struct napi_struct *napi, int work_limit)
                        goto done;
        }
 
-       /* Bug/Errata workaround?
-        * Need to kick the TX irq moderation timer.
-        */
-       if (sky2_read8(hw, STAT_TX_TIMER_CTRL) == TIM_START) {
-               sky2_write8(hw, STAT_TX_TIMER_CTRL, TIM_STOP);
-               sky2_write8(hw, STAT_TX_TIMER_CTRL, TIM_START);
-       }
        napi_complete(napi);
        sky2_read32(hw, B0_Y2_SP_LISR);
 done: