diff options
| author | Martin Kaiser <martin@kaiser.cx> | 2022-04-09 19:32:03 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2022-04-12 16:49:57 +0300 |
| commit | 05a1e43e487693fb2d53778fe20f94d399549dfe (patch) | |
| tree | 3fd941d2af8e8f943fcb1802b35b87d7198f85e8 | |
| parent | 4b6c9a368967bf060a407a4ff2f182481b23b2ed (diff) | |
| download | linux-05a1e43e487693fb2d53778fe20f94d399549dfe.tar.xz | |
staging: r8188eu: no need for an else after return
The else branch at the end of send_beacon is not necessary. We return in
the if branch.
Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Link: https://lore.kernel.org/r/20220409163212.241122-2-martin@kaiser.cx
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/r8188eu/core/rtw_mlme_ext.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/drivers/staging/r8188eu/core/rtw_mlme_ext.c b/drivers/staging/r8188eu/core/rtw_mlme_ext.c index 31192b287698..d096056fb84b 100644 --- a/drivers/staging/r8188eu/core/rtw_mlme_ext.c +++ b/drivers/staging/r8188eu/core/rtw_mlme_ext.c @@ -5767,13 +5767,11 @@ unsigned int send_beacon(struct adapter *padapter) if (padapter->bSurpriseRemoved || padapter->bDriverStopped) return _FAIL; - if (!bxmitok) { + if (!bxmitok) return _FAIL; - } else { - rtw_get_passing_time_ms(start); - return _SUCCESS; - } + rtw_get_passing_time_ms(start); + return _SUCCESS; } bool get_beacon_valid_bit(struct adapter *adapter) |