From 630498aad0f6c0428ffd6b6ff72a4927a03d8f89 Mon Sep 17 00:00:00 2001
From: Jasmine Cha <chajasmine@google.com>
Date: Tue, 30 Jan 2024 17:03:53 +0800
Subject: [PATCH] audio: fix record tuning

- remove VOICE_RECOGNITION source for spatial capture
- move Mic Spatial Module Enable to audio hal control

Bug: 322926141
Test: manual test - camcorder/voic

Change-Id: I0dc2e26b34f37dbab988bf7216f1900bc402aba4
Signed-off-by: Jasmine Cha <chajasmine@google.com>
---
 audio/comet/aidl_config/audio_platform_configuration.xml | 2 --
 audio/comet/aidl_config/mixer_paths_aidl.xml             | 8 +++++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/audio/comet/aidl_config/audio_platform_configuration.xml b/audio/comet/aidl_config/audio_platform_configuration.xml
index 794f605..fad990e 100644
--- a/audio/comet/aidl_config/audio_platform_configuration.xml
+++ b/audio/comet/aidl_config/audio_platform_configuration.xml
@@ -104,12 +104,10 @@
     <enablement useCase="primary-capture" deviceInterface="microphones">
       <source>MIC</source>
       <source>CAMCORDER</source>
-      <source>VOICE_RECOGNITION</source>
     </enablement>
     <enablement useCase="primary-capture" deviceInterface="back-microphones">
       <source>MIC</source>
       <source>CAMCORDER</source>
-      <source>VOICE_RECOGNITION</source>
     </enablement>
   </spatialCapture>
 
diff --git a/audio/comet/aidl_config/mixer_paths_aidl.xml b/audio/comet/aidl_config/mixer_paths_aidl.xml
index 4cfc807..fa31054 100644
--- a/audio/comet/aidl_config/mixer_paths_aidl.xml
+++ b/audio/comet/aidl_config/mixer_paths_aidl.xml
@@ -260,10 +260,14 @@
 
   <ctl name="AoC Modem Downlink ASRC Mode" value="ASP_ON" />
   <ctl name="Voice Call Mic Source" value="Builtin_MIC" />
-  <ctl name="Mic Spatial Module Enable" value="0" />
   <ctl name="Audio Capture Eraser Enable" value="0" />
   <ctl name="Hotword Tap Enable" value="0" />
 
+  <!-- In AIDL Hal, this controlled by audio hal directly to avoid aoc"     -->
+  <!-- crashing since speicifc order is needed between dsp mode and"        -->
+  <!-- spatial module."                                                     -->
+  <ctl name="Mic Spatial Module Enable" value="0" />
+
   <!-- audio PDM mic default state -->
   <ctl name="BUILDIN MIC ID CAPTURE LIST" id="0" value="-1" />
   <ctl name="BUILDIN MIC ID CAPTURE LIST" id="1" value="-1" />
@@ -1313,7 +1317,6 @@
 
   <path name="microphones">
     <path name="builtin-mic-id-capture-list" />
-    <ctl name="Mic Spatial Module Enable" value="1" />
     <ctl name="MIC DC Blocker" value="1" />
     <ctl name="MIC Record Soft Gain (dB)" value="22" />
   </path>
@@ -1347,7 +1350,6 @@
 
   <path name="microphones (CAMCORDER)">
     <path name="builtin-mic-id-capture-list" />
-    <ctl name="Mic Spatial Module Enable" value="1" />
     <ctl name="MIC DC Blocker" value="1" />
     <ctl name="MIC Record Soft Gain (dB)" value="0" />
   </path>