commit 90ac1ea515161f4bb2193ad59110aa8e6fe79eac
author Weiyin Jiang <wjiang@codeaurora.org> Thu Apr 13 14:18:23 2017 +0800
committer Gerrit - the friendly Code Review server <code-review@localhost> Fri Apr 14 01:15:08 2017 -0700
tree 6d4f69caed14167cfd8e62d84af8d563914f0d70
parent e8ebd830f206dfa96c4a24a21f10e5f68e987656

post_proc: return proper volume in reply data for offload effect

Since EFFECT_FLAG_VOLUME_CTRL flag is set in non offload effect
implementations, EFFECT_CMD_SET_VOLUME should be handled in the same
manner as AOSP effect bundle.

In order for volume control to work, return the received volume in the
reply data.

CRs-Fixed: 2033812
Change-Id: Iee4cd3b80c1470034a208f0c5a014f10042a1ce2

post_proc/equalizer.c

diff --git a/post_proc/equalizer.c b/post_proc/equalizer.c
index a9da8b4..45f8eb8 100644
--- a/post_proc/equalizer.c
+++ b/post_proc/equalizer.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2013-2014, The Linux Foundation. All rights reserved.
+ * Copyright (c) 2013-2014, 2017, The Linux Foundation. All rights reserved.
  * Not a Contribution.
  *
  * Copyright (C) 2013 The Android Open Source Project
@@ -34,7 +34,7 @@
         {0x0bed4300, 0xddd6, 0x11db, 0x8f34, {0x00, 0x02, 0xa5, 0xd5, 0xc5, 0x1b}}, // type
         {0xa0dac280, 0x401c, 0x11e3, 0x9379, {0x00, 0x02, 0xa5, 0xd5, 0xc5, 0x1b}}, // uuid
         EFFECT_CONTROL_API_VERSION,
-        (EFFECT_FLAG_TYPE_INSERT | EFFECT_FLAG_HW_ACC_TUNNEL),
+        (EFFECT_FLAG_TYPE_INSERT | EFFECT_FLAG_HW_ACC_TUNNEL | EFFECT_FLAG_VOLUME_CTRL),
         0, /* TODO */
         1,
         "MSM offload equalizer",
@@ -196,7 +196,7 @@
 int equalizer_get_num_presets(equalizer_context_t *context)
 {
     ALOGV("%s: ctxt %p, presets_num: %d", __func__, context,
-           sizeof(equalizer_preset_names)/sizeof(char *));
+           (int)(sizeof(equalizer_preset_names)/sizeof(char *)));
     return sizeof(equalizer_preset_names)/sizeof(char *);
 }