Staging: bcm: led_control.c: Removed dead code
authorMatthias Beyer <mail@beyermatthias.de>
Mon, 23 Jun 2014 19:18:24 +0000 (21:18 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 27 Jun 2014 01:02:13 +0000 (21:02 -0400)
This patch removes the never-entered else-block inside
blink_in_normal_bandwidth(). The condition of the preceding if statement
is always true.

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/bcm/led_control.c

index 9808f536d6fb50af2a88c6f48d37441e5928430e..4cf4212dd507e334485abf056cd6608ea0e70eef 100644 (file)
@@ -108,74 +108,50 @@ static INT blink_in_normal_bandwidth(struct bcm_mini_adapter *ad,
                                     enum bcm_led_events currdriverstate,
                                     ulong *timeout)
 {
-       bool bBlinkBothLED = TRUE;
-
        /*
-        * Blink Tx and Rx LED when both Tx and Rx is
-        * in normal bandwidth
+        * Assign minimum number of blinks of
+        * either Tx or Rx.
         */
-       if (bBlinkBothLED) {
-               /*
-                * Assign minimum number of blinks of
-                * either Tx or Rx.
-                */
-               if (*num_of_time_tx > *num_of_time_rx)
-                       *num_of_time = *num_of_time_rx;
-               else
-                       *num_of_time = *num_of_time_tx;
-               if (*num_of_time > 0) {
-                       /* Blink both Tx and Rx LEDs */
-                       if (LED_Blink(ad, 1 << GPIO_Num_tx,
-                                       uiTxLedIndex, *timeout,
-                                       *num_of_time, currdriverstate)
-                                               == EVENT_SIGNALED)
-                               return EVENT_SIGNALED;
-
-                       if (LED_Blink(ad, 1 << GPIO_Num_rx,
-                                       uiRxLedIndex, *timeout,
-                                       *num_of_time, currdriverstate)
-                                               == EVENT_SIGNALED)
-                               return EVENT_SIGNALED;
+       if (*num_of_time_tx > *num_of_time_rx)
+               *num_of_time = *num_of_time_rx;
+       else
+               *num_of_time = *num_of_time_tx;
+       if (*num_of_time > 0) {
+               /* Blink both Tx and Rx LEDs */
+               if (LED_Blink(ad, 1 << GPIO_Num_tx,
+                               uiTxLedIndex, *timeout,
+                               *num_of_time, currdriverstate)
+                                       == EVENT_SIGNALED)
+                       return EVENT_SIGNALED;
 
-               }
+               if (LED_Blink(ad, 1 << GPIO_Num_rx,
+                               uiRxLedIndex, *timeout,
+                               *num_of_time, currdriverstate)
+                                       == EVENT_SIGNALED)
+                       return EVENT_SIGNALED;
 
-               if (*num_of_time == *num_of_time_tx) {
-                       /* Blink pending rate of Rx */
-                       if (LED_Blink(ad, (1 << GPIO_Num_rx),
-                                       uiRxLedIndex, *timeout,
-                                       *num_of_time_rx - *num_of_time,
-                                       currdriverstate)
-                                               == EVENT_SIGNALED)
-                               return EVENT_SIGNALED;
+       }
 
-                       *num_of_time = *num_of_time_rx;
-               } else {
-                       /* Blink pending rate of Tx */
-                       if (LED_Blink(ad, 1 << GPIO_Num_tx,
-                                       uiTxLedIndex, *timeout,
-                                       *num_of_time_tx - *num_of_time,
-                                       currdriverstate)
-                                               == EVENT_SIGNALED)
-                               return EVENT_SIGNALED;
+       if (*num_of_time == *num_of_time_tx) {
+               /* Blink pending rate of Rx */
+               if (LED_Blink(ad, (1 << GPIO_Num_rx),
+                               uiRxLedIndex, *timeout,
+                               *num_of_time_rx - *num_of_time,
+                               currdriverstate)
+                                       == EVENT_SIGNALED)
+                       return EVENT_SIGNALED;
 
-                       *num_of_time = *num_of_time_tx;
-               }
+               *num_of_time = *num_of_time_rx;
        } else {
-               if (*num_of_time == *num_of_time_tx) {
-                       /* Blink pending rate of Rx */
-                       if (LED_Blink(ad, 1 << GPIO_Num_tx,
-                                       uiTxLedIndex, *timeout,
-                                       *num_of_time, currdriverstate)
-                                               == EVENT_SIGNALED)
-                               return EVENT_SIGNALED;
-               } else {
-                       /* Blink pending rate of Tx */
-                       if (LED_Blink(ad, 1 << GPIO_Num_rx,
-                                       uiRxLedIndex, *timeout,
-                                       *num_of_time, currdriverstate)
-                                               == EVENT_SIGNALED)
-                               return EVENT_SIGNALED;
-               }
+               /* Blink pending rate of Tx */
+               if (LED_Blink(ad, 1 << GPIO_Num_tx,
+                               uiTxLedIndex, *timeout,
+                               *num_of_time_tx - *num_of_time,
+                               currdriverstate)
+                                       == EVENT_SIGNALED)
+                       return EVENT_SIGNALED;
+
+               *num_of_time = *num_of_time_tx;
        }
 
        return 0;