Re: [PATCH v2 5/5] staging: rtl8723bs: core: Remove return variable in rtw_io.c

[Greg KH <gregkh@linuxfoundation.org>]

On Fri, Mar 22, 2019 at 08:08:27PM +0530, Nishka Dasgupta wrote:
> Remove unnecessary local return variable in rtw_io.c. Issue found with
> Coccinelle using ret.cocci.
> 
> Signed-off-by: Nishka Dasgupta <nishka.dasgupta@yahoo.com>
> ---
> Changes in v2:
> - Remove trailing whitespace.
> 
>  drivers/staging/rtl8723bs/core/rtw_io.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/rtl8723bs/core/rtw_io.c b/drivers/staging/rtl8723bs/core/rtw_io.c
> index d341069097e2..985eaa49ad3c 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_io.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_io.c
> @@ -45,8 +45,7 @@ u8 _rtw_read8(struct adapter *adapter, u32 addr)
>  
>  	_read8 = pintfhdl->io_ops._read8;
>  
> -	r_val = _read8(pintfhdl, addr);
> -	return r_val;
> +	return _read8(pintfhdl, addr);

This did not give you a build warning when you applied it?  Odd, let me
check...