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
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to