Commit 0189ef82 authored May 12, 2022 by Oliver Neukum Committed by Mauro Carvalho Chehab Jun 20, 2022

media: iguanair: no superfluous usb_unlink_urb()



Unlinking yourself while the completion handler
is running is a NOP. Drop it.

Signed-off-by: Oliver Neukum <oneukum@suse.com>
Signed-off-by: Sean Young <sean@mess.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>

parent 5e402905

drivers/media/rc/iguanair.c

+1 −4

Original line number	Diff line number	Diff line
		@@ -149,10 +149,8 @@ static void iguanair_rx(struct urb *urb)
		return;

		ir = urb->context;
		if (!ir) {
		usb_unlink_urb(urb);
		if (!ir)
		return;
		}

		switch (urb->status) {
		case 0:
		@@ -161,7 +159,6 @@ static void iguanair_rx(struct urb *urb)
		case -ECONNRESET:
		case -ENOENT:
		case -ESHUTDOWN:
		usb_unlink_urb(urb);
		return;
		case -EPIPE:
		default: