commit c89817dfdbab811cdaa3dfbf9d1cd8be7d637980
author Ram Mohan <ram.mohan@ittiam.com> Tue Mar 14 21:39:26 2023 +0530
committer Ram Mohan <ram.mohan@ittiam.com> Tue Mar 28 20:26:07 2023 +0530
tree 232a6db2e7b937596d041eea93e6a12883367ffc
parent 36f42119d3e3e9e6a99d0af9f9066b414542c951

DynamicsProcessing: Add range limits for dynamics processing effect params

Bug: 272195502
Test: atest VtsHalDynamicsProcessingTargetTest

Change-Id: Icae8b01dfce916ad05c39e254922131f3856c305

media/libeffects/dynamicsproc/aidl/DynamicsProcessing.h

diff --git a/media/libeffects/dynamicsproc/aidl/DynamicsProcessing.h b/media/libeffects/dynamicsproc/aidl/DynamicsProcessing.h
index d094c69..1e1e72e 100644
--- a/media/libeffects/dynamicsproc/aidl/DynamicsProcessing.h
+++ b/media/libeffects/dynamicsproc/aidl/DynamicsProcessing.h
@@ -18,8 +18,9 @@
 
 #include <aidl/android/hardware/audio/effect/BnEffect.h>
 
-#include "effect-impl/EffectImpl.h"
 #include "DynamicsProcessingContext.h"
+#include "EffectRangeSpecific.h"
+#include "effect-impl/EffectImpl.h"
 
 namespace aidl::android::hardware::audio::effect {
 
@@ -51,14 +52,10 @@
     std::string getEffectName() override { return kEffectName; }
 
   private:
-    static const DynamicsProcessing::EqBandConfig kEqBandConfigMin;
-    static const DynamicsProcessing::EqBandConfig kEqBandConfigMax;
-    static const Range::DynamicsProcessingRange kPreEqBandRange;
-    static const Range::DynamicsProcessingRange kPostEqBandRange;
-    static const Range kRange;
     std::shared_ptr<DynamicsProcessingContext> mContext;
     ndk::ScopedAStatus getParameterDynamicsProcessing(const DynamicsProcessing::Tag& tag,
                                                       Parameter::Specific* specific);
+    bool isParamInRange(const Parameter::Specific& specific);
 };
 
 }  // namespace aidl::android::hardware::audio::effect