diff options
| author | Michael Chan <mchan@broadcom.com> | 2005-12-13 21:09:54 -0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2005-12-13 21:09:54 -0800 |
| commit | 16fe9d74f14ed74af778c5db7f9129e29916f4a7 (patch) | |
| tree | 909a2b173d5140b22df9f35a252dc204dba56bdd | |
| parent | 6a9eba15f51c56da637e45ea1316eaa2a848986a (diff) | |
| download | linux-3.10-16fe9d74f14ed74af778c5db7f9129e29916f4a7.tar.gz linux-3.10-16fe9d74f14ed74af778c5db7f9129e29916f4a7.tar.bz2 linux-3.10-16fe9d74f14ed74af778c5db7f9129e29916f4a7.zip | |
[TG3]: Fix 5704 single-port mode
If the dual-port 5704 is configured as a single-port device with
only one PCI function, it would trigger a BUG() condition in
tg3_find_5704_peer(). This fixes the problem by returning its
own pdev if the peer cannot be found.
Signed-off-by: Michael Chan <mchan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/tg3.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/drivers/net/tg3.c b/drivers/net/tg3.c index 6b7ab4b7275..a143c18c7bc 100644 --- a/drivers/net/tg3.c +++ b/drivers/net/tg3.c @@ -10443,8 +10443,13 @@ static struct pci_dev * __devinit tg3_find_5704_peer(struct tg3 *tp) break; pci_dev_put(peer); } - if (!peer || peer == tp->pdev) - BUG(); + /* 5704 can be configured in single-port mode, set peer to + * tp->pdev in that case. + */ + if (!peer) { + peer = tp->pdev; + return peer; + } /* * We don't need to keep the refcount elevated; there's no way |