commit 4f069973bc13eec2f2bf8d4808cdd40cfffc6b03
author sasevs <quic_sasevs@quicinc.com> Sat Jun 03 14:23:25 2023 +0530
committer sasevs <quic_sasevs@quicinc.com> Sun Jun 04 08:54:50 2023 +0530
tree c1c02c5dc41cd2e5119601dd8b58691cb9b62014
parent 2e7dda219c2dc405a3498258cf71504550fd06a3

hal: fixing wrong usecase picking for aiff format

Fix for AIFF and PCM format to pick compress-offload-playback2.

Change-Id: I99a37cfbf51fd19b0b018f427153fa9f849ace8c
Suggested-by: Bhargavi Belagod Manjunatha <bbelagod@qti.qualcomm.com>

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index 2822751..d6a5684 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -8462,14 +8462,6 @@
         out->config.period_size = HDMI_MULTI_PERIOD_BYTES / (out->config.channels *
                                                          audio_bytes_per_sample(config->format));
         out->config.format = pcm_format_from_audio_format(out->format);
-    } else if ((!(out->flags & AUDIO_OUTPUT_FLAG_COMPRESS_OFFLOAD)) &&
-                compare_device_type(&out->device_list, AUDIO_DEVICE_OUT_BUS)) {
-            ret = audio_extn_auto_hal_open_output_stream(out);
-            if (ret) {
-                ALOGE("%s: Failed to open output stream for bus device", __func__);
-                ret = -EINVAL;
-                goto error_open;
-            }
      }else if ((out->flags & AUDIO_OUTPUT_FLAG_COMPRESS_OFFLOAD) ||
                (out->flags == AUDIO_OUTPUT_FLAG_DIRECT)) {
         pthread_mutex_lock(&adev->lock);
@@ -8864,6 +8856,13 @@
                 audio_channel_count_from_out_mask(out->channel_mask);
         out->config.format = pcm_format_from_audio_format(out->format);
         adev->voice_tx_output = out;
+    } else if (compare_device_type(&out->device_list, AUDIO_DEVICE_OUT_BUS)) {
+            ret = audio_extn_auto_hal_open_output_stream(out);
+            if (ret) {
+                ALOGE("%s: Failed to open output stream for bus device", __func__);
+                ret = -EINVAL;
+                goto error_open;
+            }
     } else {
         unsigned int channels = 0;
         /*Update config params to default if not set by the caller*/