Adding debug verbose output for OTALogic

Author: aentinger

src/utility/ota/OTALogic.cpp

@@ -30,6 +30,7 @@
 
 #ifndef HOST
   #include <Arduino.h>
+  #include <Arduino_DebugUtils.h>
 #else
   #include <algorithm> /* for std::min, otherwise Arduino defines min() */
   using namespace std;
@@ -123,6 +124,7 @@ void OTALogic::onOTADataReceived(uint8_t const * const data, size_t const length
 
 OTAState OTALogic::handle_Init()
 {
+  DBG_VERBOSE(__PRETTY_FUNCTION__);
   if (_ota_storage->init()) {
     return OTAState::Idle;
   } else {
@@ -141,6 +143,7 @@ OTAState OTALogic::handle_Idle()
 
 OTAState OTALogic::handle_StartDownload()
 {
+  DBG_VERBOSE(__PRETTY_FUNCTION__);
   if(_ota_storage->open()) {
     return OTAState::WaitForHeader;
   } else {
@@ -151,6 +154,7 @@ OTAState OTALogic::handle_StartDownload()
 
 OTAState OTALogic::handle_WaitForHeader()
 {
+  DBG_VERBOSE(__PRETTY_FUNCTION__);
   if(_mqtt_ota_buf.num_bytes >= OTA_BINARY_HEADER_SIZE) {
     return OTAState::HeaderReceived;
   }
@@ -159,6 +163,7 @@ OTAState OTALogic::handle_WaitForHeader()
 
 OTAState OTALogic::handle_HeaderReceived()
 {
+  DBG_VERBOSE(__PRETTY_FUNCTION__);
   /* The OTA header has been received, let's extract it
    * from the MQTT OTA receive buffer.
    */
@@ -179,6 +184,9 @@ OTAState OTALogic::handle_HeaderReceived()
   _ota_bin_data.hdr_len   = ota_header.header.len;
   _ota_bin_data.hdr_crc32 = ota_header.header.crc32;
 
+  DBG_VERBOSE("%s: header length = %d", __PRETTY_FUNCTION__, _ota_bin_data.hdr_len);
+  DBG_VERBOSE("%s: header CRC32 = %d", __PRETTY_FUNCTION__, _ota_bin_data.hdr_crc32);
+
   /* Reset the counter which is responsible for keeping tabs on how many bytes have been received */
   _ota_bin_data.bytes_received = 0;
 
@@ -218,6 +226,8 @@ OTAState OTALogic::handle_BinaryReceived()
   _ota_bin_data.bytes_received += _mqtt_ota_buf.num_bytes;
   _mqtt_ota_buf.num_bytes = 0;
 
+  DBG_VERBOSE("%s: %d bytes written", __PRETTY_FUNCTION__, _ota_bin_data.bytes_received);
+
   if(_ota_bin_data.bytes_received >= _ota_bin_data.hdr_len) {
     _ota_storage->close();
     _ota_bin_data.crc32 = crc_finalize(_ota_bin_data.crc32);
@@ -229,6 +239,7 @@ OTAState OTALogic::handle_BinaryReceived()
 
 OTAState OTALogic::handle_Verify()
 {
+  DBG_VERBOSE(__PRETTY_FUNCTION__);
   if(_ota_bin_data.crc32 == _ota_bin_data.hdr_crc32) {
     return OTAState::Rename;
   } else {
@@ -240,6 +251,7 @@ OTAState OTALogic::handle_Verify()
 
 OTAState OTALogic::handle_Rename()
 {
+  DBG_VERBOSE(__PRETTY_FUNCTION__);
   if(_ota_storage->rename()) {
     _ota_storage->deinit();
     return OTAState::Reset;
@@ -260,6 +272,8 @@ OTAState OTALogic::handle_Reset()
    * update before starting the application, otherwise the app
    * is started directly.
    */
+  DBG_VERBOSE(__PRETTY_FUNCTION__);
+  delay(250);
   NVIC_SystemReset();
 #endif /* HOST */
   return OTAState::Reset;