commit 5bcf706e04f46b7b3e85ec37f4a0e61094ed3c29
author Linux Build Service Account <lnxbuild@localhost> Tue Aug 05 00:14:23 2014 -0700
committer Gerrit - the friendly Code Review server <code-review@localhost> Tue Aug 05 00:14:23 2014 -0700
tree 4b8fe2865bb29e7ccf48bb8dcea94ffcc2180e4b
parent 2fcabb355ac38eeccff7faefcac7ab3f72595051
parent 339342fb92386bb4d0d71c6bf3147460a9eb1df1

Merge "hal: fix to update voice call devices in apq8084"

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index de8a350..5e9b258 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -628,8 +628,6 @@
 {
     snd_device_t out_snd_device = SND_DEVICE_NONE;
     snd_device_t in_snd_device = SND_DEVICE_NONE;
-    snd_device_t prev_out_snd_device = SND_DEVICE_NONE;
-    snd_device_t prev_in_snd_device = SND_DEVICE_NONE;
     struct audio_usecase *usecase = NULL;
     struct audio_usecase *vc_usecase = NULL;
     struct audio_usecase *voip_usecase = NULL;
@@ -771,12 +769,6 @@
                                                         out_snd_device,
                                                         in_snd_device);
 
-    /* Cache the current usecase devices. This is required to avoid
-     * sending device enable command to the external modem.
-     */
-    prev_in_snd_device = usecase->in_snd_device;
-    prev_out_snd_device = usecase->out_snd_device;
-
     usecase->in_snd_device = in_snd_device;
     usecase->out_snd_device = out_snd_device;
 
@@ -786,14 +778,10 @@
      * Enable device command should be sent to modem only after
      * enabling voice call mixer controls
      */
-    if ((usecase->type == VOICE_CALL) &&
-        (prev_in_snd_device != SND_DEVICE_NONE) &&
-        (prev_out_snd_device != SND_DEVICE_NONE)) {
+    if (usecase->type == VOICE_CALL)
         status = platform_switch_voice_call_usecase_route_post(adev->platform,
                                                                out_snd_device,
                                                                in_snd_device);
-    }
-
     ALOGD("%s: done",__func__);
 
     return status;