commit 2f38a25e76c56daa6b3051c076c7c2fe66626cd7
author Shunkai Yao <yaoshunkai@google.com> Wed Feb 22 20:13:02 2023 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Wed Feb 22 20:13:02 2023 +0000
tree 949d525b7ad1a3ce1a4223e63400623331bd0d71
parent 646e2fcc172f5466e3d2c634d71849967dbd1914
parent 8d2a1b05da530c9e9b79a88827d1370f5cf17ef8

Merge "[AIDL CTS] pass BassBoost implementation cts" am: eee35685d5 am: f2017ff63e am: 8d2a1b05da

Original change: https://android-review.googlesource.com/c/platform/frameworks/av/+/2448467

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

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);