hi, guinan On 7/21/2020 3:39 PM, Guinan Sun wrote:
Check return value of running DNL admin command.
What is "DNL", I am not sure i know the reason it appear here, could you double check it?
Signed-off-by: Adam Ludkiewicz <adam.ludkiew...@intel.com> Signed-off-by: Guinan Sun <guinanx....@intel.com> --- 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;
