diff options
author | Stephen Hemminger <shemminger@linux-foundation.org> | 2007-10-14 16:25:22 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-14 19:46:44 -0400 |
commit | 5c0d6b34d6ee11ff979cbdac9a59b47a74cb9f43 (patch) | |
tree | 8e7068459852029148be16ea237acdb4815caa34 | |
parent | 4fa435018d740cb83d74c92306aa1f796da91ddd (diff) |
sky2: reboot fix
The call to napi_disable() in the PCI shutdown handler is problematic,
and is aggravated by the new NAPI.
Also, make sure watchdog timer doesn't go off.
Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | drivers/net/sky2.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/sky2.c b/drivers/net/sky2.c index 68f728f0b600..7967240534d5 100644 --- a/drivers/net/sky2.c +++ b/drivers/net/sky2.c | |||
@@ -4396,7 +4396,7 @@ static void sky2_shutdown(struct pci_dev *pdev) | |||
4396 | if (!hw) | 4396 | if (!hw) |
4397 | return; | 4397 | return; |
4398 | 4398 | ||
4399 | napi_disable(&hw->napi); | 4399 | del_timer_sync(&hw->watchdog_timer); |
4400 | 4400 | ||
4401 | for (i = 0; i < hw->ports; i++) { | 4401 | for (i = 0; i < hw->ports; i++) { |
4402 | struct net_device *dev = hw->dev[i]; | 4402 | struct net_device *dev = hw->dev[i]; |