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

Commit f013135d authored by Chris Manton's avatar Chris Manton Committed by Automerger Merge Worker
Browse files

Only log when data length actually changes am: cad24262 

Original change: https://android-review.googlesource.com/c/platform/packages/modules/Bluetooth/+/2578786



Change-Id: I09fe76f83e4f08499882fa7d5f10447d9ad722f0
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 6df03b34 cad24262

system/stack/l2cap/l2c_ble.cc

+16 −5
Original line number Diff line number Diff line
@@ -1424,11 +1424,22 @@ void l2cble_process_data_length_change_event(uint16_t handle, 
  }



  if (is_legal_tx_data_len(tx_data_len)) {

    LOG_DEBUG("Received data length change event for device:%s tx_data_len:%hu",

              ADDRESS_TO_LOGGABLE_CSTR(p_lcb->remote_bd_addr), tx_data_len);

    p_lcb->tx_data_len = tx_data_len;

    if (p_lcb->tx_data_len != tx_data_len) {

      LOG_DEBUG(

          "Received data length change event for device:%s tx_data_len:%hu => "

          "%hu",

          ADDRESS_TO_LOGGABLE_CSTR(p_lcb->remote_bd_addr), p_lcb->tx_data_len,

          tx_data_len);

      BTM_LogHistory(kBtmLogTag, p_lcb->remote_bd_addr, "LE Data length change",

                   base::StringPrintf("tx_octets:%hu", tx_data_len));

                     base::StringPrintf("tx_octets:%hu => %hu",

                                        p_lcb->tx_data_len, tx_data_len));

      p_lcb->tx_data_len = tx_data_len;

    } else {

      LOG_DEBUG(

          "Received duplicated data length change event for device:%s "

          "tx_data_len:%hu",

          ADDRESS_TO_LOGGABLE_CSTR(p_lcb->remote_bd_addr), tx_data_len);

    }

  } else {

    LOG_WARN(

        "Received illegal data length change event for device:%s "