commit 02809689b0753b89988186a7facbe943e00844b9
author Ramu Gottipati <ramug@codeaurora.org> Wed Dec 19 16:46:12 2018 +0530
committer Ramu Gottipati <ramug@codeaurora.org> Thu Dec 20 18:31:30 2018 +0530
tree e75360e1a1805e685b3b58df2cba3c3b4cc6f0b5
parent 08d82e72d349a55c869c121639819097c35b79a0

hal: Fix compilation errors

- To fix compilation errors, remove unused
  variable in codebase
- Fix KW issues by adding mutex unlock

Change-Id: I392f4abf53e07524ca28c4559271b153f4190b34

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index 49145ec..75b74f2 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -3482,7 +3482,6 @@
                                     int channel_count,
                                     bool is_low_latency)
 {
-    int i = 0;
     size_t size = 0;
     uint32_t bytes_per_period_sample = 0;
 
@@ -4703,6 +4702,7 @@
                 uint32_t bitwidth_dst = format_to_bitwidth_table[dst_format];
                 if ((bitwidth_src == 0) || (bitwidth_dst == 0)) {
                     ALOGE("%s: Error bitwidth == 0", __func__);
+                    pthread_mutex_unlock(&out->lock);
                     ATRACE_END();
                     return -EINVAL;
                 }