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 Jul 1, 2022
1 parent 850c510 commit 1788094
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 1788094

Please sign in to comment.