Evolution-x/hardware_interfaces
1
0
Fork 0
mirror of https://github.com/Evolution-X/hardware_interfaces synced 2026-02-01 22:04:26 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "audio: Add supportsBluetoothVariableLatency API to IModule" am: 91ca65c35c

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2388415

Change-Id: I7557aa97b92492bc93791144f74f6621bb3c92c4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Treehugger Robot
2023-01-19 10:36:13 +00:00
committed by Automerger Merge Worker
parent d73b58e92c 91ca65c35c
commit cc29d42071
6 changed files with 29 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
audio/aidl/aidl_api/android.hardware.audio.core/current/android/hardware/audio/core/IModule.aidl
View File

@@ -69,6 +69,7 @@ interface IModule {
void addDeviceEffect(int portConfigId, in android.hardware.audio.effect.IEffect effect);
void removeDeviceEffect(int portConfigId, in android.hardware.audio.effect.IEffect effect);
android.media.audio.common.AudioMMapPolicyInfo[] getMmapPolicyInfos(android.media.audio.common.AudioMMapPolicyType mmapPolicyType);
boolean supportsVariableLatency();
@VintfStability
parcelable OpenInputStreamArguments {
int portConfigId;

11
audio/aidl/android/hardware/audio/core/IModule.aidl
View File

@@ -822,4 +822,15 @@ interface IModule {
* @return The vector with mmap policy information.
*/
AudioMMapPolicyInfo[] getMmapPolicyInfos(AudioMMapPolicyType mmapPolicyType);
/**
* Indicates if this module supports variable latency control for instance
* over Bluetooth A2DP or LE Audio links.
*
* If supported, all instances of IStreamOut interface returned by this module must
* implement getRecommendedLatencyModes() and setLatencyMode() APIs.
*
* @return Whether the module supports variable latency control.
*/
boolean supportsVariableLatency();
}

6
audio/aidl/android/hardware/audio/core/IStreamOut.aidl
View File

@@ -157,7 +157,8 @@ interface IStreamOut {
*
* Implementation for this method is mandatory only on specific spatial
* audio streams indicated by AUDIO_OUTPUT_FLAG_SPATIALIZER flag if they can
* be routed to a BT classic sink.
* be routed to a BT sinks or if the implementation indicates support
* on all streams via IModule.supportsVariableLatency().
*
* @return Currently supported latency modes.
* @throws EX_ILLEGAL_STATE If the stream is closed.
@@ -172,7 +173,8 @@ interface IStreamOut {
*
* Implementation for this method is mandatory only on specific spatial
* audio streams indicated by AUDIO_OUTPUT_FLAG_SPATIALIZER flag if they can
* be routed to a BT classic sink.
* be routed to a BT sinks or if the implementation indicates support
* on all streams via IModule.supportsVariableLatency().
*
* @throws EX_ILLEGAL_ARGUMENT If the specified mode is not supported.
* @throws EX_ILLEGAL_STATE If the stream is closed.

6
audio/aidl/default/Module.cpp
View File

@@ -1145,4 +1145,10 @@ ndk::ScopedAStatus Module::getMmapPolicyInfos(AudioMMapPolicyType mmapPolicyType
return ndk::ScopedAStatus::ok();
}
ndk::ScopedAStatus Module::supportsVariableLatency(bool* _aidl_return) {
LOG(DEBUG) << __func__;
*_aidl_return = false;
return ndk::ScopedAStatus::ok();
}
} // namespace aidl::android::hardware::audio::core

1
audio/aidl/default/include/core-impl/Module.h
View File

@@ -114,6 +114,7 @@ class Module : public BnModule {
::aidl::android::media::audio::common::AudioMMapPolicyType mmapPolicyType,
std::vector<::aidl::android::media::audio::common::AudioMMapPolicyInfo>* _aidl_return)
override;
ndk::ScopedAStatus supportsVariableLatency(bool* _aidl_return) override;
void cleanUpPatch(int32_t patchId);
ndk::ScopedAStatus createStreamContext(

6
audio/aidl/vts/VtsHalAudioCoreModuleTargetTest.cpp
View File

@@ -1907,6 +1907,12 @@ TEST_P(AudioCoreModule, GetMmapPolicyInfos) {
}
}
TEST_P(AudioCoreModule, BluetoothVariableLatency) {
bool isSupported = false;
EXPECT_IS_OK(module->supportsVariableLatency(&isSupported));
LOG(INFO) << "supportsVariableLatency: " << isSupported;
}
class AudioCoreBluetooth : public AudioCoreModuleBase, public testing::TestWithParam<std::string> {
public:
void SetUp() override {