staging: et131x: Update tx trans_start on device close to prevent tx_timeout
authorMark Einon <mark.einon@gmail.com>
Tue, 23 Aug 2011 22:40:40 +0000 (23:40 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 23 Aug 2011 23:25:29 +0000 (16:25 -0700)
Updating the tx trans_start before closing the device prevents a possible tx_timeout occuring during the closing process.
Tested on an ET1310 device.

Signed-off-by: Mark Einon <mark.einon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/et131x/et131x_netdev.c

index 70a0da3fb52d901e6fad20d0394c765e4ea7b6d4..7555a241ed234159558068b032e9deaea711b3a7 100644 (file)
@@ -153,7 +153,7 @@ int et131x_open(struct net_device *netdev)
        result = request_irq(netdev->irq, et131x_isr, IRQF_SHARED,
                                        netdev->name, netdev);
        if (result) {
-               dev_err(&adapter->pdev->dev, "c ould not register IRQ %d\n",
+               dev_err(&adapter->pdev->dev, "could not register IRQ %d\n",
                        netdev->irq);
                return result;
        }
@@ -182,6 +182,9 @@ int et131x_close(struct net_device *netdev)
 {
        struct et131x_adapter *adapter = netdev_priv(netdev);
 
+       /* Save the timestamp for the TX watchdog, prevent a timeout */
+       netdev->trans_start = jiffies;
+
        /* First thing is to stop the queue */
        netif_stop_queue(netdev);
 
@@ -449,6 +452,10 @@ void et131x_tx_timeout(struct net_device *netdev)
        struct tcb *tcb;
        unsigned long flags;
 
+       /* If the device is closed, ignore the timeout */
+       if (~(adapter->flags & fMP_ADAPTER_INTERRUPT_IN_USE));
+               return;
+
        /* Any nonrecoverable hardware error?
         * Checks adapter->flags for any failure in phy reading
         */
@@ -685,12 +692,12 @@ struct net_device *et131x_device_alloc(void)
                return NULL;
        }
 
-       /* Setup the function registration table (and other data) for a
+       /*
+        * Setup the function registration table (and other data) for a
         * net_device
         */
-
        netdev->watchdog_timeo = ET131X_TX_TIMEOUT;
-       netdev->netdev_ops = &et131x_netdev_ops;
+       netdev->netdev_ops     = &et131x_netdev_ops;
 
        /* netdev->ethtool_ops        = &et131x_ethtool_ops; */