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

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

Merge "Update DeviceEffectManager with AudioHalVersionInfo" am: 334d96ee 

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



Change-Id: I530ad22f6bcd68a6d82988fa5fa36f5340b81b1d
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents b4bfaa46 334d96ee

services/audioflinger/DeviceEffectManager.cpp

+6 −8
Original line number Diff line number Diff line
@@ -30,7 +30,7 @@ 


namespace android {



using media::AudioHalVersion;

using detail::AudioHalVersionInfo;

using media::IEffectClient;



void AudioFlinger::DeviceEffectManager::createAudioPatch(audio_patch_handle_t handle,
@@ -131,14 +131,12 @@ status_t AudioFlinger::DeviceEffectManager::checkEffectCompatibility( 
        return BAD_VALUE;

    }



    static AudioHalVersion sMinDeviceEffectHalVersion;

    sMinDeviceEffectHalVersion.type = AudioHalVersion::Type::HIDL;

    sMinDeviceEffectHalVersion.major = 6;

    sMinDeviceEffectHalVersion.minor = 0;

    AudioHalVersion halVersion = effectsFactory->getHalVersion();

    static AudioHalVersionInfo sMinDeviceEffectHalVersion =

            AudioHalVersionInfo(AudioHalVersionInfo::Type::HIDL, 6, 0);

    AudioHalVersionInfo halVersion = effectsFactory->getHalVersion();



    // We can trust AIDL generated AudioHalVersion comparison operator (based on std::tie) as long

    // as the type, major and minor sequence doesn't change in the definition.

    // We can trust AIDL generated AudioHalVersionInfo comparison operator (based on std::tie) as

    // long as the type, major and minor sequence doesn't change in the definition.

    if (((desc->flags & EFFECT_FLAG_TYPE_MASK) != EFFECT_FLAG_TYPE_PRE_PROC

            && (desc->flags & EFFECT_FLAG_TYPE_MASK) != EFFECT_FLAG_TYPE_POST_PROC)

            || halVersion < sMinDeviceEffectHalVersion) {