OSDN Git Service

staging: r8188eu: summarize two if statements
authorMartin Kaiser <martin@kaiser.cx>
Wed, 13 Apr 2022 19:36:50 +0000 (21:36 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 14 Apr 2022 07:06:59 +0000 (09:06 +0200)
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>
drivers/staging/r8188eu/core/rtw_pwrctrl.c

index fd7ea83..ff96e52 100644 (file)
@@ -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) {