wifi: ath11k: Remove ath11k_base::bd_api
authorJeff Johnson <quic_jjohnson@quicinc.com>
Fri, 13 Oct 2023 14:24:08 +0000 (07:24 -0700)
committerKalle Valo <quic_kvalo@quicinc.com>
Wed, 18 Oct 2023 08:30:37 +0000 (11:30 +0300)
Currently struct ath11k_base defines the member bd_api. However, this
member is only accessed within ath11k_core_fetch_bdf(). Since the
scope is local just to that one function, remove it from ath11k_base
and instead just use a local stack variable.

No functional changes, compile tested only.

Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://lore.kernel.org/r/20231013-ath11k_bd_api-v1-1-3fefe4629706@quicinc.com
drivers/net/wireless/ath/ath11k/core.c
drivers/net/wireless/ath/ath11k/core.h

index c3a0dd15d8ea9e4cdf6cc7f845aa9bb230be08ec..1469ab4a2df9ebd8c149d15b8da26abca8b2010f 100644 (file)
@@ -1317,6 +1317,7 @@ int ath11k_core_fetch_bdf(struct ath11k_base *ab, struct ath11k_board_data *bd)
 {
        char *boardname = NULL, *fallback_boardname = NULL, *chip_id_boardname = NULL;
        char *filename, filepath[100];
+       int bd_api;
        int ret = 0;
 
        filename = ATH11K_BOARD_API2_FILE;
@@ -1332,7 +1333,7 @@ int ath11k_core_fetch_bdf(struct ath11k_base *ab, struct ath11k_board_data *bd)
                goto exit;
        }
 
-       ab->bd_api = 2;
+       bd_api = 2;
        ret = ath11k_core_fetch_board_data_api_n(ab, bd, boardname,
                                                 ATH11K_BD_IE_BOARD,
                                                 ATH11K_BD_IE_BOARD_NAME,
@@ -1381,7 +1382,7 @@ int ath11k_core_fetch_bdf(struct ath11k_base *ab, struct ath11k_board_data *bd)
        if (!ret)
                goto exit;
 
-       ab->bd_api = 1;
+       bd_api = 1;
        ret = ath11k_core_fetch_board_data_api_1(ab, bd, ATH11K_DEFAULT_BOARD_FILE);
        if (ret) {
                ath11k_core_create_firmware_path(ab, filename,
@@ -1405,7 +1406,7 @@ exit:
        kfree(chip_id_boardname);
 
        if (!ret)
-               ath11k_dbg(ab, ATH11K_DBG_BOOT, "using board api %d\n", ab->bd_api);
+               ath11k_dbg(ab, ATH11K_DBG_BOOT, "using board api %d\n", bd_api);
 
        return ret;
 }
index 489c863215a45ac0ca9ec5aadfee474f6f329aac..cd40cbdd4cc169faf3843df3ab51f13ec0450fb2 100644 (file)
@@ -906,7 +906,6 @@ struct ath11k_base {
        struct ath11k_targ_cap target_caps;
        u32 ext_service_bitmap[WMI_SERVICE_EXT_BM_SIZE];
        bool pdevs_macaddr_valid;
-       int bd_api;
 
        struct ath11k_hw_params hw_params;