diff options
author | Mikhail Naganov <mnaganov@google.com> | 2020-12-10 18:47:51 -0800 |
---|---|---|
committer | Mikhail Naganov <mnaganov@google.com> | 2020-12-16 01:14:02 +0000 |
commit | f363ed4f963960c105f47b22be480795d39afac9 (patch) | |
tree | e845c11fa3cb2cac9e5577df40a301c9b314a2cc /audio/effect/all-versions/default/EqualizerEffect.cpp | |
parent | e1a9c8f8b851fde6f8a1ded2c39633c2e2d90cf0 (diff) |
audio: Update default effect HAL wrapper to support V7
During this conversion, the functionality of the V7 wrapper
hasn't been tested yet. This will be done in a separate CL
that will also include required updates to the VTS tests.
Since the changes were made to the code shared with pre-V7
versions, verified that V6 HAL didn't regress.
Bug: 142480271
Test: atest VtsHalAudioEffectV6_0TargetTest
Test: m VtsHalAudioEffectV7_0TargetTest
Test: m android.hardware.audio@7.0-service.example
Change-Id: I72389c8d564596bef22b47dfdcb2e77d636ef0a3
Merged-In: I72389c8d564596bef22b47dfdcb2e77d636ef0a3
Diffstat (limited to 'audio/effect/all-versions/default/EqualizerEffect.cpp')
-rw-r--r-- | audio/effect/all-versions/default/EqualizerEffect.cpp | 35 |
1 files changed, 24 insertions, 11 deletions
diff --git a/audio/effect/all-versions/default/EqualizerEffect.cpp b/audio/effect/all-versions/default/EqualizerEffect.cpp index 1b983ec5eb..fffe8cd884 100644 --- a/audio/effect/all-versions/default/EqualizerEffect.cpp +++ b/audio/effect/all-versions/default/EqualizerEffect.cpp @@ -31,9 +31,8 @@ namespace effect { namespace CPP_VERSION { namespace implementation { -EqualizerEffect::EqualizerEffect(effect_handle_t handle) : mEffect(new Effect(handle)) {} - -EqualizerEffect::~EqualizerEffect() {} +EqualizerEffect::EqualizerEffect(effect_handle_t handle) + : mEffect(new Effect(false /*isInput*/, handle)) {} void EqualizerEffect::propertiesFromHal(const t_equalizer_settings& halProperties, IEqualizerEffect::AllProperties* properties) { @@ -80,10 +79,32 @@ Return<Result> EqualizerEffect::disable() { return mEffect->disable(); } +#if MAJOR_VERSION <= 6 +Return<Result> EqualizerEffect::setAudioSource(AudioSource source) { + return mEffect->setAudioSource(source); +} + Return<Result> EqualizerEffect::setDevice(AudioDeviceBitfield device) { return mEffect->setDevice(device); } +Return<Result> EqualizerEffect::setInputDevice(AudioDeviceBitfield device) { + return mEffect->setInputDevice(device); +} +#else +Return<Result> EqualizerEffect::setAudioSource(const AudioSource& source) { + return mEffect->setAudioSource(source); +} + +Return<Result> EqualizerEffect::setDevice(const DeviceAddress& device) { + return mEffect->setDevice(device); +} + +Return<Result> EqualizerEffect::setInputDevice(const DeviceAddress& device) { + return mEffect->setInputDevice(device); +} +#endif + Return<void> EqualizerEffect::setAndGetVolume(const hidl_vec<uint32_t>& volumes, setAndGetVolume_cb _hidl_cb) { return mEffect->setAndGetVolume(volumes, _hidl_cb); @@ -103,10 +124,6 @@ Return<Result> EqualizerEffect::setConfigReverse( return mEffect->setConfigReverse(config, inputBufferProvider, outputBufferProvider); } -Return<Result> EqualizerEffect::setInputDevice(AudioDeviceBitfield device) { - return mEffect->setInputDevice(device); -} - Return<void> EqualizerEffect::getConfig(getConfig_cb _hidl_cb) { return mEffect->getConfig(_hidl_cb); } @@ -128,10 +145,6 @@ Return<Result> EqualizerEffect::setAuxChannelsConfig(const EffectAuxChannelsConf return mEffect->setAuxChannelsConfig(config); } -Return<Result> EqualizerEffect::setAudioSource(AudioSource source) { - return mEffect->setAudioSource(source); -} - Return<Result> EqualizerEffect::offload(const EffectOffloadParameter& param) { return mEffect->offload(param); } |