diff options
author | Dan Williams <dcbw@redhat.com> | 2007-04-25 21:30:29 -0400 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-04-27 20:16:41 -0400 |
commit | 1764f15016fea54db723a96234a82646dac9a036 (patch) | |
tree | efed9a20fb0ad0d9c9483bea8b4049d461503d9c | |
parent | f00546363fff1576ceddc2690d47e5f9c1dd2e05 (diff) |
usb-net/pegasus: simplify carrier detection
Simplify pegasus carrier detection; rely only on the periodic MII
polling. Reverts pieces of c43c49bd61fdb9bb085ddafcaadb17d06f95ec43.
Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
-rw-r--r-- | drivers/usb/net/pegasus.c | 10 | ||||
-rw-r--r-- | drivers/usb/net/pegasus.h | 3 |
2 files changed, 1 insertions, 12 deletions
diff --git a/drivers/usb/net/pegasus.c b/drivers/usb/net/pegasus.c index 1ad4ee54b186..a05fd97e5bc2 100644 --- a/drivers/usb/net/pegasus.c +++ b/drivers/usb/net/pegasus.c | |||
@@ -847,16 +847,6 @@ static void intr_callback(struct urb *urb) | |||
847 | * d[0].NO_CARRIER kicks in only with failed TX. | 847 | * d[0].NO_CARRIER kicks in only with failed TX. |
848 | * ... so monitoring with MII may be safest. | 848 | * ... so monitoring with MII may be safest. |
849 | */ | 849 | */ |
850 | if (pegasus->features & TRUST_LINK_STATUS) { | ||
851 | if (d[5] & LINK_STATUS) | ||
852 | netif_carrier_on(net); | ||
853 | else | ||
854 | netif_carrier_off(net); | ||
855 | } else { | ||
856 | /* Never set carrier _on_ based on ! NO_CARRIER */ | ||
857 | if (d[0] & NO_CARRIER) | ||
858 | netif_carrier_off(net); | ||
859 | } | ||
860 | 850 | ||
861 | /* bytes 3-4 == rx_lostpkt, reg 2E/2F */ | 851 | /* bytes 3-4 == rx_lostpkt, reg 2E/2F */ |
862 | pegasus->stats.rx_missed_errors += ((d[3] & 0x7f) << 8) | d[4]; | 852 | pegasus->stats.rx_missed_errors += ((d[3] & 0x7f) << 8) | d[4]; |
diff --git a/drivers/usb/net/pegasus.h b/drivers/usb/net/pegasus.h index c7aadb413e8c..c7467823cd1c 100644 --- a/drivers/usb/net/pegasus.h +++ b/drivers/usb/net/pegasus.h | |||
@@ -11,7 +11,6 @@ | |||
11 | 11 | ||
12 | #define PEGASUS_II 0x80000000 | 12 | #define PEGASUS_II 0x80000000 |
13 | #define HAS_HOME_PNA 0x40000000 | 13 | #define HAS_HOME_PNA 0x40000000 |
14 | #define TRUST_LINK_STATUS 0x20000000 | ||
15 | 14 | ||
16 | #define PEGASUS_MTU 1536 | 15 | #define PEGASUS_MTU 1536 |
17 | #define RX_SKBS 4 | 16 | #define RX_SKBS 4 |
@@ -204,7 +203,7 @@ PEGASUS_DEV( "AEI USB Fast Ethernet Adapter", VENDOR_AEILAB, 0x1701, | |||
204 | PEGASUS_DEV( "Allied Telesyn Int. AT-USB100", VENDOR_ALLIEDTEL, 0xb100, | 203 | PEGASUS_DEV( "Allied Telesyn Int. AT-USB100", VENDOR_ALLIEDTEL, 0xb100, |
205 | DEFAULT_GPIO_RESET | PEGASUS_II ) | 204 | DEFAULT_GPIO_RESET | PEGASUS_II ) |
206 | PEGASUS_DEV( "Belkin F5D5050 USB Ethernet", VENDOR_BELKIN, 0x0121, | 205 | PEGASUS_DEV( "Belkin F5D5050 USB Ethernet", VENDOR_BELKIN, 0x0121, |
207 | DEFAULT_GPIO_RESET | PEGASUS_II | TRUST_LINK_STATUS ) | 206 | DEFAULT_GPIO_RESET | PEGASUS_II ) |
208 | PEGASUS_DEV( "Billionton USB-100", VENDOR_BILLIONTON, 0x0986, | 207 | PEGASUS_DEV( "Billionton USB-100", VENDOR_BILLIONTON, 0x0986, |
209 | DEFAULT_GPIO_RESET ) | 208 | DEFAULT_GPIO_RESET ) |
210 | PEGASUS_DEV( "Billionton USBLP-100", VENDOR_BILLIONTON, 0x0987, | 209 | PEGASUS_DEV( "Billionton USBLP-100", VENDOR_BILLIONTON, 0x0987, |