staging: r8188eu: rfoff_reason is never initialised
authorMartin Kaiser <martin@kaiser.cx>
Sun, 26 Dec 2021 19:55:56 +0000 (20:55 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 28 Dec 2021 16:12:35 +0000 (17:12 +0100)
rfoff_reason in struct pwrctrl_priv is never set, its value remains 0.

Remove rfoff_reason, related defines and a check in rtw_led_control
that is always false.

Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Link: https://lore.kernel.org/r/20211226195556.159471-22-martin@kaiser.cx
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/r8188eu/core/rtw_led.c
drivers/staging/r8188eu/include/hal_intf.h
drivers/staging/r8188eu/include/rtw_pwrctrl.h

index 09a9676..ccd43ac 100644 (file)
@@ -207,14 +207,6 @@ void rtw_led_control(struct adapter *padapter, enum LED_CTL_MODE LedAction)
        if (!registry_par->led_enable)
                return;
 
-       if ((padapter->pwrctrlpriv.rf_pwrstate != rf_on &&
-            padapter->pwrctrlpriv.rfoff_reason > RF_CHANGE_BY_PS) &&
-           (LedAction == LED_CTL_TX || LedAction == LED_CTL_RX ||
-            LedAction == LED_CTL_SITE_SURVEY ||
-            LedAction == LED_CTL_LINK ||
-            LedAction == LED_CTL_NO_LINK))
-               return;
-
        switch (LedAction) {
        case LED_CTL_START_TO_LINK:
        case LED_CTL_NO_LINK:
index 6eb630f..b5474d1 100644 (file)
@@ -100,12 +100,6 @@ enum hal_odm_variable {
 
 typedef s32 (*c2h_id_filter)(u8 id);
 
-#define RF_CHANGE_BY_INIT      0
-#define RF_CHANGE_BY_IPS       BIT(28)
-#define RF_CHANGE_BY_PS                BIT(29)
-#define RF_CHANGE_BY_HW                BIT(30)
-#define RF_CHANGE_BY_SW                BIT(31)
-
 #define is_boot_from_eeprom(adapter) (adapter->eeprompriv.EepromOrEfuse)
 
 void rtl8188eu_interface_configure(struct adapter *adapt);
index a71f2f8..849e144 100644 (file)
@@ -57,7 +57,6 @@ struct pwrctrl_priv {
 
        u8      reg_rfoff;
        u8      reg_pdnmode; /* powerdown mode */
-       u32     rfoff_reason;
 
        /* RF OFF Level */
        u32     cur_ps_level;