Increase SPP_TX_MAX buffer size. Move congested message to debug level. Report more heap info.

Author: nseidle

Firmware/RTK_Surveyor/System.ino

@@ -654,7 +654,7 @@ void reportHeap()
     if (millis() - lastHeapReport > 1000)
     {
       lastHeapReport = millis();
-      Serial.printf("FreeHeap: %d\n\r", ESP.getFreeHeap());
+      Serial.printf("FreeHeap: %d / HeapLowestPoint: %d / LargestBlock: %d\n\r", ESP.getFreeHeap(), xPortGetMinimumEverFreeHeapSize(), heap_caps_get_largest_free_block(MALLOC_CAP_8BIT));
     }
   }
 }

Firmware/RTK_Surveyor/src/BluetoothSerial/BluetoothSerial.cpp

@@ -158,7 +158,18 @@ static esp_err_t _spp_queue_packet(uint8_t *data, size_t len){
     return ESP_OK;
 }
 
-const uint16_t SPP_TX_MAX = 330;
+//SPP_TX_MAX seems to default to a lower amount (330) until there is congestion it then
+//tries to catch up but at 330, it cannot and the heap begins to take the hit.
+//Increasing the max to 2048 we see normal verbose xfers of 512 until congestion
+//when it increases briefly to 2048 then returns to 512 with no heap hit.
+//The rate at which we congest is dependant on how much we are attempting to TX and
+//how much is coming in RX from the phone.
+//At ~25kBps heap lowest point is 100k and stable
+//At ~50kBps heap lowest point is ~78k and stable
+//Above 50kbps it becomes unstable
+
+//const uint16_t SPP_TX_MAX = 330; //Original 
+const uint16_t SPP_TX_MAX = 512*4;
 static uint8_t _spp_tx_buffer[SPP_TX_MAX];
 static uint16_t _spp_tx_buffer_len = 0;
 
@@ -288,10 +299,13 @@ static void esp_spp_cb(esp_spp_cb_event_t event, esp_spp_cb_param_t *param)
     case ESP_SPP_CONG_EVT://connection congestion status changed
         if(param->cong.cong){
             xEventGroupClearBits(_spp_event_group, SPP_CONGESTED);
-        } else {
+            log_d("ESP_SPP_CONG_EVT: CONGESTED");
+        }
+        else
+        {
             xEventGroupSetBits(_spp_event_group, SPP_CONGESTED);
         }
-        log_v("ESP_SPP_CONG_EVT: %s", param->cong.cong?"CONGESTED":"FREE");
+        //log_v("ESP_SPP_CONG_EVT: %s", param->cong.cong ? "CONGESTED" : "FREE");
         break;
 
     case ESP_SPP_WRITE_EVT://write operation completed

Firmware/RTK_Surveyor/src/BluetoothSerial/BluetoothSerial.h

@@ -35,7 +35,7 @@ class BluetoothSerial: public Stream
         BluetoothSerial(void);
         ~BluetoothSerial(void);
 
-        bool begin(String localName=String(), bool isMaster=false, uint16_t rxQueueSize = 512, uint16_t txQueueSize = 512);
+        bool begin(String localName=String(), bool isMaster=false, uint16_t rxQueueSize = 512 * 4, uint16_t txQueueSize = 512);
         int available(void);
         int peek(void);
         bool hasClient(void);