| 9f2890458a2e0b9fa09ceeba64c869bf3f1560c3 |
|
23-Feb-2017 |
Mikhail Naganov <mnaganov@google.com> |
Add VTS tests for audio effects
Added tests covering IEffect, IEqualizerEffect, and
ILoudnessEnhancer interfaces.
Minor corrections in the interface definitions and implementations:
- fixed descriptions and @callflow annotations in IEffect;
- fixed type used for band levels in IEqualizerEffect;
- fixed specification of frequencies in IEqualizerEffect;
- fixed some bugs in previously non-execrices Effects code;
- warning messages changed to error messages.
Test: this is a test
Bug: 32022706
Change-Id: I0e0bc111b07d944ad8a0321e8b1ec703f8d1a73e
/hardware/interfaces/audio/effect/2.0/default/Effect.h
|
| f4f2ff39746a36b3d657e34ec1724589051b1ec8 |
|
19-Jan-2017 |
Mikhail Naganov <mnaganov@google.com> |
audiohal: Fix volume changes handling
Some legacy implementations of the effects HAL use the condition
of the reply buffer being NULL as an indication that they shouldn't
apply attenuation to the input audio data. Therefore, separate
methods are needed to distinguish the use cases of delegating the
volume control to the effect, and just informing the effect of the
volume changes.
A new method added to IEffect: volumeChangeNotification that implements
the second use case. The contract of setAndGetVolume method has been
updated to indicate that it is only called in the first use case.
Also updated the wrapper for a generic IEffect commands to pass NULL
pointers to the command and reply buffers in case when the size
of the input or output data is 0, to preserve compatibility with
direct calls from the framework.
Bug: 34368451
Test: volume control works when both Bass Boost and Equalizer are
enabled in the NXP implementation of the effects
Change-Id: I3c9a5bbdff561802bc94080c51703385a8903282
/hardware/interfaces/audio/effect/2.0/default/Effect.h
|
| a331de14219b34b08f4cbb462a45f1fd82a92d48 |
|
05-Jan-2017 |
Mikhail Naganov <mnaganov@google.com> |
audiohal: Re-implement effect process using FMQ and IMemory
Result: no hwbinder calls due music processing.
Added IEffect.close method for explicitly freeing up of resources
consumed by the effect before automatic server objects reaping
gets to it.
Added IEffect.setProcessBuffers method for updating the input /
output buffers on the go.
Test: make, use Play Music with effects, check traces
Bug: 30222631
Change-Id: Ia1e1bc7098fab59aa970e0ce4acdb48007409644
/hardware/interfaces/audio/effect/2.0/default/Effect.h
|
| a9a58c150de0bdbf795e06218d1d4d19e10cf5bf |
|
02-Dec-2016 |
Mikhail Naganov <mnaganov@google.com> |
Merge "Audio HAL: fixes for issues discovered after client conversion"
|
| 6e81e9bb3c13d8c51fde503cf8be2bfb56e1ca1d |
|
17-Nov-2016 |
Mikhail Naganov <mnaganov@google.com> |
Audio HAL: fixes for issues discovered after client conversion
Several issues addressed:
-- added IDevice.supportsAudioPatches to query whether
create/removeAudioPatch is actually supported by HAL;
-- IStreamOutCallback proxy needs to be owned by IStreamOut
implementation. In order for the client to reset the reference,
added method IStreamOut.clearCallback;
-- IDevice.open{Input|Output}Stream need to return a "suggested" audio
config from HAL;
-- code for converting between system/audio.h and HIDL
data structures has been moved to
android.hardware.audio.common@2.0-util library for reuse;
-- added a workaround for the issue with QC effects HAL trying to write
into the input parameters buffer, which is r/o by Binder design.
Bug: 30222631
Change-Id: I64af24d79c12d6ac3b0f87d085a821913e29237b
Test: tried using with WIP HIDL client on N5X
/hardware/interfaces/audio/effect/2.0/default/Effect.h
|
| d6e4f03acf5fa1e76b75e33af7a26f248a92be9a |
|
29-Nov-2016 |
Steven Moreland <smoreland@google.com> |
Remove HIDL_GENERATED from things written by hand.
find . -name "*.h" -exec sed -i 's/HIDL_GENERATED_\(.*\)_H_/\U\1_H/g' {}
+
They now match what would be generated by -Lc++-impl. This prevents
confusion over whether or not the files are autogenerated.
Change-Id: I4aa89a39907024aa9525df71ff3715979f431357
Similar commit: https://android-review.googlesource.com/#/c/305996/
Test: pass
/hardware/interfaces/audio/effect/2.0/default/Effect.h
|
| 7cbf2f1fb15a2a7d9b8b61ca81ad93fa9ddcaea3 |
|
28-Oct-2016 |
Mikhail Naganov <mnaganov@google.com> |
Implement audio effects HAL delegating to legacy HAL
Changes made to the .hal definition:
- added missing generated Result for methods implemented via legacy
"command" function;
- fixed Aux Channels feature definition;
- added "size" parameter for reply data in cases where the wrapper
needs to allocate a reply buffer;
- added method for generic support of feature configs;
- added new Result type;
- use arrays instead of strings in effect descriptor to ease
conversion from / to legacy HAL;
- added missing method to the Preset Reverb interface;
- fixed names of the Visualizer enums to avoid clashes with defines
from the legacy HAL file.
The implementation isn't hooked up to the server yet. Need to implement
devices and streams first.
Bug: 30222631
Change-Id: I75bb42f19ac3303759e918b6d6a91646b1555f8c
Test: make
/hardware/interfaces/audio/effect/2.0/default/Effect.h
|