Skip to contentSkip to conversation

iwlwifi: add the MVM driver

DISCUSSION Dan Carpenter 2 participants 2 messages May 3, 2016

lore
Share & tools
lore
#0 lore raw

Hello Johannes Berg,

The patch 8ca151b568b6: "iwlwifi: add the MVM driver" from Jan 24, 2013, leads to the following static checker warning:

	drivers/net/wireless/intel/iwlwifi/mvm/d3.c:2080 __iwl_mvm_resume()
	error: uninitialized symbol 'd3_status'.

drivers/net/wireless/intel/iwlwifi/mvm/d3.c
2057 static int __iwl_mvm_resume(struct iwl_mvm *mvm, bool test) 2058 {

  2059          struct ieee80211_vif *vif = NULL;
  2060          int ret = 1;
  2061          enum iwl_d3_status d3_status;
  2062          bool keep = false;
  2063          bool unified_image = fw_has_capa(&mvm->fw->ucode_capa,
  2064                                           IWL_UCODE_TLV_CAPA_CNSLDTD_D3_D0_IMG);
  2065  
  2066          u32 flags = CMD_ASYNC | CMD_HIGH_PRIO | CMD_SEND_IN_IDLE |
  2067                                      CMD_WAKE_UP_TRANS;
  2068  
  2069          mutex_lock(&mvm->mutex);
  2070  
  2071          /* get the BSS vif pointer again */
  2072          vif = iwl_mvm_get_bss_vif(mvm);
  2073          if (IS_ERR_OR_NULL(vif))

2074 goto err;
2075
2076 ret = iwl_trans_d3_resume(mvm->trans, &d3_status, test, !unified_image);

iwl_trans_d3_resume() just calls a resume() function pointer, but if that pointer is NULL then it's a no-op.

2077 if (ret)
2078 goto err;
2079

  2080          if (d3_status != IWL_D3_STATUS_ALIVE) {
                    ^^^^^^^^^

This would still be uninitialized.

  2081                  IWL_INFO(mvm, "Device was reset during suspend\n");
  2082                  goto err;
  2083          }

regards,
dan carpenter

#1 lore raw
Quoted 2 lines · Dan Carpenter

The patch 8ca151b568b6: "iwlwifi: add the MVM driver" from Jan 24, 2013, leads to the following static checker warning:

Struggling to keep up with the backlog? :)

Bjørn