commit b84e87959d62f4f22c9e69e9d0ded609b5e2e05f
author Shunkai Yao <yaoshunkai@google.com> Tue Feb 21 18:04:26 2023 +0000
committer Shunkai Yao <yaoshunkai@google.com> Wed Feb 22 06:43:45 2023 +0000
tree c5241702f24b0b8564d68db9fac7f502dd96d84c
parent aca1a6f75c9d275aee97beee7eeb758273d89003

[AIDL CTS] pass BassBoost implementation cts

Bug: 270158223
Test: atest android.media.audio.cts.BassBoostTest
Change-Id: I7a58667c7947d8904692d3d2a42e71ea662f3931

media/libaudiohal/impl/effectsAidlConversion/AidlConversionBassBoost.cpp

diff --git a/media/libaudiohal/impl/effectsAidlConversion/AidlConversionBassBoost.cpp b/media/libaudiohal/impl/effectsAidlConversion/AidlConversionBassBoost.cpp
index 91c3dea..9ec593f 100644
--- a/media/libaudiohal/impl/effectsAidlConversion/AidlConversionBassBoost.cpp
+++ b/media/libaudiohal/impl/effectsAidlConversion/AidlConversionBassBoost.cpp
@@ -82,7 +82,7 @@
     Parameter aidlParam;
     switch (type) {
         case BASSBOOST_PARAM_STRENGTH: {
-            uint32_t value;
+            uint16_t value;
             Parameter::Id id =
                     MAKE_SPECIFIC_PARAMETER_ID(BassBoost, bassBoostTag, BassBoost::strengthPm);
             RETURN_STATUS_IF_ERROR(statusTFromBinderStatus(mEffect->getParameter(id, &aidlParam)));
@@ -92,7 +92,7 @@
         }
         case BASSBOOST_PARAM_STRENGTH_SUPPORTED: {
             // an invalid range indicates not setting support for this parameter
-            uint16_t value =
+            uint32_t value =
                     ::aidl::android::hardware::audio::effect::isRangeValid<Range::Tag::bassBoost>(
                             BassBoost::strengthPm, mDesc.capability);
             return param.writeToValue(&value);