From 0351ad9c0b2dffb38b5b36ff860122cd1628f74f Mon Sep 17 00:00:00 2001 From: Preetam Singh Ranawat Date: Mon, 27 May 2019 14:31:47 +0530 Subject: policy_hal: fix CTS concurrent audio playback -fall back to non-direct output is not working for 3rd playback request when both compress offload and direct pcm are in use as only one flag is set to true due to break statement. -Remove break statement to set both flags to true when both compress offload and direct pcm are in use. Change-Id: I60258ab8c6e3aa611268c087f2d0e02aabe51b41 --- policy_hal/AudioPolicyManager.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/policy_hal/AudioPolicyManager.cpp b/policy_hal/AudioPolicyManager.cpp index 99f1a24..26064c9 100644 --- a/policy_hal/AudioPolicyManager.cpp +++ b/policy_hal/AudioPolicyManager.cpp @@ -1663,12 +1663,10 @@ audio_io_handle_t AudioPolicyManagerCustom::getOutputForDevices( if (desc->mFlags == AUDIO_OUTPUT_FLAG_DIRECT) { direct_pcm_already_in_use = true; ALOGD("Direct PCM already in use"); - break; } if (desc->mFlags & AUDIO_OUTPUT_FLAG_COMPRESS_OFFLOAD) { compress_offload_already_in_use = true; ALOGD("Compress Offload already in use"); - break; } } // prevent direct pcm for non-music stream blindly if direct pcm already in use -- cgit v1.2.3