On 5/30/23 10:54, Dmitry Antipov wrote:
Drop redundant reads from RTX_POWER_BEFORE_IQK_A and RTX_POWER_AFTER_IQK_A
registers in _rtl88e_phy_path_a_rx_iqk().
Found by Linux Verification Center (linuxtesting.org) with SVACE.
Signed-off-by: Dmitriy Antipov <Dmitriy.Antipov@xxxxxxxxxxxx>
---
drivers/net/wireless/realtek/rtlwifi/rtl8188ee/phy.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/phy.c b/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/phy.c
index 12d0b3a87af7..380a813acda8 100644
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/phy.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/phy.c
@@ -1475,8 +1475,6 @@ static u8 _rtl88e_phy_path_a_rx_iqk(struct ieee80211_hw *hw, bool config_pathb)
mdelay(IQK_DELAY_TIME);
reg_eac = rtl_get_bbreg(hw, RRX_POWER_AFTER_IQK_A_2, MASKDWORD);
- reg_e94 = rtl_get_bbreg(hw, RTX_POWER_BEFORE_IQK_A, MASKDWORD);
- reg_e9c = rtl_get_bbreg(hw, RTX_POWER_AFTER_IQK_A, MASKDWORD);
reg_ea4 = rtl_get_bbreg(hw, RRX_POWER_BEFORE_IQK_A_2, MASKDWORD);
if (!(reg_eac & BIT(27)) &&
I do not know the answer to this question either, but how does your tool know
that the statements between the first read and the second have not caused the
firmware to change the contents of the BB registers?
NACK for this patch
Larry
