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

Commit 88d54da6 authored by Mikhail Naganov's avatar Mikhail Naganov
Browse files

Fix the format used by ThreadBase::toAudioPortConfig 

Since 'toAudioPortConfig' is used to communicate to the HAL,
the emitted port config specification must use the HAL
audio format, not the processing format. This is consistent
with use of the 'format()' getter in other code, as it
also returns the HAL format.

Bug: 298175108
Test: atest audiosystem_tests
Change-Id: I4a09a9f312a1415f7dcb7381a8198a9d60467388
parent 49aecf0d

services/audioflinger/Threads.cpp

+1 −1
Original line number Diff line number Diff line
@@ -1801,7 +1801,7 @@ void AudioFlinger::ThreadBase::toAudioPortConfig(struct audio_port_config *confi 
    config->type = AUDIO_PORT_TYPE_MIX;

    config->ext.mix.handle = mId;

    config->sample_rate = mSampleRate;

    config->format = mFormat;

    config->format = mHALFormat;

    config->channel_mask = mChannelMask;

    config->config_mask = AUDIO_PORT_CONFIG_SAMPLE_RATE|AUDIO_PORT_CONFIG_CHANNEL_MASK|

                            AUDIO_PORT_CONFIG_FORMAT;