media: mediatek: vcodec/venc: return 0 instead of 'ret'.
authorHans Verkuil <hverkuil-cisco@xs4all.nl>
Thu, 26 Jan 2023 12:59:16 +0000 (13:59 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Wed, 8 Feb 2023 07:39:14 +0000 (08:39 +0100)
Since 'ret' is known to be 0, just return '0'. This fixes a smatch warning:

venc_h264_if.c:568 h264_encode_frame() warn: missing error code? 'ret'

Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Cc: Andrew-CT Chen <andrew-ct.chen@mediatek.com>
Cc: Yunfei Dong <yunfei.dong@mediatek.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c

index 13c4f86..60fd165 100644 (file)
@@ -565,7 +565,7 @@ static int h264_encode_frame(struct venc_h264_inst *inst,
                       *bs_size);
                ++inst->frm_cnt;
                ++inst->skip_frm_cnt;
-               return ret;
+               return 0;
        }
 
        irq_status = h264_enc_wait_venc_done(inst);
@@ -580,7 +580,7 @@ static int h264_encode_frame(struct venc_h264_inst *inst,
        mtk_vcodec_debug(inst, "frm %d bs_size %d key_frm %d <-",
                         inst->frm_cnt, *bs_size, inst->vpu_inst.is_key_frm);
 
-       return ret;
+       return 0;
 }
 
 static void h264_encode_filler(struct venc_h264_inst *inst, void *buf,