On Sun, Apr 18, 2021 at 10:18:13PM +0530, Saurav Girepunje wrote: > Fix the following coccicheck warning: > > drivers/staging/rtl8723bs/os_dep/os_intfs.c:1156:5-8: > Unneeded variable: "ret". Return "0" on line 1199 > > Signed-off-by: Saurav Girepunje <saurav.girepunje@xxxxxxxxxx> > --- > drivers/staging/rtl8723bs/os_dep/os_intfs.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c > index 1f34cb2e367c..920ec25c9a4f 100644 > --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c > +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c > @@ -1153,7 +1153,6 @@ int rtw_suspend_common(struct adapter *padapter) > struct pwrctrl_priv *pwrpriv = dvobj_to_pwrctl(psdpriv); > struct mlme_priv *pmlmepriv = &padapter->mlmepriv; > > - int ret = 0; > unsigned long start_time = jiffies; > > DBG_871X_LEVEL(_drv_always_, " suspend start\n"); > @@ -1196,7 +1195,7 @@ int rtw_suspend_common(struct adapter *padapter) > > exit: > > - return ret; > + return 0; > } > > static int rtw_resume_process_normal(struct adapter *padapter) > -- > 2.25.1 > > If this function can only ever return 0, why is it returning anything at all? Please fix it up to not do that. thanks, greg k-h
