tequilaOS/platform_hardware_interfaces
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge "Effect AIDL: skip volume checking" into main

Browse source
This commit is contained in:
Shunkai Yao 2023-11-27 20:03:49 +00:00 committed by Gerrit Code Review
commit a1d092a818
4 changed files with 11 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
audio/aidl/default/acousticEchoCanceler/AcousticEchoCancelerSw.cpp
View file

@ -76,7 +76,7 @@ const Descriptor AcousticEchoCancelerSw::kDescriptor = {
.proxy = std::nullopt},
.flags = {.type = Flags::Type::PRE_PROC,
.insert = Flags::Insert::FIRST,
.volume = Flags::Volume::CTRL},
.volume = Flags::Volume::NONE},
.name = AcousticEchoCancelerSw::kEffectName,
.implementor = "The Android Open Source Project"},
.capability = AcousticEchoCancelerSw::kCapability};

2
audio/aidl/default/noiseSuppression/NoiseSuppressionSw.cpp
View file

@ -67,7 +67,7 @@ const Descriptor NoiseSuppressionSw::kDescriptor = {
.proxy = std::nullopt},
.flags = {.type = Flags::Type::PRE_PROC,
.insert = Flags::Insert::FIRST,
.volume = Flags::Volume::CTRL},
.volume = Flags::Volume::NONE},
.name = NoiseSuppressionSw::kEffectName,
.implementor = "The Android Open Source Project"}};

2
audio/aidl/default/visualizer/VisualizerSw.cpp
View file

@ -73,7 +73,7 @@ const Descriptor VisualizerSw::kDescriptor = {
.proxy = std::nullopt},
.flags = {.type = Flags::Type::INSERT,
.insert = Flags::Insert::FIRST,
.volume = Flags::Volume::CTRL},
.volume = Flags::Volume::NONE},
.name = VisualizerSw::kEffectName,
.implementor = "The Android Open Source Project"},
.capability = VisualizerSw::kCapability};

6
audio/aidl/vts/VtsHalAudioEffectTargetTest.cpp
View file

@ -596,8 +596,14 @@ TEST_P(AudioEffectTest, SetAndGetParameterVolume) {
Parameter::Id id = Parameter::Id::make<Parameter::Id::commonTag>(Parameter::volumeStereo);
Parameter::VolumeStereo volume = {.left = 10.0, .right = 10.0};
if (mDescriptor.common.flags.volume == Flags::Volume::CTRL) {
Parameter get;
EXPECT_IS_OK(mEffect->setParameter(volume));
EXPECT_IS_OK(mEffect->getParameter(id, &get));
} else {
ASSERT_NO_FATAL_FAILURE(
setAndGetParameter(id, Parameter::make<Parameter::volumeStereo>(volume)));
}
ASSERT_NO_FATAL_FAILURE(command(mEffect, CommandId::STOP));
ASSERT_NO_FATAL_FAILURE(expectState(mEffect, State::IDLE));