bnxt_en: Fix typo in firmware message timeout logic.
authorMichael Chan <michael.chan@broadcom.com>
Thu, 21 Feb 2019 00:07:31 +0000 (19:07 -0500)
committerDavid S. Miller <davem@davemloft.net>
Fri, 22 Feb 2019 23:16:56 +0000 (15:16 -0800)
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>
drivers/net/ethernet/broadcom/bnxt/bnxt.c

index 8bc7e495b027083942e6963c56bed62a25fd6f5d..1ddd6721d7cdfdf7aff6bca49c1b1a4e422393e8 100644 (file)
@@ -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