media: meson: vdec: remove redundant if statement
authorBenjamin Roszak <benjamin545@gmail.com>
Mon, 23 Jan 2023 07:17:08 +0000 (08:17 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Wed, 8 Feb 2023 08:03:03 +0000 (09:03 +0100)
Checking if sess->fmt_out->pixfmt is V4L2_PIX_FMT_VP9 was already done as a
condition to enter the if statement where this additional check is made.

Signed-off-by: Benjamin Roszak <benjamin545@gmail.com>
Signed-off-by: Christian Hewitt <christianshewitt@gmail.com>
Acked-by: Neil Armstrong <neil.armstrong@linaro.org>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/staging/media/meson/vdec/esparser.c

index 86ccc89..7b15fc5 100644 (file)
@@ -314,8 +314,7 @@ esparser_queue(struct amvdec_session *sess, struct vb2_v4l2_buffer *vbuf)
                        num_dst_bufs = codec_ops->num_pending_bufs(sess);
 
                num_dst_bufs += v4l2_m2m_num_dst_bufs_ready(sess->m2m_ctx);
-               if (sess->fmt_out->pixfmt == V4L2_PIX_FMT_VP9)
-                       num_dst_bufs -= 3;
+               num_dst_bufs -= 3;
 
                if (esparser_vififo_get_free_space(sess) < payload_size ||
                    atomic_read(&sess->esparser_queued_bufs) >= num_dst_bufs)