omxh26{4,5}enc: don't pick default 10-bit profile
authorGuillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Tue, 13 Mar 2018 15:15:30 +0000 (16:15 +0100)
committerGuillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Thu, 26 Apr 2018 08:07:54 +0000 (10:07 +0200)
The OMX stack of the zynqultrascaleplus (the only one supporting
NV12_10LE32 and NV16_10LE32) will now pick the proper profile if none
has been requested. Best to rely on its default than hardcoding a
specific one in gst-omx.

https://bugzilla.gnome.org/show_bug.cgi?id=794319

omx/gstomxh264enc.c
omx/gstomxh265enc.c

index 631e31d..4c4b3b2 100644 (file)
@@ -676,19 +676,6 @@ gst_omx_h264_enc_set_format (GstOMXVideoEnc * enc, GstOMXPort * port,
     gst_caps_unref (peercaps);
   }
 
-  /* Change default profile to high-10 if input is 10 bits */
-  if (profile == OMX_VIDEO_AVCProfileMax) {
-    GstVideoFormat format;
-
-    format = state->info.finfo->format;
-    if (format == GST_VIDEO_FORMAT_NV12_10LE32 ||
-        format == GST_VIDEO_FORMAT_NV16_10LE32) {
-      GST_DEBUG_OBJECT (self,
-          "Set profile to high-10 as input is a 10 bits format");
-      profile = OMX_VIDEO_AVCProfileHigh10;
-    }
-  }
-
   if (profile != OMX_VIDEO_AVCProfileMax || level != OMX_VIDEO_AVCLevelMax) {
     /* OMX provides 2 API to set the profile and level. We try using the
      * generic one here and the H264 specific when calling
index e099431..68fe432 100644 (file)
@@ -494,19 +494,6 @@ gst_omx_h265_enc_set_format (GstOMXVideoEnc * enc, GstOMXPort * port,
     gst_caps_unref (peercaps);
   }
 
-  /* Change default profile to main-10 if input is 10 bits */
-  if (profile == OMX_VIDEO_HEVCProfileUnknown) {
-    GstVideoFormat format;
-
-    format = state->info.finfo->format;
-    if (format == GST_VIDEO_FORMAT_NV12_10LE32 ||
-        format == GST_VIDEO_FORMAT_NV16_10LE32) {
-      GST_DEBUG_OBJECT (self,
-          "Set profile to main-10 as input is a 10 bits format");
-      profile = OMX_VIDEO_HEVCProfileMain10;
-    }
-  }
-
   if (profile != OMX_VIDEO_HEVCProfileUnknown
       || level != OMX_VIDEO_HEVCLevelUnknown) {
     /* OMX provides 2 API to set the profile and level. We try using the