commit 4847e9de6ddef9dd41b203db5e9b314e0709c499
author Ashish Jain <ashishj@codeaurora.org> Thu Aug 17 19:16:57 2017 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Tue Oct 10 22:26:11 2017 -0700
tree 913ad6ffced8f094738e9a562c058ecb22c43707
parent cc6a6e60b443a82ab97c79b160a682680df0e18f

hal: Fix QAF compilation issues with clang

-Fix bunch of compiler errors/warnings for qaf.c with clang.
-Handle DIRECT_PCM flag in HAL to fix playback using QAF.

Change-Id: I38a787ac9451ac16cbb4938b5cd3406c94ecff00

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index e5dc063..9656c83 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -1483,7 +1483,9 @@
     ssize_t count = audio_extn_usb_get_sup_sample_rates(is_playback,
                                                         supported_sample_rates,
                                                         max_rates);
-    for (ssize_t i=0; i<count; i++) {
+    ssize_t i = 0;
+
+    for (i=0; i<count; i++) {
         ALOGV("%s %s %d", __func__, is_playback ? "P" : "C",
               supported_sample_rates[i]);
     }
@@ -5467,8 +5469,8 @@
         out->stream.pause = out_pause;
         out->stream.resume = out_resume;
         out->stream.flush = out_flush;
+        out->stream.set_callback = out_set_callback;
         if (out->flags & AUDIO_OUTPUT_FLAG_COMPRESS_OFFLOAD) {
-            out->stream.set_callback = out_set_callback;
             out->stream.drain = out_drain;
             out->usecase = get_offload_usecase(adev, true /* is_compress */);
             ALOGV("Compress Offload usecase .. usecase selected %d", out->usecase);