media: microchip-isc: Drop v4l2_subdev_link_validate() for video devices
authorLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Tue, 18 Jun 2024 23:42:34 +0000 (02:42 +0300)
committerLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Mon, 26 Aug 2024 13:36:11 +0000 (16:36 +0300)
The v4l2_subdev_link_validate() function is a helper designed to
validate links whose sink is a subdev. When called on a link whose sink
is a video device, it only prints a warning and returns. As the
microchip-isc driver implements manual validate of the subdev to video
device link, we can just drop the v4l2_subdev_link_validate() to avoid
the warning.

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Reviewed-by: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com>
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
drivers/media/platform/microchip/microchip-isc-base.c

index f3a5cbacadbe0989a9033ca16765bc850e4e83c8..28e56f6a695da2a6e4035fbe99b80ff65e95517e 100644 (file)
@@ -902,8 +902,11 @@ static int isc_set_fmt(struct isc_device *isc, struct v4l2_format *f)
        return 0;
 }
 
-static int isc_validate(struct isc_device *isc)
+static int isc_link_validate(struct media_link *link)
 {
+       struct video_device *vdev =
+               media_entity_to_video_device(link->sink->entity);
+       struct isc_device *isc = video_get_drvdata(vdev);
        int ret;
        int i;
        struct isc_format *sd_fmt = NULL;
@@ -1906,20 +1909,6 @@ int microchip_isc_pipeline_init(struct isc_device *isc)
 }
 EXPORT_SYMBOL_GPL(microchip_isc_pipeline_init);
 
-static int isc_link_validate(struct media_link *link)
-{
-       struct video_device *vdev =
-               media_entity_to_video_device(link->sink->entity);
-       struct isc_device *isc = video_get_drvdata(vdev);
-       int ret;
-
-       ret = v4l2_subdev_link_validate(link);
-       if (ret)
-               return ret;
-
-       return isc_validate(isc);
-}
-
 static const struct media_entity_operations isc_entity_operations = {
        .link_validate = isc_link_validate,
 };