commit bf0bb53edbced42bbef011aeae07b3fbcb0fbeb6
author Sham Rathod <sham.rathod@ittiam.com> Tue Dec 27 11:54:35 2022 +0530
committer Sham Rathod <sham.rathod@ittiam.com> Tue Dec 27 16:09:34 2022 +0530
tree 086fe2b78c178f06160af5bcfb6b874baa201f6d
parent 351317028e1cbc07e9a3a746c1acbb74d2769b70

Virtualizer : Add minimum and maximum capabilities for params.

Bug: 263416041
Test: atest VtsHalVirtualizerTargetTest
Change-Id: I8bbae0222ca2a2170f4e68e1cb05736ee1140e02

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 9c7c312..9308ebb 100644
--- a/media/libeffects/lvm/wrapper/Aidl/BundleContext.cpp
+++ b/media/libeffects/lvm/wrapper/Aidl/BundleContext.cpp
@@ -552,8 +552,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 16917f0..f2c6776 100644
--- a/media/libeffects/lvm/wrapper/Aidl/BundleTypes.h
+++ b/media/libeffects/lvm/wrapper/Aidl/BundleTypes.h
@@ -103,7 +103,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";