Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit de898f05 authored by Shunkai Yao's avatar Shunkai Yao Committed by Automerger Merge Worker
Browse files

Merge "Ignore EQ level range if no capability defined by effect... 

Merge "Ignore EQ level range if no capability defined by effect implementation" am: 87467d3d am: a65c3af6 am: 15e8f031

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



Change-Id: I7b234a31688e4b675f7074a60901169274028dd0
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 4d188b4d 15e8f031

media/libaudiohal/impl/effectsAidlConversion/AidlConversionEq.cpp

+3 −0
Original line number Diff line number Diff line
@@ -161,6 +161,9 @@ status_t AidlConversionEq::getParameter(EffectParamWriter& param) { 
            return param.writeToValue(&bands);

        }

        case EQ_PARAM_LEVEL_RANGE: {

            if (mDesc.capability.range.getTag() != Range::equalizer) {

                return OK;

            }

            const auto& ranges = mDesc.capability.range.get<Range::equalizer>();

            for (const auto& r : ranges) {

                if (r.min.getTag() == Equalizer::bandLevels &&

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

+8 −6
Original line number Diff line number Diff line
@@ -15,6 +15,7 @@ 
 */



#include <cstddef>

#include <cstdio>



#define LOG_TAG "BundleContext"

#include <android-base/logging.h>
@@ -438,8 +439,8 @@ RetCode BundleContext::setEqualizerPreset(const std::size_t presetIdx) { 
    std::vector<Equalizer::BandLevel> bandLevels;

    bandLevels.reserve(lvm::MAX_NUM_BANDS);

    for (std::size_t i = 0; i < lvm::MAX_NUM_BANDS; i++) {

        bandLevels.emplace_back(

                Equalizer::BandLevel{static_cast<int32_t>(i), lvm::kSoftPresets[presetIdx][i]});

        bandLevels.emplace_back(Equalizer::BandLevel{static_cast<int32_t>(i),

                                                     lvm::kSoftPresets[presetIdx][i] * 100});

    }



    RetCode ret = updateControlParameter(bandLevels);
@@ -469,7 +470,8 @@ std::vector<Equalizer::BandLevel> BundleContext::getEqualizerBandLevels() const 
    std::vector<Equalizer::BandLevel> bandLevels;

    bandLevels.reserve(lvm::MAX_NUM_BANDS);

    for (std::size_t i = 0; i < lvm::MAX_NUM_BANDS; i++) {

        bandLevels.emplace_back(Equalizer::BandLevel{static_cast<int32_t>(i), mBandGaindB[i]});

        bandLevels.emplace_back(

                Equalizer::BandLevel{static_cast<int32_t>(i), mBandGaindB[i] * 100});

    }

    return bandLevels;

}
@@ -503,9 +505,9 @@ RetCode BundleContext::updateControlParameter(const std::vector<Equalizer::BandL 
    RETURN_VALUE_IF(!isBandLevelIndexInRange(bandLevels), RetCode::ERROR_ILLEGAL_PARAMETER,

                    "indexOutOfRange");



    std::array<int, lvm::MAX_NUM_BANDS> tempLevel;

    std::array<int, lvm::MAX_NUM_BANDS> tempLevel(mBandGaindB);

    for (const auto& it : bandLevels) {

        tempLevel[it.index] = it.levelMb;

        tempLevel[it.index] = it.levelMb > 0 ? (it.levelMb + 50) / 100 : (it.levelMb - 50) / 100;

    }



    LVM_ControlParams_t params;
@@ -524,7 +526,7 @@ RetCode BundleContext::updateControlParameter(const std::vector<Equalizer::BandL 
                        RetCode::ERROR_EFFECT_LIB_ERROR, " setControlParamFailed");

    }

    mBandGaindB = tempLevel;

    LOG(INFO) << __func__ << " update bandGain to " << ::android::internal::ToString(mBandGaindB);

    LOG(DEBUG) << __func__ << " update bandGain to " << ::android::internal::ToString(mBandGaindB);



    return RetCode::SUCCESS;

}

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

+2 −2
Original line number Diff line number Diff line
@@ -73,9 +73,9 @@ const std::vector<Range::EqualizerRange> kEqRanges = { 
        MAKE_RANGE(Equalizer, preset, 0, MAX_NUM_PRESETS - 1),

        MAKE_RANGE(Equalizer, bandLevels,

                   std::vector<Equalizer::BandLevel>{

                           Equalizer::BandLevel({.index = 0, .levelMb = -15})},

                           Equalizer::BandLevel({.index = 0, .levelMb = -1500})},

                   std::vector<Equalizer::BandLevel>{

                           Equalizer::BandLevel({.index = MAX_NUM_BANDS - 1, .levelMb = 15})}),

                           Equalizer::BandLevel({.index = MAX_NUM_BANDS - 1, .levelMb = 1500})}),

        /* capability definition */

        MAKE_RANGE(Equalizer, bandFrequencies, kEqBandFrequency, kEqBandFrequency),

        MAKE_RANGE(Equalizer, presets, kEqPresets, kEqPresets),