MCU8MASS-1170 Introduce adjustable timeout for Lte.begin()

Author: M65649

src/lte.cpp

@@ -1,6 +1,7 @@
+#include "lte.h"
+
 #include "led_ctrl.h"
 #include "log.h"
-#include "lte.h"
 #include "mqtt_client.h"
 #include "sequans_controller.h"
 
@@ -97,14 +98,22 @@ static void timezoneCallback(__attribute__((unused)) char* buffer) {
     got_timezone = true;
 }
 
-bool LteClass::begin(const bool print_messages) {
+bool LteClass::begin(const uint32_t timeout_ms, const bool print_messages) {
+
+    const uint32_t start = millis();
 
     // If low power is utilized, sequans controller will already been
     // initialized, so don't reset it by calling begin again
     if (!SequansController.isInitialized()) {
         SequansController.begin();
     }
 
+    // Disconnect before configuration if already connected
+    SequansController.writeCommand(AT_DISCONNECT);
+
+    delay(100);
+    SequansController.clearReceiveBuffer();
+
     SequansController.registerCallback(TIMEZONE_CALLBACK, timezoneCallback);
 
     SequansController.writeCommand(AT_ENABLE_TIME_ZONE_UPDATE);
@@ -154,7 +163,7 @@ bool LteClass::begin(const bool print_messages) {
         Log.infof("Connecting to operator");
     }
 
-    while (!isConnected()) {
+    while (!isConnected() && millis() - start < timeout_ms) {
         LedCtrl.toggle(Led::CELL, true);
         delay(500);
 
@@ -163,6 +172,20 @@ bool LteClass::begin(const bool print_messages) {
         }
     }
 
+    if (millis() - start >= timeout_ms) {
+
+        Log.rawf(" Was not able to connect to the network within the timeout "
+                 "of %d ms. Consider increasing the timeout or checking your "
+                 "cellular coverage.\r\n",
+                 timeout_ms);
+
+        SequansController.unregisterCallback(CEREG_CALLBACK);
+        SequansController.unregisterCallback(TIMEZONE_CALLBACK);
+        SequansController.writeCommand(AT_DISCONNECT);
+
+        return false;
+    }
+
     if (print_messages) {
         Log.rawf(" OK!\r\n");
     }
@@ -264,6 +287,7 @@ void LteClass::end(void) {
         while (isConnected() && millis() - start < 2000) {}
         SequansController.unregisterCallback(CEREG_CALLBACK);
 
+        SequansController.clearReceiveBuffer();
         SequansController.end();
     }
 

src/lte.h

@@ -5,6 +5,8 @@
 #ifndef LTE_H
 #define LTE_H
 
+#include <Arduino.h>
+
 #include <stdint.h>
 
 class LteClass {
@@ -20,7 +22,7 @@ class LteClass {
     /**
      * @brief Singleton instance.
      */
-    static LteClass &instance(void) {
+    static LteClass& instance(void) {
         static LteClass instance;
         return instance;
     }
@@ -29,12 +31,15 @@ class LteClass {
      * @brief Initializes the LTE module and its controller interface. Connects
      * to the network.
      *
+     * @param timeout_ms The amount of time to wait for connection before
+     * aborting.
      * @param print_messages If set to true, the messages related to connection
      * will be logged.
      *
      * @return True if initialization was successful and connection was made.
      */
-    bool begin(const bool print_messages = true);
+    bool begin(const uint32_t timeout_ms = 240000,
+               const bool print_messages = true);
 
     /**
      * @brief Disables the interface with the LTE module. Disconnects from