From ae5361b7fa961627bbea35765cb303cbcf337e80 Mon Sep 17 00:00:00 2001
From: He Junyan <junyan.he@intel.com>
Date: Fri, 19 Apr 2024 00:30:47 +0800
Subject: [PATCH] vabaseenc: No need to call _finish_subframe()

After vaav1enc is aligned to TU, there is no case that generates
multi output for one input.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/6688>
---
 subprojects/gst-plugins-bad/sys/va/gstvabaseenc.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/subprojects/gst-plugins-bad/sys/va/gstvabaseenc.c b/subprojects/gst-plugins-bad/sys/va/gstvabaseenc.c
index 823ba24524..511215e054 100644
--- a/subprojects/gst-plugins-bad/sys/va/gstvabaseenc.c
+++ b/subprojects/gst-plugins-bad/sys/va/gstvabaseenc.c
@@ -478,12 +478,9 @@ _push_buffer_to_downstream (GstVaBaseEnc * base, GstVideoCodecFrame * frame)
   if (complete) {
     ret = gst_video_encoder_finish_frame (GST_VIDEO_ENCODER (base), frame);
   } else {
-    if (frame->output_buffer) {
-      ret = gst_video_encoder_finish_subframe (GST_VIDEO_ENCODER (base), frame);
-    } else {
-      /* Allow to output later and no data here. */
-      ret = GST_FLOW_OK;
-    }
+    /* Allow to output later and no data here. */
+    g_assert (!frame->output_buffer);
+    ret = GST_FLOW_OK;
   }
 
   return ret;