diff options
author | Stephen Hemminger <shemminger@osdl.org> | 2006-06-13 04:17:28 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-06-13 16:16:41 -0400 |
commit | f05267e7dee58741a4feb20d0351706ec64bb0b5 (patch) | |
tree | b99907cae2801309761a1c89bbd57e642c3aff24 | |
parent | 2ccc99b7b71976d15822ae7c41cd2ccda66d5076 (diff) |
[PATCH] sky2: don't hard code number of ports
It is cleaner, to not loop over both ports if only one exists.
Signed-off-by: Stephen Hemminger <shemminger@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | drivers/net/sky2.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/sky2.c b/drivers/net/sky2.c index 765c8f063512..6ad676d2cbc1 100644 --- a/drivers/net/sky2.c +++ b/drivers/net/sky2.c | |||
@@ -3430,7 +3430,7 @@ static int sky2_suspend(struct pci_dev *pdev, pm_message_t state) | |||
3430 | if (!(pstate == PCI_D3hot || pstate == PCI_D3cold)) | 3430 | if (!(pstate == PCI_D3hot || pstate == PCI_D3cold)) |
3431 | return -EINVAL; | 3431 | return -EINVAL; |
3432 | 3432 | ||
3433 | for (i = 0; i < 2; i++) { | 3433 | for (i = 0; i < hw->ports; i++) { |
3434 | struct net_device *dev = hw->dev[i]; | 3434 | struct net_device *dev = hw->dev[i]; |
3435 | 3435 | ||
3436 | if (dev) { | 3436 | if (dev) { |
@@ -3460,7 +3460,7 @@ static int sky2_resume(struct pci_dev *pdev) | |||
3460 | if (err) | 3460 | if (err) |
3461 | goto out; | 3461 | goto out; |
3462 | 3462 | ||
3463 | for (i = 0; i < 2; i++) { | 3463 | for (i = 0; i < hw->ports; i++) { |
3464 | struct net_device *dev = hw->dev[i]; | 3464 | struct net_device *dev = hw->dev[i]; |
3465 | if (dev && netif_running(dev)) { | 3465 | if (dev && netif_running(dev)) { |
3466 | netif_device_attach(dev); | 3466 | netif_device_attach(dev); |