iwlagn: remove dual-indirect call to simply the code
authorWey-Yi Guy <wey-yi.w.guy@intel.com>
Thu, 7 Jul 2011 15:33:01 +0000 (08:33 -0700)
committerWey-Yi Guy <wey-yi.w.guy@intel.com>
Sat, 16 Jul 2011 14:37:28 +0000 (07:37 -0700)
After driver split, no need to make the code so complex

Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
drivers/net/wireless/iwlwifi/iwl-1000.c
drivers/net/wireless/iwlwifi/iwl-2000.c
drivers/net/wireless/iwlwifi/iwl-5000.c
drivers/net/wireless/iwlwifi/iwl-6000.c
drivers/net/wireless/iwlwifi/iwl-agn-lib.c
drivers/net/wireless/iwlwifi/iwl-agn.c
drivers/net/wireless/iwlwifi/iwl-core.h
drivers/net/wireless/iwlwifi/iwl-eeprom.c
drivers/net/wireless/iwlwifi/iwl-rx.c

index 2f56b343e869b6b3423b8e250e49af4b1babe92a..e21f718551662c5e5ae0bb2cc31e3cd95da63bf3 100644 (file)
@@ -186,10 +186,6 @@ static struct iwl_lib_ops iwl1000_lib = {
        .temperature = iwlagn_temperature,
 };
 
-static const struct iwl_ops iwl1000_ops = {
-       .lib = &iwl1000_lib,
-};
-
 static struct iwl_base_params iwl1000_base_params = {
        .num_of_queues = IWLAGN_NUM_QUEUES,
        .num_of_ampdu_queues = IWLAGN_NUM_AMPDU_QUEUES,
@@ -217,7 +213,7 @@ static struct iwl_ht_params iwl1000_ht_params = {
        .ucode_api_min = IWL1000_UCODE_API_MIN,                 \
        .eeprom_ver = EEPROM_1000_EEPROM_VERSION,               \
        .eeprom_calib_ver = EEPROM_1000_TX_POWER_VERSION,       \
-       .ops = &iwl1000_ops,                                    \
+       .lib = &iwl1000_lib,                                    \
        .base_params = &iwl1000_base_params,                    \
        .led_mode = IWL_LED_BLINK
 
@@ -238,7 +234,7 @@ struct iwl_cfg iwl1000_bg_cfg = {
        .ucode_api_min = IWL100_UCODE_API_MIN,                  \
        .eeprom_ver = EEPROM_1000_EEPROM_VERSION,               \
        .eeprom_calib_ver = EEPROM_1000_TX_POWER_VERSION,       \
-       .ops = &iwl1000_ops,                                    \
+       .lib = &iwl1000_lib,                                    \
        .base_params = &iwl1000_base_params,                    \
        .led_mode = IWL_LED_RF_STATE,                           \
        .rx_with_siso_diversity = true
index 32ac8654b79ae60b4a502a3d736974c8bbda975d..f72f619dfc110966e0bdd597ea99c4f064b7f212 100644 (file)
@@ -208,22 +208,6 @@ static struct iwl_lib_ops iwl2030_lib = {
        .temperature = iwlagn_temperature,
 };
 
-static const struct iwl_ops iwl2000_ops = {
-       .lib = &iwl2000_lib,
-};
-
-static const struct iwl_ops iwl2030_ops = {
-       .lib = &iwl2030_lib,
-};
-
-static const struct iwl_ops iwl105_ops = {
-       .lib = &iwl2000_lib,
-};
-
-static const struct iwl_ops iwl135_ops = {
-       .lib = &iwl2030_lib,
-};
-
 static struct iwl_base_params iwl2000_base_params = {
        .eeprom_size = OTP_LOW_IMAGE_SIZE,
        .num_of_queues = IWLAGN_NUM_QUEUES,
@@ -282,7 +266,7 @@ static struct iwl_bt_params iwl2030_bt_params = {
        .ucode_api_min = IWL2000_UCODE_API_MIN,                 \
        .eeprom_ver = EEPROM_2000_EEPROM_VERSION,               \
        .eeprom_calib_ver = EEPROM_2000_TX_POWER_VERSION,       \
-       .ops = &iwl2000_ops,                                    \
+       .lib = &iwl2000_lib,                                    \
        .base_params = &iwl2000_base_params,                    \
        .need_dc_calib = true,                                  \
        .need_temp_offset_calib = true,                         \
@@ -307,7 +291,7 @@ struct iwl_cfg iwl2000_2bg_cfg = {
        .ucode_api_min = IWL2030_UCODE_API_MIN,                 \
        .eeprom_ver = EEPROM_2000_EEPROM_VERSION,               \
        .eeprom_calib_ver = EEPROM_2000_TX_POWER_VERSION,       \
-       .ops = &iwl2030_ops,                                    \
+       .lib = &iwl2030_lib,                                    \
        .base_params = &iwl2030_base_params,                    \
        .bt_params = &iwl2030_bt_params,                        \
        .need_dc_calib = true,                                  \
@@ -333,7 +317,7 @@ struct iwl_cfg iwl2030_2bg_cfg = {
        .ucode_api_min = IWL105_UCODE_API_MIN,                  \
        .eeprom_ver = EEPROM_2000_EEPROM_VERSION,               \
        .eeprom_calib_ver = EEPROM_2000_TX_POWER_VERSION,       \
-       .ops = &iwl105_ops,                                     \
+       .lib = &iwl2000_lib,                                    \
        .base_params = &iwl2000_base_params,                    \
        .need_dc_calib = true,                                  \
        .need_temp_offset_calib = true,                         \
@@ -358,7 +342,7 @@ struct iwl_cfg iwl105_bgn_cfg = {
        .ucode_api_min = IWL135_UCODE_API_MIN,                  \
        .eeprom_ver = EEPROM_2000_EEPROM_VERSION,               \
        .eeprom_calib_ver = EEPROM_2000_TX_POWER_VERSION,       \
-       .ops = &iwl135_ops,                                     \
+       .lib = &iwl2030_lib,                                    \
        .base_params = &iwl2030_base_params,                    \
        .bt_params = &iwl2030_bt_params,                        \
        .need_dc_calib = true,                                  \
index 556489302da33ed305641c037e2a196a39decc72..33b383cb676bebe7779c4650fc4345b6b902b0c7 100644 (file)
@@ -360,14 +360,6 @@ static struct iwl_lib_ops iwl5150_lib = {
        .temperature = iwl5150_temperature,
 };
 
-static const struct iwl_ops iwl5000_ops = {
-       .lib = &iwl5000_lib,
-};
-
-static const struct iwl_ops iwl5150_ops = {
-       .lib = &iwl5150_lib,
-};
-
 static struct iwl_base_params iwl5000_base_params = {
        .eeprom_size = IWLAGN_EEPROM_IMG_SIZE,
        .num_of_queues = IWLAGN_NUM_QUEUES,
@@ -390,7 +382,7 @@ static struct iwl_ht_params iwl5000_ht_params = {
        .ucode_api_min = IWL5000_UCODE_API_MIN,                 \
        .eeprom_ver = EEPROM_5000_EEPROM_VERSION,               \
        .eeprom_calib_ver = EEPROM_5000_TX_POWER_VERSION,       \
-       .ops = &iwl5000_ops,                                    \
+       .lib = &iwl5000_lib,                                    \
        .base_params = &iwl5000_base_params,                    \
        .led_mode = IWL_LED_BLINK
 
@@ -433,7 +425,7 @@ struct iwl_cfg iwl5350_agn_cfg = {
        .ucode_api_min = IWL5000_UCODE_API_MIN,
        .eeprom_ver = EEPROM_5050_EEPROM_VERSION,
        .eeprom_calib_ver = EEPROM_5050_TX_POWER_VERSION,
-       .ops = &iwl5000_ops,
+       .lib = &iwl5000_lib,
        .base_params = &iwl5000_base_params,
        .ht_params = &iwl5000_ht_params,
        .led_mode = IWL_LED_BLINK,
@@ -446,7 +438,7 @@ struct iwl_cfg iwl5350_agn_cfg = {
        .ucode_api_min = IWL5150_UCODE_API_MIN,                 \
        .eeprom_ver = EEPROM_5050_EEPROM_VERSION,               \
        .eeprom_calib_ver = EEPROM_5050_TX_POWER_VERSION,       \
-       .ops = &iwl5150_ops,                                    \
+       .lib = &iwl5150_lib,                                    \
        .base_params = &iwl5000_base_params,                    \
        .need_dc_calib = true,                                  \
        .led_mode = IWL_LED_BLINK,                              \
index 80f1ef61a3d55dde437583842b930ba826e7f979..f6b309da6ebbbc09f7467d8c79e2dd70f6479835 100644 (file)
@@ -106,9 +106,9 @@ static void iwl6000_nic_config(struct iwl_priv *priv)
                             CSR_GP_DRIVER_REG_BIT_RADIO_SKU_2x2_IPA);
        }
        /* do additional nic configuration if needed */
-       if (priv->cfg->ops->nic &&
-               priv->cfg->ops->nic->additional_nic_config) {
-                       priv->cfg->ops->nic->additional_nic_config(priv);
+       if (priv->cfg->nic &&
+               priv->cfg->nic->additional_nic_config) {
+                       priv->cfg->nic->additional_nic_config(priv);
        }
 }
 
@@ -311,24 +311,6 @@ static struct iwl_nic_ops iwl6150_nic_ops = {
        .additional_nic_config = &iwl6150_additional_nic_config,
 };
 
-static const struct iwl_ops iwl6000_ops = {
-       .lib = &iwl6000_lib,
-};
-
-static const struct iwl_ops iwl6050_ops = {
-       .lib = &iwl6000_lib,
-       .nic = &iwl6050_nic_ops,
-};
-
-static const struct iwl_ops iwl6150_ops = {
-       .lib = &iwl6000_lib,
-       .nic = &iwl6150_nic_ops,
-};
-
-static const struct iwl_ops iwl6030_ops = {
-       .lib = &iwl6030_lib,
-};
-
 static struct iwl_base_params iwl6000_base_params = {
        .eeprom_size = OTP_LOW_IMAGE_SIZE,
        .num_of_queues = IWLAGN_NUM_QUEUES,
@@ -402,7 +384,7 @@ static struct iwl_bt_params iwl6000_bt_params = {
        .ucode_api_min = IWL6000G2_UCODE_API_MIN,               \
        .eeprom_ver = EEPROM_6005_EEPROM_VERSION,               \
        .eeprom_calib_ver = EEPROM_6005_TX_POWER_VERSION,       \
-       .ops = &iwl6000_ops,                                    \
+       .lib = &iwl6000_lib,                                    \
        .base_params = &iwl6000_g2_base_params,                 \
        .need_dc_calib = true,                                  \
        .need_temp_offset_calib = true,                         \
@@ -430,7 +412,7 @@ struct iwl_cfg iwl6005_2bg_cfg = {
        .ucode_api_min = IWL6000G2_UCODE_API_MIN,               \
        .eeprom_ver = EEPROM_6030_EEPROM_VERSION,               \
        .eeprom_calib_ver = EEPROM_6030_TX_POWER_VERSION,       \
-       .ops = &iwl6030_ops,                                    \
+       .lib = &iwl6030_lib,                                    \
        .base_params = &iwl6000_g2_base_params,                 \
        .bt_params = &iwl6000_bt_params,                        \
        .need_dc_calib = true,                                  \
@@ -511,7 +493,7 @@ struct iwl_cfg iwl130_bg_cfg = {
        .valid_rx_ant = ANT_BC,         /* .cfg overwrite */    \
        .eeprom_ver = EEPROM_6000_EEPROM_VERSION,               \
        .eeprom_calib_ver = EEPROM_6000_TX_POWER_VERSION,       \
-       .ops = &iwl6000_ops,                                    \
+       .lib = &iwl6000_lib,                                    \
        .base_params = &iwl6000_base_params,                    \
        .pa_type = IWL_PA_INTERNAL,                             \
        .led_mode = IWL_LED_BLINK
@@ -538,7 +520,8 @@ struct iwl_cfg iwl6000i_2bg_cfg = {
        .ucode_api_min = IWL6050_UCODE_API_MIN,                 \
        .valid_tx_ant = ANT_AB,         /* .cfg overwrite */    \
        .valid_rx_ant = ANT_AB,         /* .cfg overwrite */    \
-       .ops = &iwl6050_ops,                                    \
+       .lib = &iwl6000_lib,                                    \
+       .nic = &iwl6050_nic_ops,                                \
        .eeprom_ver = EEPROM_6050_EEPROM_VERSION,               \
        .eeprom_calib_ver = EEPROM_6050_TX_POWER_VERSION,       \
        .base_params = &iwl6050_base_params,                    \
@@ -561,7 +544,8 @@ struct iwl_cfg iwl6050_2abg_cfg = {
        .fw_name_pre = IWL6050_FW_PRE,                          \
        .ucode_api_max = IWL6050_UCODE_API_MAX,                 \
        .ucode_api_min = IWL6050_UCODE_API_MIN,                 \
-       .ops = &iwl6150_ops,                                    \
+       .lib = &iwl6000_lib,                                    \
+       .nic = &iwl6150_nic_ops,                                \
        .eeprom_ver = EEPROM_6150_EEPROM_VERSION,               \
        .eeprom_calib_ver = EEPROM_6150_TX_POWER_VERSION,       \
        .base_params = &iwl6050_base_params,                    \
@@ -587,7 +571,7 @@ struct iwl_cfg iwl6000_3agn_cfg = {
        .ucode_api_min = IWL6000_UCODE_API_MIN,
        .eeprom_ver = EEPROM_6000_EEPROM_VERSION,
        .eeprom_calib_ver = EEPROM_6000_TX_POWER_VERSION,
-       .ops = &iwl6000_ops,
+       .lib = &iwl6000_lib,
        .base_params = &iwl6000_base_params,
        .ht_params = &iwl6000_ht_params,
        .need_dc_calib = true,
index b28219003121bda4a78ae6dc31e777e6cfd61a15..d4a60105ab79df5202f16f706b68355e41d1e69b 100644 (file)
@@ -711,7 +711,7 @@ int iwlagn_hw_nic_init(struct iwl_priv *priv)
 
        iwlagn_set_pwr_vmain(priv);
 
-       priv->cfg->ops->lib->nic_config(priv);
+       priv->cfg->lib->nic_config(priv);
 
        /* Allocate the RX queue, or reset if it is already allocated */
        trans_rx_init(priv);
index dceb450636698b108bb213ba1bfb434c34dff7e6..41c3e15104ad66fe16cca7924dc780d4ead731e6 100644 (file)
@@ -375,7 +375,7 @@ static void iwl_continuous_event_trace(struct iwl_priv *priv)
        u32 next_entry; /* index of next entry to be written by uCode */
 
        base = priv->device_pointers.error_event_table;
-       if (priv->cfg->ops->lib->is_valid_rtc_data_addr(base)) {
+       if (priv->cfg->lib->is_valid_rtc_data_addr(base)) {
                capacity = iwl_read_targ_mem(priv, base);
                num_wraps = iwl_read_targ_mem(priv, base + (2 * sizeof(u32)));
                mode = iwl_read_targ_mem(priv, base + (1 * sizeof(u32)));
@@ -1614,7 +1614,7 @@ void iwl_dump_nic_error_log(struct iwl_priv *priv)
                        base = priv->_agn.inst_errlog_ptr;
        }
 
-       if (!priv->cfg->ops->lib->is_valid_rtc_data_addr(base)) {
+       if (!priv->cfg->lib->is_valid_rtc_data_addr(base)) {
                IWL_ERR(priv,
                        "Not valid error log pointer 0x%08X for %s uCode\n",
                        base,
@@ -1805,7 +1805,7 @@ int iwl_dump_nic_event_log(struct iwl_priv *priv, bool full_log,
                        base = priv->_agn.inst_evtlog_ptr;
        }
 
-       if (!priv->cfg->ops->lib->is_valid_rtc_data_addr(base)) {
+       if (!priv->cfg->lib->is_valid_rtc_data_addr(base)) {
                IWL_ERR(priv,
                        "Invalid event log pointer 0x%08X for %s uCode\n",
                        base,
@@ -2939,7 +2939,7 @@ static void iwlagn_mac_channel_switch(struct ieee80211_hw *hw,
        if (!iwl_is_associated_ctx(ctx))
                goto out;
 
-       if (!priv->cfg->ops->lib->set_channel_switch)
+       if (!priv->cfg->lib->set_channel_switch)
                goto out;
 
        ch = channel->hw_value;
@@ -2991,7 +2991,7 @@ static void iwlagn_mac_channel_switch(struct ieee80211_hw *hw,
         */
        set_bit(STATUS_CHANNEL_SWITCH_PENDING, &priv->status);
        priv->switch_channel = cpu_to_le16(ch);
-       if (priv->cfg->ops->lib->set_channel_switch(priv, ch_switch)) {
+       if (priv->cfg->lib->set_channel_switch(priv, ch_switch)) {
                clear_bit(STATUS_CHANNEL_SWITCH_PENDING, &priv->status);
                priv->switch_channel = 0;
                ieee80211_chswitch_done(ctx->vif, false);
@@ -3198,8 +3198,8 @@ static void iwl_setup_deferred_work(struct iwl_priv *priv)
 
        iwl_setup_scan_deferred_work(priv);
 
-       if (priv->cfg->ops->lib->setup_deferred_work)
-               priv->cfg->ops->lib->setup_deferred_work(priv);
+       if (priv->cfg->lib->setup_deferred_work)
+               priv->cfg->lib->setup_deferred_work(priv);
 
        init_timer(&priv->statistics_periodic);
        priv->statistics_periodic.data = (unsigned long)priv;
@@ -3216,8 +3216,8 @@ static void iwl_setup_deferred_work(struct iwl_priv *priv)
 
 static void iwl_cancel_deferred_work(struct iwl_priv *priv)
 {
-       if (priv->cfg->ops->lib->cancel_deferred_work)
-               priv->cfg->ops->lib->cancel_deferred_work(priv);
+       if (priv->cfg->lib->cancel_deferred_work)
+               priv->cfg->lib->cancel_deferred_work(priv);
 
        cancel_work_sync(&priv->run_time_calib_work);
        cancel_work_sync(&priv->beacon_update);
@@ -3397,7 +3397,7 @@ static int iwl_set_hw_params(struct iwl_priv *priv)
                priv->cfg->sku &= ~EEPROM_SKU_CAP_11N_ENABLE;
 
        /* Device-specific setup */
-       return priv->cfg->ops->lib->set_hw_params(priv);
+       return priv->cfg->lib->set_hw_params(priv);
 }
 
 static const u8 iwlagn_bss_ac_to_fifo[] = {
index 692c30cb2fac39c9e772a3eda05f6bfa42117549..d6d2760c7c3826e3e7093533b90623ec539e77e8 100644 (file)
@@ -108,11 +108,6 @@ struct iwl_nic_ops {
        void (*additional_nic_config)(struct iwl_priv *priv);
 };
 
-struct iwl_ops {
-       const struct iwl_lib_ops *lib;
-       const struct iwl_nic_ops *nic;
-};
-
 struct iwl_mod_params {
        int sw_crypto;          /* def: 0 = using hardware encryption */
        int num_of_queues;      /* def: HW dependent */
@@ -247,7 +242,8 @@ struct iwl_cfg {
        u16  sku;
        u16  eeprom_ver;
        u16  eeprom_calib_ver;
-       const struct iwl_ops *ops;
+       const struct iwl_lib_ops *lib;
+       const struct iwl_nic_ops *nic;
        /* params not likely to change within a device family */
        struct iwl_base_params *base_params;
        /* params likely to change within a device family */
index eee97bcf98022a5e74b17351b8d7a29ba154d602..19d31a5e32e5269032f2c3f06d12c294c8a1ca7d 100644 (file)
@@ -543,7 +543,7 @@ static void iwl_init_band_reference(const struct iwl_priv *priv,
                        const struct iwl_eeprom_channel **eeprom_ch_info,
                        const u8 **eeprom_ch_index)
 {
-       u32 offset = priv->cfg->ops->lib->
+       u32 offset = priv->cfg->lib->
                        eeprom_ops.regulatory_bands[eep_band - 1];
        switch (eep_band) {
        case 1:         /* 2.4GHz band */
@@ -749,9 +749,9 @@ int iwl_init_channel_map(struct iwl_priv *priv)
        }
 
        /* Check if we do have HT40 channels */
-       if (priv->cfg->ops->lib->eeprom_ops.regulatory_bands[5] ==
+       if (priv->cfg->lib->eeprom_ops.regulatory_bands[5] ==
            EEPROM_REGULATORY_BAND_NO_HT40 &&
-           priv->cfg->ops->lib->eeprom_ops.regulatory_bands[6] ==
+           priv->cfg->lib->eeprom_ops.regulatory_bands[6] ==
            EEPROM_REGULATORY_BAND_NO_HT40)
                return 0;
 
@@ -787,8 +787,8 @@ int iwl_init_channel_map(struct iwl_priv *priv)
         * driver need to process addition information
         * to determine the max channel tx power limits
         */
-       if (priv->cfg->ops->lib->eeprom_ops.update_enhanced_txpower)
-               priv->cfg->ops->lib->eeprom_ops.update_enhanced_txpower(priv);
+       if (priv->cfg->lib->eeprom_ops.update_enhanced_txpower)
+               priv->cfg->lib->eeprom_ops.update_enhanced_txpower(priv);
 
        return 0;
 }
index c5eb379246ffd145fee0862de33df96e75cedc90..bd63b785f68c8c9e5050829bd63ab32e018a78d9 100644 (file)
@@ -624,8 +624,8 @@ static void iwl_rx_statistics(struct iwl_priv *priv,
                iwl_rx_calc_noise(priv);
                queue_work(priv->workqueue, &priv->run_time_calib_work);
        }
-       if (priv->cfg->ops->lib->temperature && change)
-               priv->cfg->ops->lib->temperature(priv);
+       if (priv->cfg->lib->temperature && change)
+               priv->cfg->lib->temperature(priv);
 }
 
 static void iwl_rx_reply_statistics(struct iwl_priv *priv,
@@ -1103,7 +1103,7 @@ void iwl_setup_rx_handlers(struct iwl_priv *priv)
        handlers[REPLY_COMPRESSED_BA]           = iwlagn_rx_reply_compressed_ba;
 
        /* Set up hardware specific Rx handlers */
-       priv->cfg->ops->lib->rx_handler_setup(priv);
+       priv->cfg->lib->rx_handler_setup(priv);
 }
 
 void iwl_rx_dispatch(struct iwl_priv *priv, struct iwl_rx_mem_buffer *rxb)