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

Commit 00bab711 authored by Henri Chataing's avatar Henri Chataing Committed by Gerrit Code Review
Browse files

Merge "btif_a2dp: Incorrect check for A2DP Offload Start status" into main

parents 15fd53bc ff1586bf

system/audio_hal_interface/a2dp_encoding.h

+7 −9
Original line number Diff line number Diff line
@@ -29,16 +29,14 @@ namespace bluetooth { 
namespace audio {

namespace a2dp {



/// Copied after the definition from the Bluetooth Audio interface:

/// Loosely copied after the definition from the Bluetooth Audio interface:

/// hardware/interfaces/bluetooth/audio/aidl/android/hardware/bluetooth/audio/BluetoothAudioStatus.aidl

enum BluetoothAudioStatus {

  UNKNOWN = 0,

  SUCCESS = 1,

  UNSUPPORTED_CODEC_CONFIGURATION = 2,

  FAILURE = 3,

  // Not defined in BluetoothAudioStatus, but used internally for

  // state keeping.

  PENDING = 5,

enum class BluetoothAudioStatus {

  SUCCESS = 0,

  UNKNOWN,

  UNSUPPORTED_CODEC_CONFIGURATION,

  FAILURE,

  PENDING,

};



bool update_codec_offloading_capabilities(

system/btif/src/btif_a2dp.cc

+1 −2
Original line number Diff line number Diff line
@@ -180,8 +180,7 @@ void btif_a2dp_on_offload_started(const RawAddress& peer_addr, 
  }



  if (btif_av_is_a2dp_offload_running()) {

    if (ack != BTA_AV_SUCCESS &&

        btif_av_stream_started_ready(A2dpType::kSource)) {

    if (ack != BluetoothAudioStatus::SUCCESS && btif_av_stream_started_ready(A2dpType::kSource)) {

      // Offload request will return with failure from btif_av sm if

      // suspend is triggered for remote start. Disconnect only if SoC

      // returned failure for offload VSC