From 3ecb2a82beda08b7ade45bdd12018dbe6b34e9ac Mon Sep 17 00:00:00 2001
From: Haihao Xiang <haihao.xiang@intel.com>
Date: Thu, 9 Jul 2020 09:01:23 +0800
Subject: [PATCH] Revert "msdkh265enc: let MSDK select the encoding mode by
 default"

This reverts commit 9e977832c1761e3ad002fdfe14a9c3881a60bd45.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/1426>
---
 sys/msdk/gstmsdkh265enc.c | 15 ++++++---------
 sys/msdk/gstmsdkh265enc.h |  2 +-
 2 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/sys/msdk/gstmsdkh265enc.c b/sys/msdk/gstmsdkh265enc.c
index 177dfda..46aa375 100644
--- a/sys/msdk/gstmsdkh265enc.c
+++ b/sys/msdk/gstmsdkh265enc.c
@@ -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",
diff --git a/sys/msdk/gstmsdkh265enc.h b/sys/msdk/gstmsdkh265enc.h
index d4c323c..7b62e4a 100644
--- a/sys/msdk/gstmsdkh265enc.h
+++ b/sys/msdk/gstmsdkh265enc.h
@@ -55,7 +55,7 @@ struct _GstMsdkH265Enc
 {
   GstMsdkEnc base;
 
-  gint lowpower;
+  gboolean lowpower;
   gushort num_tile_rows;
   gushort num_tile_cols;
   guint max_slice_size;
-- 
2.7.4