Commit ac657d58 authored by Mark Einon's avatar Mark Einon Committed by Greg Kroah-Hartman

staging: et131x: Update tx trans_start on device close to prevent tx_timeout

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: default avatarMark Einon <mark.einon@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent c8b7de7f
...@@ -153,7 +153,7 @@ int et131x_open(struct net_device *netdev) ...@@ -153,7 +153,7 @@ int et131x_open(struct net_device *netdev)
result = request_irq(netdev->irq, et131x_isr, IRQF_SHARED, result = request_irq(netdev->irq, et131x_isr, IRQF_SHARED,
netdev->name, netdev); netdev->name, netdev);
if (result) { 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); netdev->irq);
return result; return result;
} }
...@@ -182,6 +182,9 @@ int et131x_close(struct net_device *netdev) ...@@ -182,6 +182,9 @@ int et131x_close(struct net_device *netdev)
{ {
struct et131x_adapter *adapter = netdev_priv(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 */ /* First thing is to stop the queue */
netif_stop_queue(netdev); netif_stop_queue(netdev);
...@@ -449,6 +452,10 @@ void et131x_tx_timeout(struct net_device *netdev) ...@@ -449,6 +452,10 @@ void et131x_tx_timeout(struct net_device *netdev)
struct tcb *tcb; struct tcb *tcb;
unsigned long flags; unsigned long flags;
/* If the device is closed, ignore the timeout */
if (~(adapter->flags & fMP_ADAPTER_INTERRUPT_IN_USE));
return;
/* Any nonrecoverable hardware error? /* Any nonrecoverable hardware error?
* Checks adapter->flags for any failure in phy reading * Checks adapter->flags for any failure in phy reading
*/ */
...@@ -685,12 +692,12 @@ struct net_device *et131x_device_alloc(void) ...@@ -685,12 +692,12 @@ struct net_device *et131x_device_alloc(void)
return NULL; return NULL;
} }
/* Setup the function registration table (and other data) for a /*
* Setup the function registration table (and other data) for a
* net_device * net_device
*/ */
netdev->watchdog_timeo = ET131X_TX_TIMEOUT; netdev->watchdog_timeo = ET131X_TX_TIMEOUT;
netdev->netdev_ops = &et131x_netdev_ops; netdev->netdev_ops = &et131x_netdev_ops;
/* netdev->ethtool_ops = &et131x_ethtool_ops; */ /* netdev->ethtool_ops = &et131x_ethtool_ops; */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment