Commit 56f2929b authored by Sara Sharon's avatar Sara Sharon Committed by Luca Coelho

iwlwifi: mvm: cleanup redundant no_power_up_nic_in_init config

This is never really used anymore.
Signed-off-by: default avatarSara Sharon <sara.sharon@intel.com>
Signed-off-by: default avatarLuca Coelho <luciano.coelho@intel.com>
parent 0979a913
...@@ -359,7 +359,6 @@ struct iwl_cfg { ...@@ -359,7 +359,6 @@ struct iwl_cfg {
high_temp:1, high_temp:1,
mac_addr_from_csr:1, mac_addr_from_csr:1,
lp_xtal_workaround:1, lp_xtal_workaround:1,
no_power_up_nic_in_init:1,
disable_dummy_notification:1, disable_dummy_notification:1,
apmg_not_supported:1, apmg_not_supported:1,
mq_rx_supported:1, mq_rx_supported:1,
......
...@@ -712,37 +712,21 @@ iwl_op_mode_mvm_start(struct iwl_trans *trans, const struct iwl_cfg *cfg, ...@@ -712,37 +712,21 @@ iwl_op_mode_mvm_start(struct iwl_trans *trans, const struct iwl_cfg *cfg,
IWL_DEBUG_EEPROM(mvm->trans->dev, IWL_DEBUG_EEPROM(mvm->trans->dev,
"working without external nvm file\n"); "working without external nvm file\n");
if (WARN(cfg->no_power_up_nic_in_init && !mvm->nvm_file_name, err = iwl_trans_start_hw(mvm->trans);
"not allowing power-up and not having nvm_file\n")) if (err)
goto out_free; goto out_free;
/* mutex_lock(&mvm->mutex);
* Even if nvm exists in the nvm_file driver should read again the nvm iwl_mvm_ref(mvm, IWL_MVM_REF_INIT_UCODE);
* from the nic because there might be entries that exist in the OTP err = iwl_run_init_mvm_ucode(mvm, true);
* and not in the file. if (!err || !iwlmvm_mod_params.init_dbg)
* for nics with no_power_up_nic_in_init: rely completley on nvm_file iwl_mvm_stop_device(mvm);
*/ iwl_mvm_unref(mvm, IWL_MVM_REF_INIT_UCODE);
if (cfg->no_power_up_nic_in_init && mvm->nvm_file_name) { mutex_unlock(&mvm->mutex);
err = iwl_nvm_init(mvm, false); /* returns 0 if successful, 1 if success but in rfkill */
if (err) if (err < 0 && !iwlmvm_mod_params.init_dbg) {
goto out_free; IWL_ERR(mvm, "Failed to run INIT ucode: %d\n", err);
} else { goto out_free;
err = iwl_trans_start_hw(mvm->trans);
if (err)
goto out_free;
mutex_lock(&mvm->mutex);
iwl_mvm_ref(mvm, IWL_MVM_REF_INIT_UCODE);
err = iwl_run_init_mvm_ucode(mvm, true);
if (!err || !iwlmvm_mod_params.init_dbg)
iwl_mvm_stop_device(mvm);
iwl_mvm_unref(mvm, IWL_MVM_REF_INIT_UCODE);
mutex_unlock(&mvm->mutex);
/* returns 0 if successful, 1 if success but in rfkill */
if (err < 0 && !iwlmvm_mod_params.init_dbg) {
IWL_ERR(mvm, "Failed to run INIT ucode: %d\n", err);
goto out_free;
}
} }
scan_size = iwl_mvm_scan_size(mvm); scan_size = iwl_mvm_scan_size(mvm);
...@@ -784,8 +768,8 @@ iwl_op_mode_mvm_start(struct iwl_trans *trans, const struct iwl_cfg *cfg, ...@@ -784,8 +768,8 @@ iwl_op_mode_mvm_start(struct iwl_trans *trans, const struct iwl_cfg *cfg,
flush_delayed_work(&mvm->fw_dump_wk); flush_delayed_work(&mvm->fw_dump_wk);
iwl_phy_db_free(mvm->phy_db); iwl_phy_db_free(mvm->phy_db);
kfree(mvm->scan_cmd); kfree(mvm->scan_cmd);
if (!cfg->no_power_up_nic_in_init || !mvm->nvm_file_name) iwl_trans_op_mode_leave(trans);
iwl_trans_op_mode_leave(trans);
ieee80211_free_hw(mvm->hw); ieee80211_free_hw(mvm->hw);
return NULL; return NULL;
} }
......
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