Comparison of 'ep' with NULL is useless since 'ep' is a result of container_of and cannot be NULL in any reasonable scenario. Found by Linux Verification Center (linuxtesting.org) with SVACE. Signed-off-by: Andrey Strachuk <strochuk@xxxxxxxxx> Fixes: 2dbc5c4c8314 ("usb: chipidea: get rid of camelcase names") --- drivers/usb/chipidea/udc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/chipidea/udc.c b/drivers/usb/chipidea/udc.c index 3b8bf6daf7d0..8ce31f9933e1 100644 --- a/drivers/usb/chipidea/udc.c +++ b/drivers/usb/chipidea/udc.c @@ -933,7 +933,7 @@ static int _ep_queue(struct usb_ep *ep, struct usb_request *req, struct ci_hdrc *ci = hwep->ci; int retval = 0; - if (ep == NULL || req == NULL || hwep->ep.desc == NULL) + if (req == NULL || hwep->ep.desc == NULL) return -EINVAL; if (hwep->type == USB_ENDPOINT_XFER_CONTROL) { -- 2.25.1