staging: et131x: Remove redundant replica loopback code
authorMark Einon <mark.einon@gmail.com>
Mon, 29 Aug 2011 17:42:39 +0000 (18:42 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Mon, 29 Aug 2011 18:20:10 +0000 (11:20 -0700)
A mechanism used to set the phy in loopback mode is not present in the driver, making associated checks and variables redundant. Removing them.

Signed-off-by: Mark Einon <mark.einon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/et131x/et1310_address_map.h
drivers/staging/et131x/et1310_rx.c
drivers/staging/et131x/et131x_adapter.h
drivers/staging/et131x/et131x_isr.c

index 410677ee22b7e402c705a60f3968a83e2600c393..38ec56c129878b458b82a7f248d1392680ab1277 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * Agere Systems Inc.
- * 10/100/1000 Base-T Ethernet Driver for the ET1301 and ET131x series MACs
+ * 10/100/1000 Base-T Ethernet Driver for the ET1310 and ET131x series MACs
  *
  * Copyright © 2005 Agere Systems Inc.
  * All rights reserved.
  * GLOBAL Module of JAGCore Address Mapping
  * Located at address 0x0000
  */
-struct global_regs {                   /* Location: */
+struct global_regs {                           /* Location: */
        u32 txq_start_addr;                     /*  0x0000 */
        u32 txq_end_addr;                       /*  0x0004 */
        u32 rxq_start_addr;                     /*  0x0008 */
@@ -163,7 +163,7 @@ struct global_regs {                        /* Location: */
        u32 sw_reset;                           /*  0x0028 */
        u32 slv_timer;                          /*  0x002C */
        u32 msi_config;                         /*  0x0030 */
-       u32 loopback;                   /*  0x0034 */
+       u32 loopback;                           /*  0x0034 */
        u32 watchdog_timer;                     /*  0x0038 */
 };
 
index e6458dbef934f24fa504a6b5c7140ca3f68a26c2..f50420c582a1f05243a6a4ff4c7acfe51945275f 100644 (file)
@@ -988,17 +988,6 @@ static struct rfd *nic_rx_pkts(struct et131x_adapter *adapter)
        }
 
        if (len) {
-               if (adapter->replica_phy_loopbk == 1) {
-                       buf = rx_local->fbr[ring_index]->virt[buff_index];
-
-                       if (memcmp(&buf[6], adapter->addr, ETH_ALEN) == 0) {
-                               if (memcmp(&buf[42], "Replica packet",
-                                          ETH_HLEN)) {
-                                       adapter->replica_phy_loopbk_passfail = 1;
-                               }
-                       }
-               }
-
                /* Determine if this is a multicast packet coming in */
                if ((word0 & ALCATEL_MULTICAST_PKT) &&
                    !(word0 & ALCATEL_BROADCAST_PKT)) {
index b03460bbb296f10090d5d5e2dffa79c502154569..508cc632ad8e7ef14c0c21eca3eddfeaee596f80 100644 (file)
@@ -222,10 +222,6 @@ struct et131x_adapter {
        /* Rx Memory Variables */
        struct rx_ring rx_ring;
 
-       /* Loopback specifics */
-       u8 replica_phy_loopbk;          /* Replica Enable */
-       u8 replica_phy_loopbk_passfail; /* Replica Enable Pass/Fail */
-
        /* Stats */
        struct ce_stats stats;
 
index 7cfd213fb136aaf8e96612163b75252f668ce2c9..4d75690e7af2862fda04bb396903fb42cdc13597 100644 (file)
@@ -389,17 +389,15 @@ void et131x_isr_handler(struct work_struct *work)
                                        (uint8_t) offsetof(struct mi_regs, isr),
                                        &myisr);
 
-                       if (!adapter->replica_phy_loopbk) {
-                               et131x_mii_read(adapter,
-                                      (uint8_t) offsetof(struct mi_regs, bmsr),
-                                      &bmsr_data);
+                       et131x_mii_read(adapter,
+                              (uint8_t) offsetof(struct mi_regs, bmsr),
+                              &bmsr_data);
 
-                               bmsr_ints = adapter->bmsr ^ bmsr_data;
-                               adapter->bmsr = bmsr_data;
+                       bmsr_ints = adapter->bmsr ^ bmsr_data;
+                       adapter->bmsr = bmsr_data;
 
-                               /* Do all the cable in / cable out stuff */
-                               et131x_mii_check(adapter, bmsr_data, bmsr_ints);
-                       }
+                       /* Do all the cable in / cable out stuff */
+                       et131x_mii_check(adapter, bmsr_data, bmsr_ints);
                }
 
                /* Let's move on to the TxMac */