Update GPSUpdateScheduling.cpp (#5160)

* Update GPSUpdateScheduling.cpp Default value is too short, resulting in unstable GPS locks on T1000-E (possibly others). Fix has been tested an confirmed working with no adverse effects, by multiple users. Also discussed at length on Discord * Coerce minimum instead of hardcode * config --------- Co-authored-by: Ben Meadors <[email protected]>
meshtastic · Oct 28, 2024 · 77dfc92 · 77dfc92
1 parent c4eb9a6
commit 77dfc92
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/gps/GPSUpdateScheduling.cpp b/src/gps/GPSUpdateScheduling.cpp
@@ -70,9 +70,10 @@ bool GPSUpdateScheduling::isUpdateDue()
 // Have we been searching for a GPS position for too long?
 bool GPSUpdateScheduling::searchedTooLong()
 {
+    uint32_t minimumOrConfiguredSecs = Default::getConfiguredOrMinimumValue(
+            config.position.position_broadcast_secs, default_broadcast_interval_secs);
     uint32_t maxSearchMs =
-        Default::getConfiguredOrDefaultMs(config.position.position_broadcast_secs, default_broadcast_interval_secs);
-
+        Default::getConfiguredOrDefaultMs(minimumOrConfiguredSecs, default_broadcast_interval_secs);
     // If broadcast interval set to max, no such thing as "too long"
     if (maxSearchMs == UINT32_MAX)
         return false;
@@ -115,4 +116,4 @@ void GPSUpdateScheduling::updateLockTimePrediction()
 uint32_t GPSUpdateScheduling::predictedSearchDurationMs()
 {
     return GPSUpdateScheduling::predictedMsToGetLock;
-}
+}