commit 9cc498b4a3e6d6b99a83755403c143f0ba5bfa96
author Dhananjay Kumar <dhakumar@codeaurora.org> Tue Dec 20 21:04:13 2016 +0530
committer Sharad Sangle <assangle@codeaurora.org> Fri Dec 23 12:02:21 2016 +0530
tree 0323229a67ff5b25800e4c62e81b6a96f86be025
parent 591c2baf486add78c4ee40da5f55195505e4879e

audio: miscellaneous fixes against iot changes

-Fix compilation warnings against tx app_type changes.
-Remove assignment of compress usecase outside compress
 input extension.
-Define input flag for compress record usage.

Change-Id: I36630c7f6524d494b76e0a3bd074185a01e19883

hal/audio_extn/audio_defs.h

diff --git a/hal/audio_extn/audio_defs.h b/hal/audio_extn/audio_defs.h
index ee4f7d4..1baa3a7 100644
--- a/hal/audio_extn/audio_defs.h
+++ b/hal/audio_extn/audio_defs.h
@@ -106,10 +106,11 @@
 /* Set or Query stream profile type */
 #define AUDIO_PARAMETER_STREAM_PROFILE "audio_stream_profile"
 
-/* audio inout flag for timestamp mode.
+/* audio input flags for compress and timestamp mode.
  * check other input flags defined in audio.h for conflicts
  */
 #define AUDIO_INPUT_FLAG_TIMESTAMP 0x80000000
+#define AUDIO_INPUT_FLAG_COMPRESS  0x40000000
 
 /* MAX SECTORS for sourcetracking feature */
 #define MAX_SECTORS 8

hal/audio_extn/compress_in.c

diff --git a/hal/audio_extn/compress_in.c b/hal/audio_extn/compress_in.c
index 403ee8b..1a6c789 100644
--- a/hal/audio_extn/compress_in.c
+++ b/hal/audio_extn/compress_in.c
@@ -84,8 +84,7 @@
 
 bool audio_extn_cin_applicable_stream(struct stream_in *in)
 {
-    if ((in->flags & AUDIO_INPUT_FLAG_TIMESTAMP) ||
-        in->usecase == USECASE_AUDIO_RECORD_COMPRESS2)
+    if (in->flags & (AUDIO_INPUT_FLAG_COMPRESS | AUDIO_INPUT_FLAG_TIMESTAMP))
         return true;
 
     return false;

hal/audio_extn/utils.c

diff --git a/hal/audio_extn/utils.c b/hal/audio_extn/utils.c
index a0eb5cd..367e9a8 100644
--- a/hal/audio_extn/utils.c
+++ b/hal/audio_extn/utils.c
@@ -380,9 +380,9 @@
             if (app_type_cfg[i+1] == 0)
                 break;
             else if (app_type_cfg[i+1] == (size_t)s_info->app_type_cfg.app_type) {
-                if (app_type_cfg[i+2] < s_info->app_type_cfg.sample_rate)
+                if (app_type_cfg[i+2] < (size_t)s_info->app_type_cfg.sample_rate)
                     app_type_cfg[i+2] = s_info->app_type_cfg.sample_rate;
-                if (app_type_cfg[i+3] < s_info->app_type_cfg.bit_width)
+                if (app_type_cfg[i+3] < (size_t)s_info->app_type_cfg.bit_width)
                     app_type_cfg[i+3] = s_info->app_type_cfg.bit_width;
                 update = false;
                 break;
@@ -402,9 +402,9 @@
             if (app_type_cfg[i+1] == 0)
                 break;
             else if (app_type_cfg[i+1] == (size_t)s_info->app_type_cfg.app_type) {
-                if (app_type_cfg[i+2] < s_info->app_type_cfg.sample_rate)
+                if (app_type_cfg[i+2] < (size_t)s_info->app_type_cfg.sample_rate)
                     app_type_cfg[i+2] = s_info->app_type_cfg.sample_rate;
-                if (app_type_cfg[i+3] < s_info->app_type_cfg.bit_width)
+                if (app_type_cfg[i+3] < (size_t)s_info->app_type_cfg.bit_width)
                     app_type_cfg[i+3] = s_info->app_type_cfg.bit_width;
                 update = false;
                 break;