media: v4l: async: Drop useless list move operation
authorSebastian Reichel <sre@kernel.org>
Mon, 13 Nov 2023 22:57:24 +0000 (23:57 +0100)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Mon, 4 Dec 2023 10:21:47 +0000 (11:21 +0100)
v4l2_async_unbind_subdev_one(), which is called in the line
following the list_move() operation contains list_move_tail()
for the same entry and overrides anything list_move() did.
Thus it can be removed.

Signed-off-by: Sebastian Reichel <sre@kernel.org>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
drivers/media/v4l2-core/v4l2-async.c

index 8cfd593d293d10d754b21ef828da1dd9bcacc30a..3ec323bd528b161e156284f35f43e006196fff71 100644 (file)
@@ -876,9 +876,6 @@ void v4l2_async_unregister_subdev(struct v4l2_subdev *sd)
        if (sd->asc_list.next) {
                list_for_each_entry_safe(asc, asc_tmp, &sd->asc_list,
                                         asc_subdev_entry) {
-                       list_move(&asc->asc_entry,
-                                 &asc->notifier->waiting_list);
-
                        v4l2_async_unbind_subdev_one(asc->notifier, asc);
                }
        }