Commit 8a1e918d authored by Arkadiusz Kubalewski's avatar Arkadiusz Kubalewski Committed by Tony Nguyen

i40e: Fix sparse warning: missing error code 'err'

Set proper return values inside error checking if-statements.

Previously following warning was produced when compiling against sparse.
i40e_main.c:15162 i40e_init_recovery_mode() warn: missing error code 'err'

Fixes: 4ff0ee1a ("i40e: Introduce recovery mode support")
Signed-off-by: default avatarAleksandr Loktionov <aleksandr.loktionov@intel.com>
Signed-off-by: default avatarArkadiusz Kubalewski <arkadiusz.kubalewski@intel.com>
Tested-by: default avatarDave Switzer <david.switzer@intel.com>
Signed-off-by: default avatarTony Nguyen <anthony.l.nguyen@intel.com>
parent 6b5674fe
...@@ -15138,12 +15138,16 @@ static int i40e_init_recovery_mode(struct i40e_pf *pf, struct i40e_hw *hw) ...@@ -15138,12 +15138,16 @@ static int i40e_init_recovery_mode(struct i40e_pf *pf, struct i40e_hw *hw)
* in order to register the netdev * in order to register the netdev
*/ */
v_idx = i40e_vsi_mem_alloc(pf, I40E_VSI_MAIN); v_idx = i40e_vsi_mem_alloc(pf, I40E_VSI_MAIN);
if (v_idx < 0) if (v_idx < 0) {
err = v_idx;
goto err_switch_setup; goto err_switch_setup;
}
pf->lan_vsi = v_idx; pf->lan_vsi = v_idx;
vsi = pf->vsi[v_idx]; vsi = pf->vsi[v_idx];
if (!vsi) if (!vsi) {
err = -EFAULT;
goto err_switch_setup; goto err_switch_setup;
}
vsi->alloc_queue_pairs = 1; vsi->alloc_queue_pairs = 1;
err = i40e_config_netdev(vsi); err = i40e_config_netdev(vsi);
if (err) if (err)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment