From ee27c15c2c085f1dbe327f0140d45fd15ad5c36a Mon Sep 17 00:00:00 2001
From: geeksville <kevinh@geeksville.com>
Date: Thu, 10 Sep 2020 09:24:48 -0700
Subject: [PATCH] likely fix for bug #373. fix #339. great gps fixes from
 @a-f-G-U-C fixes described in bug #376

---
 proto                |  2 +-
 src/gps/UBloxGPS.cpp | 16 ++++++++++------
 src/mesh/mesh.pb.h   |  4 ++--
 3 files changed, 13 insertions(+), 9 deletions(-)

diff --git a/proto b/proto
index 3caee2e5..ce422b7c 160000
--- a/proto
+++ b/proto
@@ -1 +1 @@
-Subproject commit 3caee2e5b92d4429c6cf47a4c88d4615001f300b
+Subproject commit ce422b7c448906c6fee3eef64bbd41adfbc990f0
diff --git a/src/gps/UBloxGPS.cpp b/src/gps/UBloxGPS.cpp
index 36203964..624ab77b 100644
--- a/src/gps/UBloxGPS.cpp
+++ b/src/gps/UBloxGPS.cpp
@@ -46,8 +46,9 @@ bool UBloxGPS::setup()
     // ublox.enableDebugging(Serial);
 
     // try a second time, the ublox lib serial parsing is buggy?
-    if (!tryConnect())
-        tryConnect();
+    // see https://github.com/meshtastic/Meshtastic-device/issues/376
+    for (int i = 0; (i < 3) && !tryConnect(); i++)
+        delay(500);
 
     if (isConnected) {
         DEBUG_MSG("Connected to UBLOX GPS successfully\n");
@@ -81,8 +82,11 @@ bool UBloxGPS::setUBXMode()
     // assert(ok);
     // ok = ublox.setDynamicModel(DYN_MODEL_BIKE); // probably PEDESTRIAN but just in case assume bike speeds
     // assert(ok);
-    if (!ublox.powerSaveMode(true, 2000)) // use power save mode, the default timeout (1100ms seems a bit too tight)
-        return false;
+
+    // per https://github.com/meshtastic/Meshtastic-device/issues/376 powerSaveMode might not work with the marginal
+    // TTGO antennas
+    // if (!ublox.powerSaveMode(true, 2000)) // use power save mode, the default timeout (1100ms seems a bit too tight)
+    //     return false;
 
     if (!ublox.saveConfiguration(3000))
         return false;
@@ -106,8 +110,8 @@ bool UBloxGPS::factoryReset()
     tryConnect(); // sets isConnected
 
     // try a second time, the ublox lib serial parsing is buggy?
-    if (!tryConnect())
-        tryConnect();
+    for (int i = 0; (i < 3) && !tryConnect(); i++)
+        delay(500);
 
     DEBUG_MSG("GPS Factory reset success=%d\n", isConnected);
     if (isConnected)
diff --git a/src/mesh/mesh.pb.h b/src/mesh/mesh.pb.h
index 27ff63f9..64b2758c 100644
--- a/src/mesh/mesh.pb.h
+++ b/src/mesh/mesh.pb.h
@@ -364,7 +364,7 @@ typedef struct _ToRadio {
 #define DeviceState_version_tag                  8
 #define DeviceState_rx_text_message_tag          7
 #define DeviceState_no_save_tag                  9
-#define DeviceState_did_gps_reset_tag            10
+#define DeviceState_did_gps_reset_tag            11
 #define FromRadio_packet_tag                     2
 #define FromRadio_my_info_tag                    3
 #define FromRadio_node_info_tag                  4
@@ -519,7 +519,7 @@ X(a, STATIC,   REPEATED, MESSAGE,  receive_queue,     5) \
 X(a, STATIC,   OPTIONAL, MESSAGE,  rx_text_message,   7) \
 X(a, STATIC,   SINGULAR, UINT32,   version,           8) \
 X(a, STATIC,   SINGULAR, BOOL,     no_save,           9) \
-X(a, STATIC,   SINGULAR, BOOL,     did_gps_reset,    10)
+X(a, STATIC,   SINGULAR, BOOL,     did_gps_reset,    11)
 #define DeviceState_CALLBACK NULL
 #define DeviceState_DEFAULT NULL
 #define DeviceState_radio_MSGTYPE RadioConfig