# Re: \[PATCH v8 09/19\] staging: r8188eu: remove unnecessary braces in usbctrl\_vendorreq()

- From: Greg Kroah-Hartman \<gregkh@linuxfoundation.org\>
- Date: 2021-09-20 11:49:45 
- List: linux-staging
- Page: https://ratatoskr.run/linux-staging/2021/09/1514531
- Thread: https://ratatoskr.run/linux-staging/2021/09/1514201/t.md

```text
On Mon, Sep 20, 2021 at 01:53:46AM +0200, Fabio M. De Francesco wrote:
> Remove unnecessary braces in usbctrl_vendorreq().
> 
> Co-developed-by: Pavel Skripkin <redacted>
> Signed-off-by: Pavel Skripkin <redacted>
> Signed-off-by: Fabio M. De Francesco <redacted>
> ---
>  drivers/staging/r8188eu/hal/usb_ops_linux.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/r8188eu/hal/usb_ops_linux.c b/drivers/staging/r8188eu/hal/usb_ops_linux.c
> index 3ca2959f4bcd..a270cb4249b5 100644
> --- a/drivers/staging/r8188eu/hal/usb_ops_linux.c
> +++ b/drivers/staging/r8188eu/hal/usb_ops_linux.c
> @@ -72,9 +72,8 @@ static int usbctrl_vendorreq(struct intf_hdl *intfhdl, u16 value, void *data, u1
>  			} else {
>  				/* status != len && status >= 0 */
>  				if (status > 0) {
> -					if (requesttype == REALTEK_USB_VENQT_READ) {
> +					if (requesttype == REALTEK_USB_VENQT_READ)
>  						memcpy(data, io_buf,  len);
> -					}

This should have gone in the previous commit.


```