diff options
author | Luciano Coelho | 2014-11-10 10:10:10 +0100 |
---|---|---|
committer | Emmanuel Grumbach | 2014-11-24 07:30:28 +0100 |
commit | 6fd1fb63abeb78fd7fe30f08f478732ad601702c (patch) | |
tree | 89b4e6cd7395272c0ab72d3aafb2d77a9cb4908f /drivers/net | |
parent | iwlwifi: mvm: add support for CHANCTX_SWMODE_REASSIGN_VIF (diff) | |
download | kernel-qcow2-linux-6fd1fb63abeb78fd7fe30f08f478732ad601702c.tar.gz kernel-qcow2-linux-6fd1fb63abeb78fd7fe30f08f478732ad601702c.tar.xz kernel-qcow2-linux-6fd1fb63abeb78fd7fe30f08f478732ad601702c.zip |
iwlwifi: mvm: return the actual error code when switch_vif_chanctx fails
We have code to recover and go back to the original channel context if
something fails in the middle of switch_vif_chanctx, but we return the
error code of the recover calls instead of the original code, so if
the recovery succeeds, we will return 0 (success). Fix this by not
assigning the return value of the recovery calls to ret.
Signed-off-by: Luciano Coelho <luciano.coelho@intel.com>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/wireless/iwlwifi/mvm/mac80211.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/drivers/net/wireless/iwlwifi/mvm/mac80211.c b/drivers/net/wireless/iwlwifi/mvm/mac80211.c index 542e59f61d1c..1d556173fcb6 100644 --- a/drivers/net/wireless/iwlwifi/mvm/mac80211.c +++ b/drivers/net/wireless/iwlwifi/mvm/mac80211.c @@ -2988,15 +2988,13 @@ out_remove: __iwl_mvm_remove_chanctx(mvm, vifs[0].new_ctx); out_reassign: - ret = __iwl_mvm_add_chanctx(mvm, vifs[0].old_ctx); - if (ret) { + if (__iwl_mvm_add_chanctx(mvm, vifs[0].old_ctx)) { IWL_ERR(mvm, "failed to add old_ctx back after failure.\n"); goto out_restart; } - ret = __iwl_mvm_assign_vif_chanctx(mvm, vifs[0].vif, vifs[0].old_ctx, - true); - if (ret) { + if (__iwl_mvm_assign_vif_chanctx(mvm, vifs[0].vif, vifs[0].old_ctx, + true)) { IWL_ERR(mvm, "failed to reassign old_ctx after failure.\n"); goto out_restart; } |