From: Michael Chan <michael.chan@broadcom.com>

Date: Thu, 26 Oct 2017 11:51:24 -0400

Subject: bnxt_en: Optimize .ndo_set_mac_address() for VFs.

Patch-mainline: v4.15-rc1

Git-commit: c1a7bdff17247332ecff7f243e42d269b3f74c65

References: bsc#1050242 FATE#322914

No need to call bnxt_approve_mac() which will send a message to the

PF if the MAC address hasn't changed.

Signed-off-by: Michael Chan <michael.chan@broadcom.com>

Signed-off-by: David S. Miller <davem@davemloft.net>

Acked-by: Thomas Bogendoerfer <tbogendoerfer@suse.de>

---

 drivers/net/ethernet/broadcom/bnxt/bnxt.c |    6 +++---

 1 file changed, 3 insertions(+), 3 deletions(-)

--- a/drivers/net/ethernet/broadcom/bnxt/bnxt.c

+++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.c

@@ -7251,13 +7251,13 @@ static int bnxt_change_mac_addr(struct n

 	if (!is_valid_ether_addr(addr->sa_data))

 		return -EADDRNOTAVAIL;

+	if (ether_addr_equal(addr->sa_data, dev->dev_addr))

+		return 0;

+

 	rc = bnxt_approve_mac(bp, addr->sa_data);

 	if (rc)

 		return rc;

-	if (ether_addr_equal(addr->sa_data, dev->dev_addr))

-		return 0;

-

 	memcpy(dev->dev_addr, addr->sa_data, dev->addr_len);

 	if (netif_running(dev)) {

 		bnxt_close_nic(bp, false, false);