commit 71a8b971f373993826ad8ddf02e1814a14f2fec4
author Avinash Vaish <avaish@codeaurora.org> Thu Jul 24 15:36:33 2014 +0530
committer Karthik Reddy Katta <a_katta@codeaurora.org> Sat Sep 27 09:53:02 2014 +0530
tree aa401ea6d8de9cf14dc8b6bae78a4d382f1ee828
parent 86ed63f7cf45f3cda143ca304d61fea9f08bd16e

hal: Send voice cal data before routing voice usecases

-The sound intensity is drastically reduced in a DSDA call when
a particular device is re-selected after a device switch.
It is observed that for the active sub, acdb data tables are
registered to ADSP even before the right acdb tables are
sent to the acdb driver. Therefore, the tables which are infact
registered to the ADSP correspond to the previous device.
- Load the acdb files into the kernel memory before routing
voice usecases to prevent any incorrect tables being sent.

CRs-Fixed: 694397
Change-Id: Id2f2a43c887d9b9e74cd98858013d1ef655a93e3

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index 1f53aa7..d02ec19 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -268,6 +268,26 @@
     return 0;
 }
 
+static int enable_audio_route_for_voice_usecases(struct audio_device *adev,
+                                                 struct audio_usecase *uc_info)
+{
+    struct listnode *node;
+    struct audio_usecase *usecase;
+
+    if (uc_info == NULL)
+        return -EINVAL;
+
+    /* Re-route all voice usecases on the shared backend other than the
+       specified usecase to new snd devices */
+    list_for_each(node, &adev->usecase_list) {
+        usecase = node_to_item(node, struct audio_usecase, list);
+        if ((usecase->type == VOICE_CALL || usecase->type == VOIP_CALL) &&
+             (usecase != uc_info))
+            enable_audio_route(adev, usecase);
+    }
+    return 0;
+}
+
 int enable_audio_route(struct audio_device *adev,
                        struct audio_usecase *usecase)
 {
@@ -518,7 +538,8 @@
             /* Update the out_snd_device only before enabling the audio route */
             if (switch_device[usecase->id] ) {
                 usecase->out_snd_device = snd_device;
-                enable_audio_route(adev, usecase);
+                if (usecase->type != VOICE_CALL && usecase->type != VOIP_CALL)
+                    enable_audio_route(adev, usecase);
             }
         }
     }
@@ -586,7 +607,8 @@
             /* Update the in_snd_device only before enabling the audio route */
             if (switch_device[usecase->id] ) {
                 usecase->in_snd_device = snd_device;
-                enable_audio_route(adev, usecase);
+                if (usecase->type != VOICE_CALL && usecase->type != VOIP_CALL)
+                    enable_audio_route(adev, usecase);
             }
         }
     }
@@ -789,10 +811,12 @@
         enable_snd_device(adev, in_snd_device);
     }
 
-    if (usecase->type == VOICE_CALL || usecase->type == VOIP_CALL)
+    if (usecase->type == VOICE_CALL || usecase->type == VOIP_CALL) {
         status = platform_switch_voice_call_device_post(adev->platform,
                                                         out_snd_device,
                                                         in_snd_device);
+        enable_audio_route_for_voice_usecases(adev, usecase);
+    }
 
     usecase->in_snd_device = in_snd_device;
     usecase->out_snd_device = out_snd_device;