diff options
| author |   Martin Kaiser <martin@kaiser.cx> | 2022-04-13 21:36:50 +0200 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2022-04-14 09:06:59 +0200 |
| commit | 294e69c69c4d4a4e29dda2a0b5a5c11cf53ea7f4 (patch) | |
| tree | b25cd95e6094e00cf55a763f23a1b3b7dab263ed | |
| parent | staging: r8188eu: simplify the ps_processing check (diff) | |
| download | linux-dev-294e69c69c4d4a4e29dda2a0b5a5c11cf53ea7f4.tar.xz linux-dev-294e69c69c4d4a4e29dda2a0b5a5c11cf53ea7f4.zip | |
staging: r8188eu: summarize two if statements
Summarize two if statements in rtw_pwr_wakeup and place the constants
on the right side of the comparison.
Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Link: https://lore.kernel.org/r/20220413193654.258507-5-martin@kaiser.cx
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/r8188eu/core/rtw_pwrctrl.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/staging/r8188eu/core/rtw_pwrctrl.c b/drivers/staging/r8188eu/core/rtw_pwrctrl.c index fd7ea83968ed..ff96e5229b52 100644 --- a/drivers/staging/r8188eu/core/rtw_pwrctrl.c +++ b/drivers/staging/r8188eu/core/rtw_pwrctrl.c @@ -394,11 +394,10 @@ int rtw_pwr_wakeup(struct adapter *padapter) ret = _SUCCESS; goto exit; } - if (rf_off == pwrpriv->rf_pwrstate) { - if (_FAIL == ips_leave(padapter)) { - ret = _FAIL; - goto exit; - } + + if (pwrpriv->rf_pwrstate == rf_off && ips_leave(padapter) == _FAIL) { + ret = _FAIL; + goto exit; } if (padapter->bDriverStopped || !padapter->bup || !padapter->hw_init_completed) { |