[PATCH] staging: rtl8188eu: os_dep: usb_intf.c: Cleaning up unnecessary code

navin patidar navin.patidar at gmail.com
Fri Jun 6 16:47:25 UTC 2014


sorry,  my script picked up Rickard's patch and send it out again
along my patches.
I need to fix my script.

regards,
navin patidar

On Fri, Jun 6, 2014 at 9:45 PM, navin patidar <navin.patidar at gmail.com> wrote:
> From: Rickard Strandqvist <rickard_strandqvist at spectrumdigital.se>
>
> Removes unnecessary code that does not do anything useful.
>
> This was partly found using a static code analysis program called cppcheck.
>
> Signed-off-by: Rickard Strandqvist <rickard_strandqvist at spectrumdigital.se>
> ---
>  drivers/staging/rtl8188eu/os_dep/usb_intf.c |    8 +-------
>  1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/drivers/staging/rtl8188eu/os_dep/usb_intf.c b/drivers/staging/rtl8188eu/os_dep/usb_intf.c
> index 2e49cd5..fb3789e 100644
> --- a/drivers/staging/rtl8188eu/os_dep/usb_intf.c
> +++ b/drivers/staging/rtl8188eu/os_dep/usb_intf.c
> @@ -563,14 +563,8 @@ static int rtw_resume(struct usb_interface *pusb_intf)
>  {
>         struct dvobj_priv *dvobj = usb_get_intfdata(pusb_intf);
>         struct adapter *padapter = dvobj->if1;
> -       struct pwrctrl_priv *pwrpriv = &padapter->pwrctrlpriv;
> -        int ret = 0;
>
> -       if (pwrpriv->bInternalAutoSuspend)
> -               ret = rtw_resume_process(padapter);
> -       else
> -               ret = rtw_resume_process(padapter);
> -       return ret;
> +       return rtw_resume_process(padapter);
>  }
>
>  int rtw_resume_process(struct adapter *padapter)
> --
> 1.7.10.4


More information about the devel mailing list