Skip to content

Commit 04d3398

Browse files
ribaldamchehab
authored andcommitted
media: uvcvideo: Remove redundant NULL assignment
ctrl->handle will only be different than NULL for controls that have mappings. This is because that assignment is only done inside uvc_ctrl_set() for mapped controls. Cc: [email protected] Fixes: e5225c8 ("media: uvcvideo: Send a control event when a Control Change interrupt arrives") Reviewed-by: Laurent Pinchart <[email protected]> Reviewed-by: Hans de Goede <[email protected]> Signed-off-by: Ricardo Ribalda <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Laurent Pinchart <[email protected]> Signed-off-by: Mauro Carvalho Chehab <[email protected]>
1 parent d9fecd0 commit 04d3398

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

drivers/media/usb/uvc/uvc_ctrl.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1640,10 +1640,8 @@ bool uvc_ctrl_status_event_async(struct urb *urb, struct uvc_video_chain *chain,
16401640
struct uvc_device *dev = chain->dev;
16411641
struct uvc_ctrl_work *w = &dev->async_ctrl;
16421642

1643-
if (list_empty(&ctrl->info.mappings)) {
1644-
ctrl->handle = NULL;
1643+
if (list_empty(&ctrl->info.mappings))
16451644
return false;
1646-
}
16471645

16481646
w->data = data;
16491647
w->urb = urb;

0 commit comments

Comments
 (0)