commit 8e1225b28bac7bed714b8dab3b5c43961c539582
author hanghang.luo <hanghang.luo@amlogic.com> Tue Oct 10 08:54:28 2023 +0000
committer gerrit autosubmit <gerrit.autosubmit@amlogic.com> Wed Oct 11 23:03:46 2023 -0700
tree 38ed60a83446337bc09e11a6abd0a0df5566dec7
parent 2a06e040db6f2759323b09b0bb97834e7eb08f01

gst-plugin-aml-v4l2dec: CB1 set foramt when codec or resolution changed [1/1]

PD#SWPL-89188

Problem:
the "profile" field of casp is changed, should not set foramt

Solution:
set foramt when codec or resolution changed

Verify:
N/A

Change-Id: I843b7d1b1c4a530aef36f6505341a6c4c4a3693b
Signed-off-by: hanghang.luo <hanghang.luo@amlogic.com>

src/gstamlv4l2videodec.c

diff --git a/src/gstamlv4l2videodec.c b/src/gstamlv4l2videodec.c
index 4a275b4..9953ddc 100644
--- a/src/gstamlv4l2videodec.c
+++ b/src/gstamlv4l2videodec.c
@@ -347,6 +347,69 @@
 }
 
 static gboolean
+gst_aml_v4l2_video_dec_codec_chg(GstVideoDecoder *decoder,
+                                  GstVideoCodecState *state)
+{
+    GstAmlV4l2VideoDec *self = GST_AML_V4L2_VIDEO_DEC(decoder);
+    GstStructure *s_old = NULL;
+    GstStructure *s_new = NULL;
+
+    // first play, must set foramt;
+    if (!self->input_state)
+        return TRUE;
+
+    if (self->input_state->caps)
+         s_old = gst_caps_get_structure(self->input_state->caps,0);
+    if (state->caps)
+         s_new = gst_caps_get_structure(state->caps,0);
+
+    if (s_new && s_old && strcmp(gst_structure_get_name(s_new),gst_structure_get_name(s_old)))
+        return TRUE;
+    return FALSE;
+}
+
+static gboolean
+gst_aml_v4l2_video_dec_res_chg(GstVideoDecoder *decoder,
+                                  GstVideoCodecState *state)
+{
+    GstAmlV4l2VideoDec *self = GST_AML_V4L2_VIDEO_DEC(decoder);
+    gboolean ret = FALSE;
+    gint width_new = -1,height_new = -1,width_old = -1,height_old = -1;
+    GstStructure *s_old = NULL;
+    GstStructure *s_new = NULL;
+
+    // first play, must set foramt;
+    if (!self->input_state)
+    {
+        ret = TRUE;
+        goto done;
+    }
+
+    if (self->input_state->caps)
+         s_old = gst_caps_get_structure(self->input_state->caps,0);
+    if (state->caps)
+         s_new = gst_caps_get_structure(state->caps,0);
+
+    if (s_new && gst_structure_has_field(s_new,"width") && gst_structure_has_field(s_new,"height"))
+    {
+        gst_structure_get_int(s_new,"width",&width_new);
+        gst_structure_get_int(s_new,"height",&height_new);
+    }
+    if (s_old && gst_structure_has_field(s_old,"width") && gst_structure_has_field(s_old,"height"))
+    {
+        gst_structure_get_int(s_old,"width",&width_old);
+        gst_structure_get_int(s_old,"height",&height_old);
+    }
+
+    if (width_new != width_old || height_new != height_old)
+        ret = TRUE;
+
+done:
+    GST_DEBUG_OBJECT(self, "ret is %d",ret);
+    return ret;
+}
+
+static gboolean
 gst_aml_v4l2_video_dec_set_format(GstVideoDecoder *decoder,
                                   GstVideoCodecState *state)
 {
@@ -356,6 +419,14 @@
     GstCaps *caps;
 
     GST_DEBUG_OBJECT(self, "Setting format: %" GST_PTR_FORMAT, state->caps);
+    if (self->input_state)
+    {
+        if (gst_aml_v4l2_video_dec_res_chg(decoder,state) || gst_aml_v4l2_video_dec_codec_chg(decoder,state))
+            GST_DEBUG_OBJECT(self, "resolution or codec changed");
+        else
+            goto done;
+    }
+
     GstCapsFeatures *const features = gst_caps_get_features(state->caps, 0);
     GstStructure *s = gst_caps_get_structure(state->caps,0);
     if (s && gst_structure_has_field(s,"format"))