wifi: rtw89: refine element naming used by queue empty check
authorZong-Zhe Yang <kevin_yang@realtek.com>
Fri, 24 Nov 2023 07:16:59 +0000 (15:16 +0800)
committerKalle Valo <kvalo@kernel.org>
Fri, 1 Dec 2023 12:39:29 +0000 (14:39 +0200)
In queue empty check, one group contains 32 queues. And, the two elements,
wde_qempty_acq_num and wde_qempty_mgq_sel, are number of group and select
of group. To avoid confusing them with queue number and queue selection,
we refine their naming.

(don't change logic at all)

Signed-off-by: Zong-Zhe Yang <kevin_yang@realtek.com>
Signed-off-by: Ping-Ke Shih <pkshih@realtek.com>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20231124071703.132549-5-pkshih@realtek.com
drivers/net/wireless/realtek/rtw89/core.h
drivers/net/wireless/realtek/rtw89/mac.c
drivers/net/wireless/realtek/rtw89/rtw8851b.c
drivers/net/wireless/realtek/rtw89/rtw8852a.c
drivers/net/wireless/realtek/rtw89/rtw8852b.c
drivers/net/wireless/realtek/rtw89/rtw8852c.c
drivers/net/wireless/realtek/rtw89/rtw8922a.c

index 5124f98..c315ef9 100644 (file)
@@ -3692,8 +3692,8 @@ struct rtw89_chip_info {
        u32 rsvd_ple_ofst;
        const struct rtw89_hfc_param_ini *hfc_param_ini;
        const struct rtw89_dle_mem *dle_mem;
-       u8 wde_qempty_acq_num;
-       u8 wde_qempty_mgq_sel;
+       u8 wde_qempty_acq_grpnum;
+       u8 wde_qempty_mgq_grpsel;
        u32 rf_base_addr[2];
        u8 support_chanctx_num;
        u8 support_bands;
index bdd9c15..b5e32b8 100644 (file)
@@ -1621,13 +1621,13 @@ int rtw89_mac_get_dle_rsvd_qt_cfg(struct rtw89_dev *rtwdev,
 static bool mac_is_txq_empty(struct rtw89_dev *rtwdev)
 {
        struct rtw89_mac_dle_dfi_qempty qempty;
-       u32 qnum, qtmp, val32, msk32;
+       u32 grpnum, qtmp, val32, msk32;
        int i, j, ret;
 
-       qnum = rtwdev->chip->wde_qempty_acq_num;
+       grpnum = rtwdev->chip->wde_qempty_acq_grpnum;
        qempty.dle_type = DLE_CTRL_TYPE_WDE;
 
-       for (i = 0; i < qnum; i++) {
+       for (i = 0; i < grpnum; i++) {
                qempty.grpsel = i;
                ret = dle_dfi_qempty(rtwdev, &qempty);
                if (ret) {
@@ -1643,7 +1643,7 @@ static bool mac_is_txq_empty(struct rtw89_dev *rtwdev)
                }
        }
 
-       qempty.grpsel = rtwdev->chip->wde_qempty_mgq_sel;
+       qempty.grpsel = rtwdev->chip->wde_qempty_mgq_grpsel;
        ret = dle_dfi_qempty(rtwdev, &qempty);
        if (ret) {
                rtw89_warn(rtwdev, "dle dfi mgq empty %d\n", ret);
index ed1f056..e850e34 100644 (file)
@@ -2374,8 +2374,8 @@ const struct rtw89_chip_info rtw8851b_chip_info = {
        .rsvd_ple_ofst          = 0x2f800,
        .hfc_param_ini          = rtw8851b_hfc_param_ini_pcie,
        .dle_mem                = rtw8851b_dle_mem_pcie,
-       .wde_qempty_acq_num     = 4,
-       .wde_qempty_mgq_sel     = 4,
+       .wde_qempty_acq_grpnum  = 4,
+       .wde_qempty_mgq_grpsel  = 4,
        .rf_base_addr           = {0xe000},
        .pwr_on_seq             = NULL,
        .pwr_off_seq            = NULL,
index 561049d..90f39a8 100644 (file)
@@ -2109,8 +2109,8 @@ const struct rtw89_chip_info rtw8852a_chip_info = {
        .rsvd_ple_ofst          = 0x6f800,
        .hfc_param_ini          = rtw8852a_hfc_param_ini_pcie,
        .dle_mem                = rtw8852a_dle_mem_pcie,
-       .wde_qempty_acq_num     = 16,
-       .wde_qempty_mgq_sel     = 16,
+       .wde_qempty_acq_grpnum  = 16,
+       .wde_qempty_mgq_grpsel  = 16,
        .rf_base_addr           = {0xc000, 0xd000},
        .pwr_on_seq             = pwr_on_seq_8852a,
        .pwr_off_seq            = pwr_off_seq_8852a,
index 9c62844..7b2062c 100644 (file)
@@ -2543,8 +2543,8 @@ const struct rtw89_chip_info rtw8852b_chip_info = {
        .rsvd_ple_ofst          = 0x2f800,
        .hfc_param_ini          = rtw8852b_hfc_param_ini_pcie,
        .dle_mem                = rtw8852b_dle_mem_pcie,
-       .wde_qempty_acq_num     = 4,
-       .wde_qempty_mgq_sel     = 4,
+       .wde_qempty_acq_grpnum  = 4,
+       .wde_qempty_mgq_grpsel  = 4,
        .rf_base_addr           = {0xe000, 0xf000},
        .pwr_on_seq             = NULL,
        .pwr_off_seq            = NULL,
index 976d34b..d618faf 100644 (file)
@@ -2879,8 +2879,8 @@ const struct rtw89_chip_info rtw8852c_chip_info = {
        .rsvd_ple_ofst          = 0x6f800,
        .hfc_param_ini          = rtw8852c_hfc_param_ini_pcie,
        .dle_mem                = rtw8852c_dle_mem_pcie,
-       .wde_qempty_acq_num     = 16,
-       .wde_qempty_mgq_sel     = 16,
+       .wde_qempty_acq_grpnum  = 16,
+       .wde_qempty_mgq_grpsel  = 16,
        .rf_base_addr           = {0xe000, 0xf000},
        .pwr_on_seq             = NULL,
        .pwr_off_seq            = NULL,
index b75d555..a82c17f 100644 (file)
@@ -342,6 +342,8 @@ const struct rtw89_chip_info rtw8922a_chip_info = {
        .rsvd_ple_ofst          = 0x8f800,
        .hfc_param_ini          = rtw8922a_hfc_param_ini_pcie,
        .dle_mem                = rtw8922a_dle_mem_pcie,
+       .wde_qempty_acq_grpnum  = 4,
+       .wde_qempty_mgq_grpsel  = 4,
        .rf_base_addr           = {0xe000, 0xf000},
        .pwr_on_seq             = NULL,
        .pwr_off_seq            = NULL,