media: async: Warn on attept to create an ancillary link to a non-subdev
authorSakari Ailus <sakari.ailus@linux.intel.com>
Mon, 20 May 2024 07:28:20 +0000 (10:28 +0300)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Tue, 4 Jun 2024 06:31:24 +0000 (08:31 +0200)
Warn if a lens or a flash async sub-device is bound to a non-sub-device
notifier. This isn't meaningful and unlikely to happen in practice but
print a warning as we nevertheless won't consider this an error.

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 c477723c07bf8e1cfe96659041ceafecd966ba6a..81a9b5473969d29b058375bb544382fce504a1c9 100644 (file)
@@ -323,8 +323,12 @@ static int v4l2_async_create_ancillary_links(struct v4l2_async_notifier *n,
            sd->entity.function != MEDIA_ENT_F_FLASH)
                return 0;
 
-       if (!n->sd)
+       if (!n->sd) {
+               dev_warn(notifier_dev(n),
+                        "not a sub-device notifier, not creating an ancillary link for %s!\n",
+                        dev_name(sd->dev));
                return 0;
+       }
 
        link = media_create_ancillary_link(&n->sd->entity, &sd->entity);