commit e5198f07f476ca4b29244be153ccb40f06d50b0c
author TreeHugger Robot <treehugger-gerrit@google.com> Fri Oct 27 00:42:36 2017 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Fri Oct 27 00:42:36 2017 +0000
tree 5b9f9e1bf295a22e5ff595a1185540fe9bab5ea1
parent d743e3b6dcf53b7e329a4126cce9f0bca634177c
parent a3c98d8c1fd70d6c480b9e98d44fc1b16a899247

Merge "Fix the double PCM open during HFP call"

hal/audio_extn/hfp.c

diff --git a/hal/audio_extn/hfp.c b/hal/audio_extn/hfp.c
index 07a9711..ad1530a 100644
--- a/hal/audio_extn/hfp.c
+++ b/hal/audio_extn/hfp.c
@@ -438,10 +438,17 @@
     ret = str_parms_get_str(parms, AUDIO_PARAMETER_HFP_ENABLE, value,
                             sizeof(value));
     if (ret >= 0) {
-           if (!strncmp(value,"true",sizeof(value)))
-               ret = start_hfp(adev,parms);
-           else
-               stop_hfp(adev);
+           if (!strncmp(value,"true",sizeof(value))) {
+               if (!hfpmod.is_hfp_running)
+                   start_hfp(adev,parms);
+               else
+                   ALOGW("%s: HFP is already active.", __func__);
+           } else {
+               if (hfpmod.is_hfp_running)
+                   stop_hfp(adev);
+               else
+                   ALOGW("%s: ignore STOP, HFC not active", __func__);
+           }
     }
     memset(value, 0, sizeof(value));
     ret = str_parms_get_str(parms,AUDIO_PARAMETER_HFP_SET_SAMPLING_RATE, value,