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

Commit e2432ea1 authored by Eric Laurent's avatar Eric Laurent
Browse files

audio: Add supportsBluetoothVariableLatency API to IModule 

Add API to query if the Module supports control of variable latency over
playback paths.

Bug: 197823000
Test: atest VtsHalAudioCoreTargetTest
Change-Id: I3a08d682a1a639d8373869725e12702fff962d53
parent da4d0bcb

audio/aidl/aidl_api/android.hardware.audio.core/current/android/hardware/audio/core/IModule.aidl

+1 −0
Original line number Diff line number Diff line
@@ -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;

audio/aidl/android/hardware/audio/core/IModule.aidl

+11 −0
Original line number Diff line number Diff line
@@ -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();

}

audio/aidl/android/hardware/audio/core/IStreamOut.aidl

+4 −2
Original line number Diff line number Diff line
@@ -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.

audio/aidl/default/Module.cpp

+6 −0
Original line number Diff line number Diff line
@@ -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

audio/aidl/default/include/core-impl/Module.h

+1 −0
Original line number Diff line number Diff line
@@ -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(