commit 973fc364610f8e9031274247f077c2024a4f7b0f
author Dhananjay Kumar <dhakumar@codeaurora.org> Mon Jan 09 18:48:15 2017 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Mon Jan 23 01:11:21 2017 -0800
tree 9b18ffa4724c2a56723ad6eb84e8eb712e3ab550
parent d77d63c1aba73803090e2fd2392a795b36f11280

hal: fix mismatch in configuration for NOIRQ record path

- Initialization of 'realtime' mode for an input stream got
  moved down its usage during propagation of compress input
  changes to 2.2 component. Incorrect mode causes legacy low-latency
  configuration being used even for NOIRQ pcm devices and hence
  the error in setting hw and sw parameters.

- Realign initialization of 'realtime' mode to place it properly
  before the usage and hence fix this issue.

Change-Id: Ifdc7b8ff9d129e93a48d34c622e5a76bd83617eb

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index 7d60af3..8c37bae 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -4705,6 +4705,16 @@
     in->capture_handle = handle;
     in->flags = flags;
 
+    in->usecase = USECASE_AUDIO_RECORD;
+    if (config->sample_rate == LOW_LATENCY_CAPTURE_SAMPLE_RATE &&
+            (flags & AUDIO_INPUT_FLAG_FAST) != 0) {
+        is_low_latency = true;
+#if LOW_LATENCY_CAPTURE_USE_CASE
+        in->usecase = USECASE_AUDIO_RECORD_LOW_LATENCY;
+#endif
+        in->realtime = may_use_noirq_mode(adev, in->usecase, in->flags);
+    }
+
     in->format = config->format;
     if (in->realtime) {
         in->config = pcm_config_audio_capture_rt;
@@ -4759,16 +4769,6 @@
         }
     }
 
-    in->usecase = USECASE_AUDIO_RECORD;
-    if (config->sample_rate == LOW_LATENCY_CAPTURE_SAMPLE_RATE &&
-            (flags & AUDIO_INPUT_FLAG_FAST) != 0) {
-        is_low_latency = true;
-#if LOW_LATENCY_CAPTURE_USE_CASE
-        in->usecase = USECASE_AUDIO_RECORD_LOW_LATENCY;
-#endif
-        in->realtime = may_use_noirq_mode(adev, in->usecase, in->flags);
-    }
-
     /* Update config params with the requested sample rate and channels */
     if ((in->device == AUDIO_DEVICE_IN_TELEPHONY_RX) &&
           (adev->mode != AUDIO_MODE_IN_CALL)) {