diff options
author | Michael Chan <michael.chan@broadcom.com> | 2019-02-21 01:07:31 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-02-23 00:16:56 +0100 |
commit | 67681d02aaa1db9044a16df4ca9c77cde1221a3e (patch) | |
tree | e09897c5f69147cda43aaeb2b5fcde31871a35db /drivers/net/ethernet/broadcom | |
parent | Merge tag 'mac80211-for-davem-2019-02-22' of git://git.kernel.org/pub/scm/lin... (diff) | |
download | linux-67681d02aaa1db9044a16df4ca9c77cde1221a3e.tar.xz linux-67681d02aaa1db9044a16df4ca9c77cde1221a3e.zip |
bnxt_en: Fix typo in firmware message timeout logic.
The logic that polls for the firmware message response uses a shorter
sleep interval for the first few passes. But there was a typo so it
was using the wrong counter (larger counter) for these short sleep
passes. The result is a slightly shorter timeout period for these
firmware messages than intended. Fix it by using the proper counter.
Fixes: 9751e8e71487 ("bnxt_en: reduce timeout on initial HWRM calls")
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/broadcom')
-rw-r--r-- | drivers/net/ethernet/broadcom/bnxt/bnxt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt.c b/drivers/net/ethernet/broadcom/bnxt/bnxt.c index 8bc7e495b027..1ddd6721d7cd 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.c @@ -3903,7 +3903,7 @@ static int bnxt_hwrm_do_send_msg(struct bnxt *bp, void *msg, u32 msg_len, if (len) break; /* on first few passes, just barely sleep */ - if (i < DFLT_HWRM_CMD_TIMEOUT) + if (i < HWRM_SHORT_TIMEOUT_COUNTER) usleep_range(HWRM_SHORT_MIN_TIMEOUT, HWRM_SHORT_MAX_TIMEOUT); else |