commit 882a72dc4a76ff14bb65da0eab029a9f5a258dcd
author Eric Laurent <elaurent@google.com> Wed Apr 12 13:41:03 2017 -0700
committer Eric Laurent <elaurent@google.com> Wed Apr 12 13:41:03 2017 -0700
tree e8299d52ce7eb7f86ad7dd72195c69aa50838cf4
parent f568373527b570e692317c245858cd25da8223bc

post processing: add volume command support

By design of the effect offloading mechanism, offloaded implementations
must have the same behavior as non offloaded implementations.
As the non offloaded effects (EQ, Bass boost, Virtualizer ...) from the
AOSP effect bundle implement the EFFECT_FLAG_VOLUME_CTRL flag and
behavior, offloaded effects should.

In order for volume control to work, EFFECT_CMD_SET_VOLUME must be
implemented and return the received volume in the reply data.

Bug: 37281217
Test: Music playback in offload and non offload mode with and without EQ.
Change-Id: I2425f4dce95c8ab877db19cefbaa6226895b4015

post_proc/equalizer.c

diff --git a/post_proc/equalizer.c b/post_proc/equalizer.c
index c3a438f..a8e5d1f 100644
--- a/post_proc/equalizer.c
+++ b/post_proc/equalizer.c
@@ -31,7 +31,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",