Skip to content

Commit

Permalink
[ESP32] fix BLE indicate (#20108)
Browse files Browse the repository at this point in the history
19f5469 changes the method of 
"GATT C2 Characteristic" from NOTIFICATION to INDICATION.

So the ESP32 should support INDICATION, or it is unable to
communicate with android app.
  • Loading branch information
Kxuan authored and pull[bot] committed Jan 19, 2024
1 parent 895bf95 commit c5d1304
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/platform/ESP32/nimble/BLEManagerImpl.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ const struct ble_gatt_svc_def BLEManagerImpl::CHIPoBLEGATTAttrs[] = {
{
.uuid = (ble_uuid_t *) (&UUID_CHIPoBLEChar_TX),
.access_cb = gatt_svr_chr_access,
.flags = BLE_GATT_CHR_F_READ | BLE_GATT_CHR_F_NOTIFY,
.flags = BLE_GATT_CHR_F_READ | BLE_GATT_CHR_F_NOTIFY | BLE_GATT_CHR_F_INDICATE,
.val_handle = &sInstance.mTXCharCCCDAttrHandle,
},
#if CHIP_ENABLE_ADDITIONAL_DATA_ADVERTISING
Expand Down

0 comments on commit c5d1304

Please sign in to comment.