summaryrefslogtreecommitdiff
path: root/drivers/media/rc
diff options
context:
space:
mode:
authorOliver Neukum <oneukum@suse.com>2022-05-12 15:50:00 +0300
committerMauro Carvalho Chehab <mchehab@kernel.org>2022-06-20 12:30:33 +0300
commit0189ef82722894bc8c9361f90f18577977d58ef4 (patch)
tree9a9f53cd5cd475dc62076890595aad0f0522a9f7 /drivers/media/rc
parent5e4029056263f65a157eaee75c9306cd8254f567 (diff)
downloadlinux-0189ef82722894bc8c9361f90f18577977d58ef4.tar.xz
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>
Diffstat (limited to 'drivers/media/rc')
-rw-r--r--drivers/media/rc/iguanair.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/media/rc/iguanair.c b/drivers/media/rc/iguanair.c
index c9cb8277723f..276bf3c8a8cb 100644
--- a/drivers/media/rc/iguanair.c
+++ b/drivers/media/rc/iguanair.c
@@ -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: