Revert "msdkh265enc: let MSDK select the encoding mode by default"
authorHaihao Xiang <haihao.xiang@intel.com>
Thu, 9 Jul 2020 01:01:23 +0000 (09:01 +0800)
committerHaihao Xiang <haihao.xiang@intel.com>
Sat, 11 Jul 2020 13:57:52 +0000 (13:57 +0000)
This reverts commit 9e977832c1761e3ad002fdfe14a9c3881a60bd45.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/1426>

sys/msdk/gstmsdkh265enc.c
sys/msdk/gstmsdkh265enc.h

index 177dfda..46aa375 100644 (file)
@@ -48,7 +48,7 @@ enum
   PROP_MAX_SLICE_SIZE,
 };
 
-#define PROP_LOWPOWER_DEFAULT           -1
+#define PROP_LOWPOWER_DEFAULT           FALSE
 #define PROP_TILE_ROW_DEFAULT           1
 #define PROP_TILE_COL_DEFAULT           1
 #define PROP_MAX_SLICE_SIZE_DEFAULT     0
@@ -266,8 +266,7 @@ gst_msdkh265enc_configure (GstMsdkEnc * encoder)
   }
 
   encoder->param.mfx.LowPower =
-      (h265enc->lowpower == 1 ? MFX_CODINGOPTION_ON : (h265enc->lowpower ==
-          0 ? MFX_CODINGOPTION_OFF : MFX_CODINGOPTION_UNKNOWN));
+      (h265enc->lowpower ? MFX_CODINGOPTION_ON : MFX_CODINGOPTION_OFF);
 
   return TRUE;
 }
@@ -389,7 +388,7 @@ gst_msdkh265enc_set_property (GObject * object, guint prop_id,
 
   switch (prop_id) {
     case PROP_LOW_POWER:
-      thiz->lowpower = g_value_get_int (value);
+      thiz->lowpower = g_value_get_boolean (value);
       break;
 
     case PROP_TILE_ROW:
@@ -423,7 +422,7 @@ gst_msdkh265enc_get_property (GObject * object, guint prop_id, GValue * value,
   GST_OBJECT_LOCK (thiz);
   switch (prop_id) {
     case PROP_LOW_POWER:
-      g_value_set_int (value, thiz->lowpower);
+      g_value_set_boolean (value, thiz->lowpower);
       break;
 
     case PROP_TILE_ROW:
@@ -525,10 +524,8 @@ gst_msdkh265enc_class_init (GstMsdkH265EncClass * klass)
   gst_msdkenc_install_common_properties (encoder_class);
 
   g_object_class_install_property (gobject_class, PROP_LOW_POWER,
-      g_param_spec_int ("low-power", "Low power",
-          "Enable low power mode(-1: default, 0: disable, 1: enable)",
-          -1, 1, PROP_LOWPOWER_DEFAULT,
-          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+      g_param_spec_boolean ("low-power", "Low power", "Enable low power mode",
+          PROP_LOWPOWER_DEFAULT, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
 
   g_object_class_install_property (gobject_class, PROP_TILE_ROW,
       g_param_spec_uint ("num-tile-rows", "number of rows for tiled encoding",
index d4c323c..7b62e4a 100644 (file)
@@ -55,7 +55,7 @@ struct _GstMsdkH265Enc
 {
   GstMsdkEnc base;
 
-  gint lowpower;
+  gboolean lowpower;
   gushort num_tile_rows;
   gushort num_tile_cols;
   guint max_slice_size;