Staging: rtl8188eu: core: Remove else after break
authorPuranjay Mohan <puranjay12@gmail.com>
Mon, 20 May 2019 17:58:51 +0000 (23:28 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 21 May 2019 06:16:40 +0000 (08:16 +0200)
Remove else after break statements to fix following checkpatch.pl
warnings:
WARNING: else is not generally useful after a break or return.

Signed-off-by: Puranjay Mohan <puranjay12@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8188eu/core/rtw_mlme.c

index 9a4aad5ec36555500023e3727e7878c8fa3deb84..0abb2df326453f8f0c06639709722bf29b5bf55b 100644 (file)
@@ -1330,11 +1330,10 @@ void _rtw_join_timeout_handler (struct timer_list *t)
                                        continue;
                                }
                                break;
-                       } else {
-                               DBG_88E("%s We've try roaming but fail\n", __func__);
-                               rtw_indicate_disconnect(adapter);
-                               break;
                        }
+                       DBG_88E("%s We've try roaming but fail\n", __func__);
+                       rtw_indicate_disconnect(adapter);
+                       break;
                }
        } else {
                rtw_indicate_disconnect(adapter);
@@ -2058,17 +2057,16 @@ void _rtw_roaming(struct adapter *padapter, struct wlan_network *tgt_network)
                        do_join_r = rtw_do_join(padapter);
                        if (do_join_r == _SUCCESS) {
                                break;
-                       } else {
-                               DBG_88E("roaming do_join return %d\n", do_join_r);
-                               pmlmepriv->to_roaming--;
+                       }
+                       DBG_88E("roaming do_join return %d\n", do_join_r);
+                       pmlmepriv->to_roaming--;
 
-                               if (pmlmepriv->to_roaming > 0) {
-                                       continue;
-                               } else {
-                                       DBG_88E("%s(%d) -to roaming fail, indicate_disconnect\n", __func__, __LINE__);
-                                       rtw_indicate_disconnect(padapter);
-                                       break;
-                               }
+                       if (pmlmepriv->to_roaming > 0) {
+                               continue;
+                       } else {
+                               DBG_88E("%s(%d) -to roaming fail, indicate_disconnect\n", __func__, __LINE__);
+                               rtw_indicate_disconnect(padapter);
+                               break;
                        }
                }
        }