media: mediatek: vcodec: Stop direct calls to queue num_buffers field
authorBenjamin Gaignard <benjamin.gaignard@collabora.com>
Thu, 9 Nov 2023 16:34:29 +0000 (17:34 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Thu, 23 Nov 2023 11:02:17 +0000 (12:02 +0100)
Use vb2_get_num_buffers() to avoid using queue num_buffers field directly.
This allows us to change how the number of buffers is computed in the
future.

Signed-off-by: Benjamin Gaignard <benjamin.gaignard@collabora.com>
Reviewed-by: Andrzej Pietrasiewicz <andrzej.p@collabora.com>
CC: Bin Liu <bin.liu@mediatek.com>
CC: Matthias Brugger <matthias.bgg@gmail.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/platform/mediatek/vcodec/encoder/mtk_vcodec_enc.c

index eb381fa6e7d14ec94a774afa1bb39d5399374e6f..181884e798fd4df962881bc89c73fcbf6a7ca7b1 100644 (file)
@@ -912,7 +912,7 @@ static int vb2ops_venc_start_streaming(struct vb2_queue *q, unsigned int count)
        return 0;
 
 err_start_stream:
-       for (i = 0; i < q->num_buffers; ++i) {
+       for (i = 0; i < vb2_get_num_buffers(q); ++i) {
                struct vb2_buffer *buf = vb2_get_buffer(q, i);
 
                /*