igb: Fix error that repeat to add filter when changing adapter speed

From: Gangfeng Huang <[email protected]>

Function igb_nfc_filter_exit()/igb_nfc_filter_restore() should be paried, igb_nfc_filter_restore exist in igb_up(), but igb_nfc_filter_exit() exist in __igb_close(), so move the igb_nfc_filter_exit() to igb_down().

---
 drivers/net/ethernet/intel/igb/igb_main.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

index 8f624af..e01e01a 100644
@@ -1791,6 +1791,8 @@ void igb_down(struct igb_adapter *adapter)
 	wr32(E1000_RCTL, rctl & ~E1000_RCTL_EN);
 	/* flush and sleep below */
 
+	igb_nfc_filter_exit(adapter);
+
 	netif_carrier_off(netdev);
 	netif_tx_stop_all_queues(netdev);
 

@@ -3317,8 +3319,6 @@ static int __igb_close(struct net_device *netdev, bool suspending)
 	igb_down(adapter);
 	igb_free_irq(adapter);
 
-	igb_nfc_filter_exit(adapter);
-
 	igb_free_all_tx_resources(adapter);
 	igb_free_all_rx_resources(adapter);
 

2.7.2

Hi, thanks for your patch. Are you fixing a bug here? If so, your commit message should say what you're fixing and how to reproduce it. Say why you are doing something, not just what you're doing.

In this case, it looks like you probably are fixing a bug, but definitely need a better commit message.