On Mon Jan 20 15:10:38 2025 +0200, Tomi Valkeinen wrote:
> Use v4l2_subdev_{enable|disable}_streams() instead of calling s_stream
> op directly. This allows the called subdev to drop the legacy s_stream
> op.
> 
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen+rene...@ideasonboard.com>
> Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
> Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/renesas/rcar-vin/rcar-dma.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

---

diff --git a/drivers/media/platform/renesas/rcar-vin/rcar-dma.c 
b/drivers/media/platform/renesas/rcar-vin/rcar-dma.c
index 8773998101ff..8de871240440 100644
--- a/drivers/media/platform/renesas/rcar-vin/rcar-dma.c
+++ b/drivers/media/platform/renesas/rcar-vin/rcar-dma.c
@@ -1397,7 +1397,7 @@ static int rvin_set_stream(struct rvin_dev *vin, int on)
 
        if (!on) {
                video_device_pipeline_stop(&vin->vdev);
-               return v4l2_subdev_call(sd, video, s_stream, 0);
+               return v4l2_subdev_disable_streams(sd, pad->index, BIT_ULL(0));
        }
 
        ret = rvin_mc_validate_format(vin, sd, pad);
@@ -1408,7 +1408,7 @@ static int rvin_set_stream(struct rvin_dev *vin, int on)
        if (ret)
                return ret;
 
-       ret = v4l2_subdev_call(sd, video, s_stream, 1);
+       ret = v4l2_subdev_enable_streams(sd, pad->index, BIT_ULL(0));
        if (ret == -ENOIOCTLCMD)
                ret = 0;
        if (ret)

Reply via email to