OSDN Git Service

staging: rtl8723bs: remove pointless if else in rtw_sdio_resume()
authorMichael Straube <straube.linux@gmail.com>
Thu, 28 Jun 2018 17:02:52 +0000 (19:02 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 2 Jul 2018 08:39:07 +0000 (10:39 +0200)
Whether any of the conditions is true or not, the return variable
is always set to rtw_resume_process(padapter). Replace the if else
construct with a single call to rtw_resume_process(). Also remove
the now unused local variable pwrpriv.

Signed-off-by: Michael Straube <straube.linux@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/os_dep/sdio_intf.c

index 22191c9..6d02904 100644 (file)
@@ -605,7 +605,6 @@ static int rtw_sdio_resume(struct device *dev)
 {
        struct sdio_func *func =dev_to_sdio_func(dev);
        struct dvobj_priv *psdpriv = sdio_get_drvdata(func);
-       struct pwrctrl_priv *pwrpriv = dvobj_to_pwrctl(psdpriv);
        struct adapter *padapter = psdpriv->if1;
        struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv;
        int ret = 0;
@@ -615,25 +614,11 @@ static int rtw_sdio_resume(struct device *dev)
 
        pdbgpriv->dbg_resume_cnt++;
 
-       if (pwrpriv->bInternalAutoSuspend)
-       {
-               ret = rtw_resume_process(padapter);
-       }
-       else
-       {
-               if (pwrpriv->wowlan_mode || pwrpriv->wowlan_ap_mode)
-               {
-                       ret = rtw_resume_process(padapter);
-               }
-               else
-               {
-                       ret = rtw_resume_process(padapter);
-               }
-       }
+       ret = rtw_resume_process(padapter);
+
        pmlmeext->last_scan_time = jiffies;
        DBG_871X("<========  %s return %d\n", __func__, ret);
        return ret;
-
 }
 
 static int __init rtw_drv_entry(void)