On Fri Jun 13 17:34:34 2025 +0200, Niklas Söderlund wrote:
> With Gen2 converted to use the common media device there is only one
> caller left for the helper to notify a video device of an event, fold it
> in.
> 
> Signed-off-by: Niklas Söderlund <niklas.soderlund+rene...@ragnatech.se>
> Reviewed-by: Laurent Pinchart <laurent.pinchart+rene...@ideasonboard.com>
> Link: 
> https://lore.kernel.org/r/20250613153434.2001800-14-niklas.soderlund+rene...@ragnatech.se
> Signed-off-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/renesas/rcar-vin/rcar-v4l2.c | 20 +++++++-------------
 1 file changed, 7 insertions(+), 13 deletions(-)

---

diff --git a/drivers/media/platform/renesas/rcar-vin/rcar-v4l2.c 
b/drivers/media/platform/renesas/rcar-vin/rcar-v4l2.c
index 3a4c7433f061..9b1e3a9d3249 100644
--- a/drivers/media/platform/renesas/rcar-vin/rcar-v4l2.c
+++ b/drivers/media/platform/renesas/rcar-vin/rcar-v4l2.c
@@ -656,18 +656,6 @@ void rvin_v4l2_unregister(struct rvin_dev *vin)
        video_unregister_device(&vin->vdev);
 }
 
-static void rvin_notify_video_device(struct rvin_dev *vin,
-                                    unsigned int notification, void *arg)
-{
-       switch (notification) {
-       case V4L2_DEVICE_NOTIFY_EVENT:
-               v4l2_event_queue(&vin->vdev, arg);
-               break;
-       default:
-               break;
-       }
-}
-
 static void rvin_notify(struct v4l2_subdev *sd,
                        unsigned int notification, void *arg)
 {
@@ -693,7 +681,13 @@ static void rvin_notify(struct v4l2_subdev *sd,
                if (remote != sd)
                        continue;
 
-               rvin_notify_video_device(vin, notification, arg);
+               switch (notification) {
+               case V4L2_DEVICE_NOTIFY_EVENT:
+                       v4l2_event_queue(&vin->vdev, arg);
+                       break;
+               default:
+                       break;
+               }
        }
 }
 

Reply via email to