bnxt_en: reverse order of TX disable and carrier off
authorEdwin Peer <edwin.peer@broadcom.com>
Thu, 11 Feb 2021 07:24:23 +0000 (02:24 -0500)
committerDavid S. Miller <davem@davemloft.net>
Thu, 11 Feb 2021 22:36:22 +0000 (14:36 -0800)
commit132e0b65dc2b8bfa9721bfce834191f24fd1d7ed
tree2e5d68e4d0fd65c79fa99dc48d688e9ac208c56e
parentd4083d3c00f60a09ad82e3bf17ff57fec69c8aa6
bnxt_en: reverse order of TX disable and carrier off

A TX queue can potentially immediately timeout after it is stopped
and the last TX timestamp on that queue was more than 5 seconds ago with
carrier still up.  Prevent these intermittent false TX timeouts
by bringing down carrier first before calling netif_tx_disable().

Fixes: c0c050c58d84 ("bnxt_en: New Broadcom ethernet driver.")
Signed-off-by: Edwin Peer <edwin.peer@broadcom.com>
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnxt/bnxt.c