[PATCH RESEND 3/3] staging: et131x: fix make W=1 build warning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



fix make W=1 warning:
drivers/staging/et131x/et131x.c: In function ‘et1310_setup_device_for_multicast’:
drivers/staging/et131x/et131x.c:1052:6: warning: variable ‘pm_csr’ set but not used [-Wunused-but-set-variable]
  u32 pm_csr;
      ^
drivers/staging/et131x/et131x.c: In function ‘et1310_setup_device_for_unicast’:
drivers/staging/et131x/et131x.c:1101:6: warning: variable ‘pm_csr’ set but not used [-Wunused-but-set-variable]
  u32 pm_csr;
      ^
drivers/staging/et131x/et131x.c: In function ‘et131x_isr_handler’:
drivers/staging/et131x/et131x.c:4006:8: warning: variable ‘pm_csr’ set but not used [-Wunused-but-set-variable]
    u32 pm_csr;
        ^

Signed-off-by: Zhao, Gang <gamerh2o@xxxxxxxxx>
---
 drivers/staging/et131x/et131x.c | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c
index 6413500..0c44014 100644
--- a/drivers/staging/et131x/et131x.c
+++ b/drivers/staging/et131x/et131x.c
@@ -1049,7 +1049,6 @@ static void et1310_setup_device_for_multicast(struct et131x_adapter *adapter)
 	u32 hash2 = 0;
 	u32 hash3 = 0;
 	u32 hash4 = 0;
-	u32 pm_csr;
 
 	/* If ET131X_PACKET_TYPE_MULTICAST is specified, then we provision
 	 * the multi-cast LIST.  If it is NOT specified, (and "ALL" is not
@@ -1083,7 +1082,6 @@ static void et1310_setup_device_for_multicast(struct et131x_adapter *adapter)
 	}
 
 	/* Write out the new hash to the device */
-	pm_csr = readl(&adapter->regs->global.pm_csr);
 	if (!et1310_in_phy_coma(adapter)) {
 		writel(hash1, &rxmac->multi_hash1);
 		writel(hash2, &rxmac->multi_hash2);
@@ -1098,7 +1096,6 @@ static void et1310_setup_device_for_unicast(struct et131x_adapter *adapter)
 	u32 uni_pf1;
 	u32 uni_pf2;
 	u32 uni_pf3;
-	u32 pm_csr;
 
 	/* Set up unicast packet filter reg 3 to be the first two octets of
 	 * the MAC address for both address
@@ -1124,7 +1121,6 @@ static void et1310_setup_device_for_unicast(struct et131x_adapter *adapter)
 		  (adapter->addr[4] << ET_RX_UNI_PF_ADDR1_5_SHIFT) |
 		   adapter->addr[5];
 
-	pm_csr = readl(&adapter->regs->global.pm_csr);
 	if (!et1310_in_phy_coma(adapter)) {
 		writel(uni_pf1, &rxmac->uni_pf_addr1);
 		writel(uni_pf2, &rxmac->uni_pf_addr2);
@@ -4003,12 +3999,9 @@ static void et131x_isr_handler(struct work_struct *work)
 		 */
 		if (adapter->flowcontrol == FLOW_TXONLY ||
 		    adapter->flowcontrol == FLOW_BOTH) {
-			u32 pm_csr;
-
 			/* Tell the device to send a pause packet via the back
 			 * pressure register (bp req and bp xon/xoff)
 			 */
-			pm_csr = readl(&iomem->global.pm_csr);
 			if (!et1310_in_phy_coma(adapter))
 				writel(3, &iomem->txmac.bp_ctrl);
 		}
-- 
1.8.5.3

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel





[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux