iwlwifi: remove wide_cmd_header field
authorMordechay Goodstein <mordechay.goodstein@intel.com>
Mon, 28 Sep 2020 09:23:19 +0000 (12:23 +0300)
committerLuca Coelho <luciano.coelho@intel.com>
Thu, 1 Oct 2020 18:59:37 +0000 (21:59 +0300)
Driver doesn't support fw without wide_cmd
so driver always sets it to true.  instead of setting
it always to true just remove the field.

Signed-off-by: Mordechay Goodstein <mordechay.goodstein@intel.com>
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
Link: https://lore.kernel.org/r/iwlwifi.20200928121852.e6137861d917.I93405604eb503568688b28d3169fea7fbb88ed7e@changeid
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
drivers/net/wireless/intel/iwlwifi/iwl-trans.c
drivers/net/wireless/intel/iwlwifi/iwl-trans.h
drivers/net/wireless/intel/iwlwifi/mvm/ops.c
drivers/net/wireless/intel/iwlwifi/pcie/tx.c

index a99e0c5af5120869056fe19541db25ef4af71451..073efce47e74d4214b494c953139c0640f0caa0f 100644 (file)
@@ -181,7 +181,7 @@ int iwl_trans_send_cmd(struct iwl_trans *trans, struct iwl_host_cmd *cmd)
        if (!(cmd->flags & CMD_ASYNC))
                lock_map_acquire_read(&trans->sync_cmd_lockdep_map);
 
-       if (trans->wide_cmd_header && !iwl_cmd_groupid(cmd->id))
+       if (!iwl_cmd_groupid(cmd->id))
                cmd->id = DEF_ID(cmd->id);
 
        ret = trans->ops->send_cmd(trans, cmd);
index 868e137c51f04268d6eee6c06b4c3aa9e76b0927..8fe720ac1c74caee3b71e112aa9e166dc6286866 100644 (file)
@@ -960,7 +960,6 @@ struct iwl_trans_txqs {
  * @hw_id_str: a string with info about HW ID. Set during transport allocation.
  * @pm_support: set to true in start_hw if link pm is supported
  * @ltr_enabled: set to true if the LTR is enabled
- * @wide_cmd_header: true when ucode supports wide command header format
  * @num_rx_queues: number of RX queues allocated by the transport;
  *     the transport must set this before calling iwl_drv_start()
  * @iml_len: the length of the image loader
@@ -1000,7 +999,6 @@ struct iwl_trans {
 
        const struct iwl_hcmd_arr *command_groups;
        int command_groups_size;
-       bool wide_cmd_header;
 
        u8 num_rx_queues;
 
index e184a163946ab40515186595d8b6ba12ff842850..5a6f0fe2fb90254d9c6e1f2eb794e3d44c764d7a 100644 (file)
@@ -757,7 +757,6 @@ iwl_op_mode_mvm_start(struct iwl_trans *trans, const struct iwl_cfg *cfg,
                trans_cfg.rx_buf_size = rb_size_default;
        }
 
-       trans->wide_cmd_header = true;
        trans_cfg.bc_table_dword =
                mvm->trans->trans_cfg->device_family < IWL_DEVICE_FAMILY_AX210;
 
index e15939849a23fe2bb9b4eeb8bed5b6cc2b660ae8..027b4e787ee6f0d16ff8cb625d29b2cf7792ad11 100644 (file)
@@ -1538,11 +1538,6 @@ static int iwl_pcie_enqueue_hcmd(struct iwl_trans *trans,
        const u8 *cmddata[IWL_MAX_CMD_TBS_PER_TFD];
        u16 cmdlen[IWL_MAX_CMD_TBS_PER_TFD];
 
-       if (WARN(!trans->wide_cmd_header &&
-                group_id > IWL_ALWAYS_LONG_GROUP,
-                "unsupported wide command %#x\n", cmd->id))
-               return -EINVAL;
-
        if (group_id != 0) {
                copy_size = sizeof(struct iwl_cmd_header_wide);
                cmd_size = sizeof(struct iwl_cmd_header_wide);