Commit a6db0cd3 authored by Fabio M. De Francesco's avatar Fabio M. De Francesco Committed by Greg Kroah-Hartman
Browse files

staging: r8188eu: Remove a test from usbctrl_vendorreq() 



Remove an unnecessary 'if' test from usbctrl_vendorreq() because
"length" is never greater than MAX_VENDOR_REQ_CMD_SIZE.

Co-developed-by: default avatarPavel Skripkin <paskripkin@gmail.com>
Signed-off-by: default avatarPavel Skripkin <paskripkin@gmail.com>
Signed-off-by: default avatarFabio M. De Francesco <fmdefrancesco@gmail.com>
Link: https://lore.kernel.org/r/20210924122705.3781-13-fmdefrancesco@gmail.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 1b77e29e

drivers/staging/r8188eu/hal/usb_ops_linux.c

+0 −6
Original line number Diff line number Diff line
@@ -23,12 +23,6 @@ static int usbctrl_vendorreq(struct intf_hdl *intf, u16 value, void *data, u16 l 
		goto exit;

	}



	if (len > MAX_VENDOR_REQ_CMD_SIZE) {

		DBG_88E("[%s] Buffer len error ,vendor request failed\n", __func__);

		status = -EINVAL;

		goto exit;

	}



	mutex_lock(&dvobjpriv->usb_vendor_req_mutex);



	/*  Acquire IO memory for vendorreq */