Improve shut down sequence for SequansController and LTE

Author: M65649

src/lte.cpp

@@ -252,18 +252,23 @@ bool LteClass::begin(const bool print_messages) {
 }
 
 void LteClass::end(void) {
-    is_connected = false;
-    got_timezone = false;
 
-    SequansController.unregisterCallback(CEREG_CALLBACK);
-    SequansController.unregisterCallback(TIMEZONE_CALLBACK);
-    SequansController.writeCommand(AT_DISCONNECT);
+    if (SequansController.isInitialized()) {
 
-    // Wait for the CEREG URC after disconnect so that the modem doesn't have
-    // any pending URCs and won't prevent going to sleep
-    SequansController.waitForURC(CEREG_CALLBACK);
+        SequansController.unregisterCallback(TIMEZONE_CALLBACK);
+        SequansController.writeCommand(AT_DISCONNECT);
+
+        // Wait for the CEREG URC after disconnect so that the modem doesn't
+        // have any pending URCs and won't prevent going to sleep
+        uint32_t start = millis();
+        while (isConnected() && millis() - start < 2000) {}
+        SequansController.unregisterCallback(CEREG_CALLBACK);
 
-    SequansController.end();
+        SequansController.end();
+    }
+
+    got_timezone = false;
+    is_connected = false;
 }
 
 String LteClass::getOperator(void) {

src/sequans_controller.cpp

@@ -538,15 +538,22 @@ void SequansControllerClass::end(void) {
     USART1.CTRLB = 0;
     USART1.CTRLC = 0;
 
-    pinConfigure(CTS_PIN, PIN_ISC_DISABLE);
+    pinConfigure(RESET_PIN, PIN_INPUT_DISABLE | PIN_DIR_INPUT);
 
-    // Set RTS high to halt the modem
-    pinConfigure(RTS_PIN, PIN_DIR_OUTPUT);
+    // Set RTS high to halt the modem. Has external pull-up, so is just set to
+    // input afterwards
     digitalWrite(RTS_PIN, HIGH);
+    pinConfigure(RTS_PIN, PIN_DIR_INPUT | PIN_INPUT_DISABLE);
 
+    pinConfigure(RING_PIN, PIN_DIR_INPUT | PIN_INPUT_DISABLE);
     detachInterrupt(RING_PIN);
+
+    pinConfigure(CTS_PIN, PIN_DIR_INPUT | PIN_INPUT_DISABLE);
     detachInterrupt(CTS_PIN);
 
+    pinConfigure(TX_PIN, PIN_DIR_INPUT | PIN_PULLUP_ON | PIN_INPUT_DISABLE);
+    pinConfigure(RX_PIN, PIN_DIR_INPUT | PIN_PULLUP_ON | PIN_INPUT_DISABLE);
+
     initialized = false;
 }