media: cedrus: remove superfluous call
authorJernej Skrabec <jernej.skrabec@gmail.com>
Wed, 2 Nov 2022 18:08:00 +0000 (19:08 +0100)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Fri, 4 Nov 2022 15:56:31 +0000 (16:56 +0100)
cedrus_try_fmt_vid_out() is called two times inside
cedrus_s_fmt_vid_out(), but nothing changes between calls which would
influence output format. Remove first call, which was added later.

Acked-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
Signed-off-by: Jernej Skrabec <jernej.skrabec@gmail.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
drivers/staging/media/sunxi/cedrus/cedrus_video.c

index 66714609b577cf79336499a55deec5ace63a818d..3c31dd141027663b1ab6a678ae08dc00c87c7bd8 100644 (file)
@@ -309,10 +309,6 @@ static int cedrus_s_fmt_vid_out(struct file *file, void *priv,
        struct vb2_queue *peer_vq;
        int ret;
 
-       ret = cedrus_try_fmt_vid_out(file, priv, f);
-       if (ret)
-               return ret;
-
        vq = v4l2_m2m_get_vq(ctx->fh.m2m_ctx, f->type);
        /*
         * In order to support dynamic resolution change,