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

Commit ef65be72 authored by Mikhail Naganov's avatar Mikhail Naganov
Browse files

audio AIDL: Consider `BUS` devices as attached 

After analyzing Auto APM config files and checking with
the Auto team, it has turned out that `BUS` devices are
considered to be permanently attached. In the AIDL schema
that means, they must have an empty connection type.

A solution which seems to work well is to map AIDL types
`IN_|OUT_DEVICE` with no connection to legacy BUS devices.

Bug: 303926380
Test: atest audio_aidl_ndk_conversion_tests
Change-Id: Id98d632f47f4b4498fac5f92e092bc7d7485482e
parent 1dc7cc18

media/audioaidlconversion/AidlConversionCppNdk.cpp

+1 −2
Original line number Diff line number Diff line
@@ -562,8 +562,7 @@ const detail::AudioDevicePairs& getAudioDevicePairs() { 
                GET_DEVICE_DESC_CONNECTION(IP_V4));

        append_AudioDeviceDescription(pairs,

                AUDIO_DEVICE_IN_BUS, AUDIO_DEVICE_OUT_BUS,

                AudioDeviceType::IN_DEVICE, AudioDeviceType::OUT_DEVICE,

                GET_DEVICE_DESC_CONNECTION(BUS));

                AudioDeviceType::IN_DEVICE, AudioDeviceType::OUT_DEVICE);

        append_AudioDeviceDescription(pairs,

                AUDIO_DEVICE_IN_PROXY, AUDIO_DEVICE_OUT_PROXY,

                AudioDeviceType::IN_AFE_PROXY, AudioDeviceType::OUT_AFE_PROXY);