staging: r8188eu: remove LedPin from struct struct LED_871x
authorMartin Kaiser <martin@kaiser.cx>
Tue, 7 Dec 2021 21:05:37 +0000 (22:05 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 9 Dec 2021 07:57:03 +0000 (08:57 +0100)
We only support a single LED. LedPin in struct struct LED_871x is
always LED_PIN_LED0. There's no need to store this info or to pass it
to functions as a parameter.

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

index 1eda366c61aad40feb81ade42cb610f92230e525..e1be1ba189cb47a03dce27285571487b3479bc60 100644 (file)
@@ -28,10 +28,9 @@ void ResetLedStatus(struct LED_871x *pLed)
        pLed->bLedScanBlinkInProgress = false;
 }
 
-void InitLed871x(struct adapter *padapter, struct LED_871x *pLed, enum LED_PIN_871x LedPin)
+void InitLed871x(struct adapter *padapter, struct LED_871x *pLed)
 {
        pLed->padapter = padapter;
-       pLed->LedPin = LedPin;
 
        ResetLedStatus(pLed);
 
index dd692d6a3ab7148efbdce8b1306d80857e8cb64f..6fd1d1d476c7ed24121cd76e1e2bf59b89035b9e 100644 (file)
@@ -8,9 +8,6 @@
 
 /*  LED object. */
 
-/*  LED_819xUsb routines. */
-/*     Description: */
-/*             Turn on LED according to LedPin specified. */
 void SwLedOn(struct adapter *padapter, struct LED_871x *pLed)
 {
        u8      LedCfg;
@@ -23,8 +20,6 @@ void SwLedOn(struct adapter *padapter, struct LED_871x *pLed)
        pLed->bLedOn = true;
 }
 
-/*     Description: */
-/*             Turn off LED according to LedPin specified. */
 void SwLedOff(struct adapter *padapter, struct LED_871x *pLed)
 {
        u8      LedCfg;
@@ -54,7 +49,7 @@ void rtl8188eu_InitSwLeds(struct adapter *padapter)
 
        pledpriv->LedControlHandler = LedControl8188eu;
 
-       InitLed871x(padapter, &pledpriv->SwLed0, LED_PIN_LED0);
+       InitLed871x(padapter, &pledpriv->SwLed0);
 }
 
 /*     Description: */
index ff182878e3712178082b980403619bc09851ab1b..e9b6d7768c2d0e913245955d798352db03ccee47 100644 (file)
@@ -51,15 +51,9 @@ enum LED_STATE_871x {
        LED_BLINK_ALWAYS_ON = 16,
 };
 
-enum LED_PIN_871x {
-       LED_PIN_LED0 = 1,
-};
-
 struct LED_871x {
        struct adapter *padapter;
 
-       enum LED_PIN_871x       LedPin; /* Identify how to implement this
-                                        * SW led. */
        enum LED_STATE_871x     CurrLedState; /*  Current LED state. */
        enum LED_STATE_871x     BlinkingLedState; /*  Next state for blinking,
                                   * either RTW_LED_ON or RTW_LED_OFF are. */
@@ -110,9 +104,7 @@ void BlinkWorkItemCallback(struct work_struct *work);
 
 void ResetLedStatus(struct LED_871x * pLed);
 
-void InitLed871x(struct adapter *padapter, struct LED_871x *pLed,
-                enum LED_PIN_871x LedPin);
-
+void InitLed871x(struct adapter *padapter, struct LED_871x *pLed);
 void DeInitLed871x(struct LED_871x *pLed);
 
 /* hal... */