commit 122b241e6bd95992b6a700d6b2cda6fac3decd07
author Tom Cherry <tomcherry@google.com> Thu Nov 17 17:07:54 2016 -0800
committer Tom Cherry <tomcherry@google.com> Thu Nov 17 17:07:54 2016 -0800
tree 02669b454ebe34123c366c0f31953414773d3bfb
parent c172b5dcedf23e7c8f98059c269388069d10200a

Fix the wrong volume setting on HFP call

hfp volume value should be saved before normalization.

Bug: 32975922
Change-Id: I49a386f207325d647ab3cac088fd2d98ec468284

hal/audio_extn/hfp.c

diff --git a/hal/audio_extn/hfp.c b/hal/audio_extn/hfp.c
index ebc0fd2..0ac01f1 100644
--- a/hal/audio_extn/hfp.c
+++ b/hal/audio_extn/hfp.c
@@ -79,6 +79,8 @@
     ALOGV("%s: entry", __func__);
     ALOGD("%s: (%f)\n", __func__, value);
 
+    hfpmod.hfp_volume = value;
+
     if (value < 0.0) {
         ALOGW("%s: (%f) Under 0.0, assuming 0.0\n", __func__, value);
         value = 0.0;
@@ -87,7 +89,6 @@
         ALOGW("%s: Volume brought with in range (%f)\n", __func__, value);
     }
     vol  = lrint((value * 0x2000) + 0.5);
-    hfpmod.hfp_volume = value;
 
     if (!hfpmod.is_hfp_running) {
         ALOGV("%s: HFP not active, ignoring set_hfp_volume call", __func__);