Commit b25e755e authored by Zong-Zhe Yang's avatar Zong-Zhe Yang Committed by Kalle Valo
Browse files

wifi: rtw89: debug: txpwr table access only valid page according to chip 



We now support RTL8851B which has only single RF path. For chip with
single RF path, TX power page is valid only in single path section.
So, we refine debugfs txpwr table to access TX power page according
to RF path number of runtime chip. It can prevent us from reading
beyond valid sections.

Signed-off-by: default avatarZong-Zhe Yang <kevin_yang@realtek.com>
Signed-off-by: default avatarPing-Ke Shih <pkshih@realtek.com>
Signed-off-by: default avatarKalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20230531060713.57203-3-pkshih@realtek.com
parent 9c52e8bf

drivers/net/wireless/realtek/rtw89/debug.c

+12 −1
Original line number Diff line number Diff line
@@ -376,6 +376,7 @@ struct txpwr_map { 
	u8 size;

	u32 addr_from;

	u32 addr_to;

	u32 addr_to_1ss;

};



#define __GEN_TXPWR_ENT2(_t, _e0, _e1) \
@@ -413,6 +414,7 @@ static const struct txpwr_map __txpwr_map_byr = { 
	.size = ARRAY_SIZE(__txpwr_ent_byr),

	.addr_from = R_AX_PWR_BY_RATE,

	.addr_to = R_AX_PWR_BY_RATE_MAX,

	.addr_to_1ss = R_AX_PWR_BY_RATE_1SS_MAX,

};



static const struct txpwr_ent __txpwr_ent_lmt[] = {
@@ -468,6 +470,7 @@ static const struct txpwr_map __txpwr_map_lmt = { 
	.size = ARRAY_SIZE(__txpwr_ent_lmt),

	.addr_from = R_AX_PWR_LMT,

	.addr_to = R_AX_PWR_LMT_MAX,

	.addr_to_1ss = R_AX_PWR_LMT_1SS_MAX,

};



static const struct txpwr_ent __txpwr_ent_lmt_ru[] = {
@@ -495,6 +498,7 @@ static const struct txpwr_map __txpwr_map_lmt_ru = { 
	.size = ARRAY_SIZE(__txpwr_ent_lmt_ru),

	.addr_from = R_AX_PWR_RU_LMT,

	.addr_to = R_AX_PWR_RU_LMT_MAX,

	.addr_to_1ss = R_AX_PWR_RU_LMT_1SS_MAX,

};



static u8 __print_txpwr_ent(struct seq_file *m, const struct txpwr_ent *ent,
@@ -527,6 +531,8 @@ static int __print_txpwr_map(struct seq_file *m, struct rtw89_dev *rtwdev, 
			     const struct txpwr_map *map)

{

	u8 fct = rtwdev->chip->txpwr_factor_mac;

	u8 path_num = rtwdev->chip->rf_path_num;

	u32 max_valid_addr;

	u32 val, addr;

	s8 *buf, tmp;

	u8 cur, i;
@@ -536,7 +542,12 @@ static int __print_txpwr_map(struct seq_file *m, struct rtw89_dev *rtwdev, 
	if (!buf)

		return -ENOMEM;



	for (addr = map->addr_from; addr <= map->addr_to; addr += 4) {

	if (path_num == 1)

		max_valid_addr = map->addr_to_1ss;

	else

		max_valid_addr = map->addr_to;



	for (addr = map->addr_from; addr <= max_valid_addr; addr += 4) {

		ret = rtw89_mac_txpwr_read32(rtwdev, RTW89_PHY_0, addr, &val);

		if (ret)

			val = MASKDWORD;

drivers/net/wireless/realtek/rtw89/reg.h

+6 −0
Original line number Diff line number Diff line
@@ -3342,16 +3342,22 @@ 
#define B_AX_PWR_UL_TB_2T_MASK GENMASK(4, 0)

#define B_AX_PWR_UL_TB_2T_V1_MASK GENMASK(7, 0)

#define R_AX_PWR_BY_RATE_TABLE0 0xD2C0

#define R_AX_PWR_BY_RATE_TABLE6 0xD2D8

#define R_AX_PWR_BY_RATE_TABLE10 0xD2E8

#define R_AX_PWR_BY_RATE R_AX_PWR_BY_RATE_TABLE0

#define R_AX_PWR_BY_RATE_1SS_MAX R_AX_PWR_BY_RATE_TABLE6

#define R_AX_PWR_BY_RATE_MAX R_AX_PWR_BY_RATE_TABLE10

#define R_AX_PWR_LMT_TABLE0 0xD2EC

#define R_AX_PWR_LMT_TABLE9 0xD310

#define R_AX_PWR_LMT_TABLE19 0xD338

#define R_AX_PWR_LMT R_AX_PWR_LMT_TABLE0

#define R_AX_PWR_LMT_1SS_MAX R_AX_PWR_LMT_TABLE9

#define R_AX_PWR_LMT_MAX R_AX_PWR_LMT_TABLE19

#define R_AX_PWR_RU_LMT_TABLE0 0xD33C

#define R_AX_PWR_RU_LMT_TABLE5 0xD350

#define R_AX_PWR_RU_LMT_TABLE11 0xD368

#define R_AX_PWR_RU_LMT R_AX_PWR_RU_LMT_TABLE0

#define R_AX_PWR_RU_LMT_1SS_MAX R_AX_PWR_RU_LMT_TABLE5

#define R_AX_PWR_RU_LMT_MAX R_AX_PWR_RU_LMT_TABLE11

#define R_AX_PWR_MACID_LMT_TABLE0 0xD36C

#define R_AX_PWR_MACID_LMT_TABLE127 0xD568