commit 9381e58a756631f0dbf83195cc52c01c9b038533
author Linux Build Service Account <lnxbuild@localhost> Sat Nov 09 21:47:36 2013 -0800
committer Gerrit - the friendly Code Review server <code-review@localhost> Sat Nov 09 21:47:36 2013 -0800
tree 746b0357d6f8986c3ccb3827b238b6ead53b1edc
parent f41dcba99f9169a1bad915240052417994f25dac
parent dc4f757fac69d81e1692d068ad17a307a0dcacd5

Merge "hal : Fix for stability offload issues."

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index a90caa4..64db646 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -371,10 +371,15 @@
             usecase = node_to_item(node, struct audio_usecase, list);
             if (switch_device[usecase->id]) {
                 disable_snd_device(adev, usecase->out_snd_device, false);
-                enable_snd_device(adev, snd_device, false);
             }
         }
 
+        list_for_each(node, &adev->usecase_list) {
+            usecase = node_to_item(node, struct audio_usecase, list);
+            if (switch_device[usecase->id]) {
+                enable_snd_device(adev, snd_device, false);
+            }
+        }
         /* Make sure new snd device is enabled before re-routing the streams */
         audio_route_update_mixer(adev->audio_route);
 
@@ -1211,6 +1216,7 @@
           out->usecase, use_case_table[out->usecase]);
 
     pthread_mutex_lock(&out->lock);
+    pthread_mutex_lock(&adev->lock);
     if (!out->standby) {
         out->standby = true;
         if (out->usecase != USECASE_AUDIO_PLAYBACK_OFFLOAD) {
@@ -1227,10 +1233,9 @@
                 out->compr = NULL;
             }
         }
-        pthread_mutex_lock(&adev->lock);
         stop_output_stream(out);
-        pthread_mutex_unlock(&adev->lock);
     }
+    pthread_mutex_unlock(&adev->lock);
     pthread_mutex_unlock(&out->lock);
     ALOGV("%s: exit", __func__);
     return 0;