linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* re: iwlwifi: mvm: set the TX disable bit when doing a chanctx switch
@ 2014-09-09  9:40 Dan Carpenter
  2014-09-11  7:04 ` Coelho, Luciano
  0 siblings, 1 reply; 2+ messages in thread
From: Dan Carpenter @ 2014-09-09  9:40 UTC (permalink / raw)
  To: luciano.coelho; +Cc: linux-wireless

Hello Luciano Coelho,

The patch 0ce04ce797f8: "iwlwifi: mvm: set the TX disable bit when
doing a chanctx switch" from May 8, 2014, leads to the following
static checker warning:

	drivers/net/wireless/iwlwifi/mvm/mac80211.c:2768 __iwl_mvm_assign_vif_chanctx()
	warn: missing error code here? 'iwl_mvm_sta_from_staid_protected()' failed. 'ret' = '0'

drivers/net/wireless/iwlwifi/mvm/mac80211.c
  2761          if (vif->csa_active && vif->type == NL80211_IFTYPE_STATION) {
  2762                  struct iwl_mvm_sta *mvmsta;
  2763  
  2764                  mvmsta = iwl_mvm_sta_from_staid_protected(mvm,
  2765                                                            mvmvif->ap_sta_id);
  2766  
  2767                  if (WARN_ON(!mvmsta))
  2768                          goto out;

Did you want to set an error code here?  I don't know the code well
enough to say.

  2769  
  2770                  /* TODO: only re-enable after the first beacon */
  2771                  iwl_mvm_sta_modify_disable_tx(mvm, mvmsta, false);
  2772          }
  2773  
  2774          goto out;
  2775  
  2776  out_remove_binding:
  2777          iwl_mvm_binding_remove_vif(mvm, vif);
  2778          iwl_mvm_power_update_mac(mvm);
  2779  out:
  2780          if (ret)
  2781                  mvmvif->phy_ctxt = NULL;
  2782          return ret;
  2783  }

regards,
dan carpenter

^ permalink raw reply	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-09-11  7:05 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-09  9:40 iwlwifi: mvm: set the TX disable bit when doing a chanctx switch Dan Carpenter
2014-09-11  7:04 ` Coelho, Luciano

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).