commit 753b1f31850240dfbe4b9836c94f3c2e7b08c921
author Satya Krishna Pindiproli <satyak@codeaurora.org> Wed May 31 17:37:53 2017 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Thu Jun 01 00:59:31 2017 -0700
tree c52c480656c0907fb28cf0de13f9dc612cb13054
parent ffe6cef04569624aaf263c27b1267fc47eca29cb

policy_hal: allow direct output only for music streams

Ringtones/Alarms take direct PCM path which is not expected.

Allow only music streams to take direct path by forcefully
setting flags to none for other stream types.

CRs-Fixed: 2049687
Change-Id: I1994f3958b9b10991e45f1a8a28fb26764d0af13

policy_hal/AudioPolicyManager.cpp

diff --git a/policy_hal/AudioPolicyManager.cpp b/policy_hal/AudioPolicyManager.cpp
index 8009784..72bd644 100644
--- a/policy_hal/AudioPolicyManager.cpp
+++ b/policy_hal/AudioPolicyManager.cpp
@@ -1714,9 +1714,10 @@
         ((offloadInfo->usage == AUDIO_USAGE_MEDIA) || (offloadInfo->usage == AUDIO_USAGE_GAME))) {
         audio_output_flags_t old_flags = flags;
         flags = (audio_output_flags_t)(AUDIO_OUTPUT_FLAG_DIRECT);
-        ALOGD("AudioCustomHAL --> Force Direct Flag .. old flags(0x%x)", old_flags);
-    } else if (flags == AUDIO_OUTPUT_FLAG_DIRECT && offload_disabled) {
-        ALOGD("AudioCustomHAL --> offloading is disabled: Force Remove Direct Flag");
+        ALOGD("Force Direct Flag .. old flags(0x%x)", old_flags);
+    } else if (flags == AUDIO_OUTPUT_FLAG_DIRECT &&
+                (offload_disabled || stream != AUDIO_STREAM_MUSIC)) {
+        ALOGD("Offloading is disabled or Stream is not music --> Force Remove Direct Flag");
         flags = (audio_output_flags_t)(AUDIO_OUTPUT_FLAG_NONE);
     }