Skip to content

Commit

Permalink
iwlwifi: remove firmware info from iwl_shared
Browse files Browse the repository at this point in the history
With error logging now completely handled in
the op_mode, the transport layer does not
need to know information about the loaded
firmware.

Remove this state information from the
iwl_shared data structure.

Signed-off-by: Meenakshi Venkataraman <meenakshi.venkataraman@intel.com>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
  • Loading branch information
Meenakshi Venkataraman authored and John W. Linville committed Apr 9, 2012
1 parent c736f23 commit e3ec26d
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 12 deletions.
20 changes: 9 additions & 11 deletions drivers/net/wireless/iwlwifi/iwl-agn.c
Original file line number Diff line number Diff line change
Expand Up @@ -1498,8 +1498,6 @@ static struct iwl_op_mode *iwl_op_mode_dvm_start(struct iwl_trans *trans,
priv = IWL_OP_MODE_GET_DVM(op_mode);
priv->shrd = trans->shrd;
priv->fw = fw;
/* TODO: remove fw from shared data later */
priv->shrd->fw = fw;

/*
* Populate the state variables that the transport layer needs
Expand Down Expand Up @@ -1817,10 +1815,10 @@ static void iwl_dump_nic_error_log(struct iwl_priv *priv)
base = priv->device_pointers.error_event_table;
if (priv->cur_ucode == IWL_UCODE_INIT) {
if (!base)
base = priv->shrd->fw->init_errlog_ptr;
base = priv->fw->init_errlog_ptr;
} else {
if (!base)
base = priv->shrd->fw->inst_errlog_ptr;
base = priv->fw->inst_errlog_ptr;
}

if (!iwlagn_hw_valid_rtc_data_addr(base)) {
Expand Down Expand Up @@ -1908,10 +1906,10 @@ static int iwl_print_event_log(struct iwl_priv *priv, u32 start_idx,
base = priv->device_pointers.log_event_table;
if (priv->cur_ucode == IWL_UCODE_INIT) {
if (!base)
base = priv->shrd->fw->init_evtlog_ptr;
base = priv->fw->init_evtlog_ptr;
} else {
if (!base)
base = priv->shrd->fw->inst_evtlog_ptr;
base = priv->fw->inst_evtlog_ptr;
}

if (mode == 0)
Expand Down Expand Up @@ -2022,13 +2020,13 @@ int iwl_dump_nic_event_log(struct iwl_priv *priv, bool full_log,

base = priv->device_pointers.log_event_table;
if (priv->cur_ucode == IWL_UCODE_INIT) {
logsize = priv->shrd->fw->init_evtlog_size;
logsize = priv->fw->init_evtlog_size;
if (!base)
base = priv->shrd->fw->init_evtlog_ptr;
base = priv->fw->init_evtlog_ptr;
} else {
logsize = priv->shrd->fw->inst_evtlog_size;
logsize = priv->fw->inst_evtlog_size;
if (!base)
base = priv->shrd->fw->inst_evtlog_ptr;
base = priv->fw->inst_evtlog_ptr;
}

if (!iwlagn_hw_valid_rtc_data_addr(base)) {
Expand Down Expand Up @@ -2117,7 +2115,7 @@ static void iwl_nic_error(struct iwl_op_mode *op_mode)
struct iwl_priv *priv = IWL_OP_MODE_GET_DVM(op_mode);

IWL_ERR(priv, "Loaded firmware version: %s\n",
priv->shrd->fw->fw_version);
priv->fw->fw_version);

iwl_dump_nic_error_log(priv);
iwl_dump_nic_event_log(priv, false, NULL, false);
Expand Down
1 change: 0 additions & 1 deletion drivers/net/wireless/iwlwifi/iwl-shared.h
Original file line number Diff line number Diff line change
Expand Up @@ -354,7 +354,6 @@ struct iwl_shared {
struct iwl_trans *trans;
void *drv;
struct iwl_hw_params hw_params;
const struct iwl_fw *fw;

/* eeprom -- this is in the card's little endian byte order */
u8 *eeprom;
Expand Down

0 comments on commit e3ec26d

Please sign in to comment.