Commit fb3d5940 authored Sep 11, 2023 by Mikhail Naganov Committed by Cherrypicker Worker Sep 18, 2023

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
(cherry picked from https://android-review.googlesource.com/q/commit:88d54da6215bb47f5b0ac6b0c548b6a76c9582bf)
Merged-In: I4a09a9f312a1415f7dcb7381a8198a9d60467388
Change-Id: I4a09a9f312a1415f7dcb7381a8198a9d60467388

parent c1a739c3

services/audioflinger/Threads.cpp

+1 −1

Original line number	Diff line number	Diff line
		@@ -1847,7 +1847,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;