staging: r8188eu: don't check for stop/removal in the blink worker
authorMartin Kaiser <martin@kaiser.cx>
Sun, 25 Sep 2022 14:04:06 +0000 (16:04 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 26 Sep 2022 08:34:44 +0000 (10:34 +0200)
The blink_work function calls either SwLedOff or SwLedOn. These two
functions handle bSurpriseRemoved and bDriverStopped.

There's no need to check bSurpriseRemoved and bDriverStopped again in the
blink worker.

Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150
Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Link: https://lore.kernel.org/r/20220925140406.112991-3-martin@kaiser.cx
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/r8188eu/core/rtw_led.c

index d3bd9a97efdcfe2c2af8fdd34f407d39199a2281..1e316e6358ea2deda4c80ef0bb8710fc156167e6 100644 (file)
@@ -77,9 +77,6 @@ static void blink_work(struct work_struct *work)
        struct adapter *padapter = pLed->padapter;
        struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
 
-       if ((padapter->bSurpriseRemoved) || (padapter->bDriverStopped))
-               return;
-
        if (padapter->pwrctrlpriv.rf_pwrstate != rf_on) {
                SwLedOff(padapter, pLed);
                ResetLedStatus(pLed);