Acked-by: Jeff Guo<jia....@intel.com> On 7/27/2020 1:34 PM, Guinan Sun wrote:
Fix possible uninitialized variable in i40e in the i40e_get_lpi_counters function.Fixes: 429bdc0cd967 ("net/i40e/base: add function to read LPI counters") Cc: sta...@dpdk.org Signed-off-by: Adam Ludkiewicz <adam.ludkiew...@intel.com> Signed-off-by: Guinan Sun <guinanx....@intel.com> --- V2: * Modify commit message. --- drivers/net/i40e/base/i40e_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/i40e/base/i40e_common.c b/drivers/net/i40e/base/i40e_common.c index 46a0b7881..85c22849e 100644 --- a/drivers/net/i40e/base/i40e_common.c +++ b/drivers/net/i40e/base/i40e_common.c @@ -7097,7 +7097,7 @@ enum i40e_status_code i40e_get_lpi_counters(struct i40e_hw *hw, I40E_AQ_RUN_PHY_ACT_DNL_OPCODE_GET_EEE_STAT, &cmd_status, tx_counter, rx_counter, NULL);- if (cmd_status != I40E_AQ_RUN_PHY_ACT_CMD_STAT_SUCC)+ if (!retval && cmd_status != I40E_AQ_RUN_PHY_ACT_CMD_STAT_SUCC) retval = I40E_ERR_ADMIN_QUEUE_ERROR;return retval;
