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

Commit b6bcd5dc authored by Andre Eisenbach's avatar Andre Eisenbach Committed by android-build-merger
Browse files

DO NOT MERGE Blacklist devices for absolute volume control 

am: 20383eff

* commit '20383eff':
  DO NOT MERGE Blacklist devices for absolute volume control
parents 8f8fb28e 20383eff

system/btif/src/btif_rc.c

+3 −5
Original line number Diff line number Diff line
@@ -36,6 +36,7 @@ 
#include "btif_common.h"

#include "btif_util.h"

#include "btif_av.h"

#include "device/include/interop.h"

#include "hardware/bt_rc.h"

#include "uinput.h"
@@ -324,11 +325,8 @@ void handle_rc_features() 
    bt_bdaddr_t rc_addr;

    bdcpy(rc_addr.address, btif_rc_cb.rc_addr);



    // TODO(eisenbach): If devices need to be blacklisted for absolute

    // volume, it should be added to device/include/interop_database.h

    // For now, everything goes... If blacklisting is necessary, exclude

    // the following bit here:

    //    btif_rc_cb.rc_features &= ~BTA_AV_FEAT_ADV_CTRL;

    if (interop_match(INTEROP_DISABLE_ABSOLUTE_VOLUME, &rc_addr))

        btif_rc_cb.rc_features &= ~BTA_AV_FEAT_ADV_CTRL;



    if (btif_rc_cb.rc_features & BTA_AV_FEAT_BROWSE)

    {

system/device/include/interop.h

+6 −1
Original line number Diff line number Diff line
@@ -34,7 +34,12 @@ typedef enum { 
  // requiring multiple retries to complete pairing. To avoid degrading the user

  // experience for those devices, automatically re-try pairing if page

  // timeouts are received during pairing.

  INTEROP_AUTO_RETRY_PAIRING

  INTEROP_AUTO_RETRY_PAIRING,



  // Devices requiring this workaround do not handle Bluetooth Absolute Volume

  // control correctly, leading to undesirable (potentially harmful) volume levels

  // or general lack of controlability.

  INTEROP_DISABLE_ABSOLUTE_VOLUME

} interop_feature_t;



// Check if a given |addr| matches a known interoperability workaround as identified

system/device/include/interop_database.h

+17 −1
Original line number Diff line number Diff line
@@ -43,5 +43,21 @@ static const interop_entry_t interop_database[] = { 
  {{0x80, 0xe4, 0xda, 0x70,   0,0}, 4, INTEROP_DISABLE_LE_SECURE_CONNECTIONS},



  // BMW car kits (Harman/Becker)

  {{0x9c, 0xdf, 0x03,       0,0,0}, 3, INTEROP_AUTO_RETRY_PAIRING}

  {{0x9c, 0xdf, 0x03,       0,0,0}, 3, INTEROP_AUTO_RETRY_PAIRING},



  // JayBird BlueBuds X - low granularity on volume control

  {{0x44, 0x5e, 0xf3,       0,0,0}, 3, INTEROP_DISABLE_ABSOLUTE_VOLUME},



  // LG Tone HBS-730 - unacceptably loud volume

  {{0x00, 0x18, 0x6b,       0,0,0}, 3, INTEROP_DISABLE_ABSOLUTE_VOLUME},

  {{0xb8, 0xad, 0x3e,       0,0,0}, 3, INTEROP_DISABLE_ABSOLUTE_VOLUME},



  // LG Tone HV-800 - unacceptably loud volume

  {{0xa0, 0xe9, 0xdb,       0,0,0}, 3, INTEROP_DISABLE_ABSOLUTE_VOLUME},



  // SOL REPUBLIC Tracks Air - unable to adjust volume back off from max

  {{0xa4, 0x15, 0x66,       0,0,0}, 3, INTEROP_DISABLE_ABSOLUTE_VOLUME},



  // VW Car Kit - not enough granularity with volume

  {{0x00, 0x26, 0x7e,       0,0,0}, 3, INTEROP_DISABLE_ABSOLUTE_VOLUME},

};

system/device/src/interop.c

+1 −0
Original line number Diff line number Diff line
@@ -96,6 +96,7 @@ static const char* interop_feature_string_(const interop_feature_t feature) { 
  switch (feature) {

    CASE_RETURN_STR(INTEROP_DISABLE_LE_SECURE_CONNECTIONS)

    CASE_RETURN_STR(INTEROP_AUTO_RETRY_PAIRING)

    CASE_RETURN_STR(INTEROP_DISABLE_ABSOLUTE_VOLUME)

  }



  return "UNKNOWN";