staging: r8188eu: merge InitLed871x and rtl8188eu_InitSwLeds
authorMartin Kaiser <martin@kaiser.cx>
Sun, 26 Dec 2021 19:55:39 +0000 (20:55 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 28 Dec 2021 16:12:34 +0000 (17:12 +0100)
Copy InitLed871x into rtl8188eu_InitSwLeds. There's no need for two
separate functions.

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

index 50dabc3d2e29e50273640abb9ee52c941638ad70..541aba6faaf00cc65d2ae497b6ad49a3e59c9645 100644 (file)
@@ -59,15 +59,6 @@ exit:
        pLed->bLedOn = false;
 }
 
-void InitLed871x(struct adapter *padapter, struct LED_871x *pLed)
-{
-       pLed->padapter = padapter;
-
-       ResetLedStatus(pLed);
-
-       INIT_DELAYED_WORK(&pLed->blink_work, BlinkWorkItemCallback);
-}
-
 void DeInitLed871x(struct LED_871x *pLed)
 {
        cancel_delayed_work_sync(&pLed->blink_work);
@@ -420,10 +411,13 @@ void BlinkHandler(struct LED_871x *pLed)
 void rtl8188eu_InitSwLeds(struct adapter *padapter)
 {
        struct led_priv *pledpriv = &padapter->ledpriv;
+       struct LED_871x *pLed = &pledpriv->SwLed0;
 
        pledpriv->LedControlHandler = LedControl8188eu;
 
-       InitLed871x(padapter, &pledpriv->SwLed0);
+       pLed->padapter = padapter;
+       ResetLedStatus(pLed);
+       INIT_DELAYED_WORK(&pLed->blink_work, BlinkWorkItemCallback);
 }
 
 void rtl8188eu_DeInitSwLeds(struct adapter *padapter)
index d4ba5ac3e48af1a37d0d6e374031dd548a226333..2c4dc361e4691bec6cd94b1f6bae39cefb5aecc1 100644 (file)
@@ -104,7 +104,6 @@ void BlinkWorkItemCallback(struct work_struct *work);
 
 void ResetLedStatus(struct LED_871x * pLed);
 
-void InitLed871x(struct adapter *padapter, struct LED_871x *pLed);
 void DeInitLed871x(struct LED_871x *pLed);
 
 void rtl8188eu_InitSwLeds(struct adapter *padapter);