commit 06dc621b86475f1c21210ffdfa2e2a0ae2cfa3b1
author Mikhail Naganov <mnaganov@google.com> Sat Oct 14 00:28:39 2023 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Sat Oct 14 00:28:39 2023 +0000
tree 2e676efef1f94449ea0b970b2a9c9cc63ce33812
parent eeb4d30cddf5f9e4b2b2e4c575fdc1bfaac9d1ee
parent 3727401bcba0b8651f443bc05d328779816222e5

Revert "audio: Use hardcoded r_submix configuration with XML" am: 567d0de485 am: a37a4feed2 am: 3727401bcb

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2786754

Change-Id: I080214e99acae9276c5662e4ccbe93a7a17f9848
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

audio/aidl/default/AudioPolicyConfigXmlConverter.cpp

diff --git a/audio/aidl/default/AudioPolicyConfigXmlConverter.cpp b/audio/aidl/default/AudioPolicyConfigXmlConverter.cpp
index 2f1282a..5d06cb6 100644
--- a/audio/aidl/default/AudioPolicyConfigXmlConverter.cpp
+++ b/audio/aidl/default/AudioPolicyConfigXmlConverter.cpp
@@ -184,15 +184,9 @@
             // 'primary' in the XML schema used by HIDL is equivalent to 'default' module.
             const std::string name =
                     xsdcModule.getName() != "primary" ? xsdcModule.getName() : "default";
-            if (name != "r_submix") {
-                mModuleConfigurations->emplace_back(
-                        name, VALUE_OR_FATAL(convertModuleConfigToAidl(xsdcModule)));
-            } else {
-                // See the note on the 'getRSubmixConfiguration' function.
-                mModuleConfigurations->emplace_back(name, nullptr);
-            }
+            mModuleConfigurations->emplace_back(
+                    name, VALUE_OR_FATAL(convertModuleConfigToAidl(xsdcModule)));
         }
     }
 }
-
 }  // namespace aidl::android::hardware::audio::core::internal