From ec98d5f8a51dc497259610a4038ac25a35dbe112 Mon Sep 17 00:00:00 2001 From: Lloyd Date: Wed, 2 Jul 2025 23:41:31 +0100 Subject: [PATCH] BLE: Remove ScanResponse.addName() to fix re-advertising after disconnect Removed the call to Bluefruit.ScanResponse.addName() in startAdv(), as it was preventing BLE from reliably restarting advertising after a disconnect. Hypothesis: adding the device name to the scan response exceeds internal buffer limits or causes a conflict with advertising timing, leading to the BLE stack silently failing to re-advertise. Tested successfully (on T-1000) without this line, advertising now resumes correctly after disconnection (on Iphone) --- src/helpers/nrf52/SerialBLEInterface.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/helpers/nrf52/SerialBLEInterface.cpp b/src/helpers/nrf52/SerialBLEInterface.cpp index f43a3767..c6c99d5b 100644 --- a/src/helpers/nrf52/SerialBLEInterface.cpp +++ b/src/helpers/nrf52/SerialBLEInterface.cpp @@ -27,7 +27,7 @@ void SerialBLEInterface::startAdv() { // Secondary Scan Response packet (optional) // Since there is no room for 'Name' in Advertising packet - Bluefruit.ScanResponse.addName(); + // Bluefruit.ScanResponse.addName(); /* Start Advertising * - Enable auto advertising if disconnected