wifi: rtlwifi: cleanup few rtlxxxx_set_hw_reg() routines

Since 'u8' comparison against zero is always false, drop the
corresponding branches of AMPDU_MIN_SPACE adjustment within
'rtlxxxx_set_hw_reg()' for rtl8188ee, rtl8192ce, rtl8192de,
rtl8723ae, rtl8723be, and rtl8821ae. Compile tested only.

Found by Linux Verification Center (linuxtesting.org) with SVACE.

Signed-off-by: Dmitry Antipov <dmantipov@yandex.ru>
Acked-by: Ping-Ke Shih <pkshih@realtek.com>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20230925090452.25633-1-dmantipov@yandex.ru

diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/hw.c b/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/hw.c
index 58b1a46066b57d96fa17e9a95186c651e73dd4ac..27f6c35ba0f9fdabe0286d68a334a8da02cb34f3 100644 (file)
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/hw.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/hw.c
@@ -433,14 +433,9 @@ void rtl88ee_set_hw_reg(struct ieee80211_hw *hw, u8 variable, u8 *val)
                break;
        case HW_VAR_AMPDU_MIN_SPACE:{
                u8 min_spacing_to_set;
-               u8 sec_min_space;
 
                min_spacing_to_set = *val;
                if (min_spacing_to_set <= 7) {
-                       sec_min_space = 0;
-
-                       if (min_spacing_to_set < sec_min_space)
-                               min_spacing_to_set = sec_min_space;
 
                        mac->min_space_cfg = ((mac->min_space_cfg &
                                               0xf8) |

diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/hw.c b/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/hw.c
index 049c4fe9eeed4bf1234aee1a8c23e365eae028bc..0bc915723b93955723af7d139070026a1e8c6cfb 100644 (file)
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/hw.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/hw.c
@@ -208,14 +208,9 @@ void rtl92ce_set_hw_reg(struct ieee80211_hw *hw, u8 variable, u8 *val)
                }
        case HW_VAR_AMPDU_MIN_SPACE:{
                        u8 min_spacing_to_set;
-                       u8 sec_min_space;
 
                        min_spacing_to_set = *val;
                        if (min_spacing_to_set <= 7) {
-                               sec_min_space = 0;
-
-                               if (min_spacing_to_set < sec_min_space)
-                                       min_spacing_to_set = sec_min_space;
 
                                mac->min_space_cfg = ((mac->min_space_cfg &
                                                       0xf8) |

diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192de/hw.c b/drivers/net/wireless/realtek/rtlwifi/rtl8192de/hw.c
index 31a18bbface94c55e8614ad52a083d50cde10f35..743ac6871bf4bff75bc13277aae9e5babd93b79b 100644 (file)
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8192de/hw.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192de/hw.c
@@ -225,13 +225,9 @@ void rtl92de_set_hw_reg(struct ieee80211_hw *hw, u8 variable, u8 *val)
        }
        case HW_VAR_AMPDU_MIN_SPACE: {
                u8 min_spacing_to_set;
-               u8 sec_min_space;
 
                min_spacing_to_set = *val;
                if (min_spacing_to_set <= 7) {
-                       sec_min_space = 0;
-                       if (min_spacing_to_set < sec_min_space)
-                               min_spacing_to_set = sec_min_space;
                        mac->min_space_cfg = ((mac->min_space_cfg & 0xf8) |
                                              min_spacing_to_set);
                        *val = min_spacing_to_set;

diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/hw.c b/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/hw.c
index d26d4c4314a3d6d20e661004f764d0417eb8e1de..6991713a66d0179e00cc6ed8f23102f24eb67a30 100644 (file)
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/hw.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/hw.c
@@ -212,14 +212,9 @@ void rtl8723e_set_hw_reg(struct ieee80211_hw *hw, u8 variable, u8 *val)
                }
        case HW_VAR_AMPDU_MIN_SPACE:{
                        u8 min_spacing_to_set;
-                       u8 sec_min_space;
 
                        min_spacing_to_set = *((u8 *)val);
                        if (min_spacing_to_set <= 7) {
-                               sec_min_space = 0;
-
-                               if (min_spacing_to_set < sec_min_space)
-                                       min_spacing_to_set = sec_min_space;
 
                                mac->min_space_cfg = ((mac->min_space_cfg &
                                                       0xf8) |

diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8723be/hw.c b/drivers/net/wireless/realtek/rtlwifi/rtl8723be/hw.c
index 15575644551f0d8a84e787317b1605de5f03ac62..0e77de1baaf878d4d9558e4f634b648fa821064d 100644 (file)
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8723be/hw.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8723be/hw.c
@@ -468,15 +468,9 @@ void rtl8723be_set_hw_reg(struct ieee80211_hw *hw, u8 variable, u8 *val)
                break;
        case HW_VAR_AMPDU_MIN_SPACE:{
                u8 min_spacing_to_set;
-               u8 sec_min_space;
 
                min_spacing_to_set = *((u8 *)val);
                if (min_spacing_to_set <= 7) {
-                       sec_min_space = 0;
-
-                       if (min_spacing_to_set < sec_min_space)
-                               min_spacing_to_set = sec_min_space;
-
                        mac->min_space_cfg = ((mac->min_space_cfg & 0xf8) |
                                              min_spacing_to_set);
 

diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/hw.c b/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/hw.c
index 3f8f6da33b12dcada076636c389059e091e4d78a..1633328bc3d1ddd26f8d7263426eb5cdfe196eeb 100644 (file)
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/hw.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/hw.c
@@ -546,14 +546,9 @@ void rtl8821ae_set_hw_reg(struct ieee80211_hw *hw, u8 variable, u8 *val)
                break;
        case HW_VAR_AMPDU_MIN_SPACE:{
                u8 min_spacing_to_set;
-               u8 sec_min_space;
 
                min_spacing_to_set = *((u8 *)val);
                if (min_spacing_to_set <= 7) {
-                       sec_min_space = 0;
-
-                       if (min_spacing_to_set < sec_min_space)
-                               min_spacing_to_set = sec_min_space;
 
                        mac->min_space_cfg = ((mac->min_space_cfg &
                                               0xf8) |