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

audiohal: Support stateful downmixer effects

Browse source 
Fraunhofer's downmixer has additional buffering, this is incompatible
with how DownmixerBufferProvider used to process incomplete buffers.

Now the effects HIDL wrapper delivers frameCount updates in
audiobuffers to the server side. This fixes playback of multichannel
sound streams.

Bug: 36181621
Test: play 5.1 track from Play Music
Change-Id: Iffe3327c6eedef85151eec2b41e0aedfb7ffa6dc
This commit is contained in:
Mikhail Naganov 2017-04-26 17:22:31 -07:00
parent 8de7a17b82
commit c8c6a81f0f
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
audio/effect/2.0/default/AudioBufferManager.cpp
View file

@ -30,7 +30,10 @@ bool AudioBufferManager::wrap(const AudioBuffer& buffer, sp<AudioBufferWrapper>*
ssize_t idx = mBuffers.indexOfKey(buffer.id);
if (idx >= 0) {
*wrapper = mBuffers[idx].promote();
if (*wrapper != nullptr) return true;
if (*wrapper != nullptr) {
(*wrapper)->getHalBuffer()->frameCount = buffer.frameCount;
return true;
}
mBuffers.removeItemsAt(idx);
}
// Need to create and init a new AudioBufferWrapper.