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

Commit c2a3a824 authored by Ricardo Garcia's avatar Ricardo Garcia Committed by Eric Laurent
Browse files

Fix for untimely suspension of Dynamics Processing 

Added exception to avoid suspending Dynamics Processing Effect.

Test: Manual testing with popular apps
Bug: 134852575
Change-Id: I1ffee570984897304169b30b7af417f61f762ffa
parent d2d66abd

services/audioflinger/Effects.cpp

+3 −1
Original line number Diff line number Diff line
@@ -24,6 +24,7 @@ 
#include "Configuration.h"

#include <utils/Log.h>

#include <system/audio_effects/effect_aec.h>

#include <system/audio_effects/effect_dynamicsprocessing.h>

#include <system/audio_effects/effect_ns.h>

#include <system/audio_effects/effect_visualizer.h>

#include <audio_utils/channels.h>
@@ -2569,7 +2570,8 @@ bool AudioFlinger::EffectChain::isEffectEligibleForSuspend(const effect_descript 
    if ((mSessionId == AUDIO_SESSION_OUTPUT_MIX) &&

        (((desc.flags & EFFECT_FLAG_TYPE_MASK) == EFFECT_FLAG_TYPE_AUXILIARY) ||

         (memcmp(&desc.type, SL_IID_VISUALIZATION, sizeof(effect_uuid_t)) == 0) ||

         (memcmp(&desc.type, SL_IID_VOLUME, sizeof(effect_uuid_t)) == 0))) {

         (memcmp(&desc.type, SL_IID_VOLUME, sizeof(effect_uuid_t)) == 0) ||

         (memcmp(&desc.type, SL_IID_DYNAMICSPROCESSING, sizeof(effect_uuid_t)) == 0))) {

        return false;

    }

    return true;