aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLinus Torvalds <torvalds@g5.osdl.org>2006-06-12 15:53:27 -0400
committerLinus Torvalds <torvalds@g5.osdl.org>2006-06-12 15:53:27 -0400
commitd374c1c1281d6188a0d0676172b1c0e3de35c6e7 (patch)
treeaeae46b8a329faa0ad7e51b8a1900c1d937f9836
parent0e838b72d54ed189033939258a961f2a0cd59647 (diff)
[sky2] Fix sky2 network driver suspend/resume
This fixes two independent problems: it would not save the PCI state on suspend (and thus try to resume a nonexistent state on resume), and while shut off, if an interrupt happened on the same shared irq, the irq handler would react very badly to the interrupt status being an invalid all-ones state. Acked-by: Jeff Garzik <jgarzik@pobox.com> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r--drivers/net/sky2.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/net/sky2.c b/drivers/net/sky2.c
index 959109609d85..6b87c7a5c906 100644
--- a/drivers/net/sky2.c
+++ b/drivers/net/sky2.c
@@ -2183,6 +2183,9 @@ static int sky2_poll(struct net_device *dev0, int *budget)
2183 int work_done = 0; 2183 int work_done = 0;
2184 u32 status = sky2_read32(hw, B0_Y2_SP_EISR); 2184 u32 status = sky2_read32(hw, B0_Y2_SP_EISR);
2185 2185
2186 if (!~status)
2187 return 0;
2188
2186 if (status & Y2_IS_HW_ERR) 2189 if (status & Y2_IS_HW_ERR)
2187 sky2_hw_intr(hw); 2190 sky2_hw_intr(hw);
2188 2191
@@ -3438,6 +3441,7 @@ static int sky2_suspend(struct pci_dev *pdev, pm_message_t state)
3438 } 3441 }
3439 } 3442 }
3440 3443
3444 pci_save_state(pdev);
3441 return sky2_set_power_state(hw, pci_choose_state(pdev, state)); 3445 return sky2_set_power_state(hw, pci_choose_state(pdev, state));
3442} 3446}
3443 3447