commit 53b0aa6efef187486cdb3f2b6eab384f4f2bf844
author Treehugger Robot <treehugger-gerrit@google.com> Wed Dec 28 21:11:50 2022 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Wed Dec 28 21:11:50 2022 +0000
tree 74a1e654227f5a3b7f976c234a85e2c1b2ee21ce
parent 4379e29d9646ae6778bfc6a2fbbc13c4d78a3847
parent bf0bb53edbced42bbef011aeae07b3fbcb0fbeb6

Merge "Virtualizer : Add minimum and maximum capabilities for params."

media/libeffects/lvm/wrapper/Aidl/BundleContext.cpp

diff --git a/media/libeffects/lvm/wrapper/Aidl/BundleContext.cpp b/media/libeffects/lvm/wrapper/Aidl/BundleContext.cpp
index 2b69289..3aee721 100644
--- a/media/libeffects/lvm/wrapper/Aidl/BundleContext.cpp
+++ b/media/libeffects/lvm/wrapper/Aidl/BundleContext.cpp
@@ -551,8 +551,7 @@
 }
 
 RetCode BundleContext::setVirtualizerStrength(int strength) {
-    if (strength < Virtualizer::MIN_PER_MILLE_STRENGTH ||
-        strength > Virtualizer::MAX_PER_MILLE_STRENGTH) {
+    if (strength < 0 || strength > lvm::kVirtualizerCap.maxStrengthPm) {
         return RetCode::ERROR_ILLEGAL_PARAMETER;
     }
 

media/libeffects/lvm/wrapper/Aidl/BundleTypes.h

diff --git a/media/libeffects/lvm/wrapper/Aidl/BundleTypes.h b/media/libeffects/lvm/wrapper/Aidl/BundleTypes.h
index 893943e..1996240 100644
--- a/media/libeffects/lvm/wrapper/Aidl/BundleTypes.h
+++ b/media/libeffects/lvm/wrapper/Aidl/BundleTypes.h
@@ -104,7 +104,8 @@
                    .implementor = "NXP Software Ltd."},
         .capability = Capability::make<Capability::bassBoost>(kBassBoostCap)};
 
-static const Virtualizer::Capability kVirtualizerCap = {.strengthSupported = mStrengthSupported};
+static const Virtualizer::Capability kVirtualizerCap = {.maxStrengthPm = 1000,
+                                                        .strengthSupported = mStrengthSupported};
 
 static const std::string kVirtualizerEffectName = "Virtualizer";