don't re-init pigeon while offroad

commaai · Dec 3, 2020 · 92beda6 · 92beda6
1 parent 4836617
commit 92beda6
Showing 1 changed file with 16 additions and 3 deletions.
diff --git a/selfdrive/boardd/boardd.cc b/selfdrive/boardd/boardd.cc
@@ -474,26 +474,39 @@ void pigeon_thread() {
 
   // ubloxRaw = 8042
   PubMaster pm({"ubloxRaw"});
+  bool ignition_last = false;
 
 #ifdef QCOM2
   Pigeon * pigeon = Pigeon::connect("/dev/ttyHS0");
 #else
   Pigeon * pigeon = Pigeon::connect(panda);
 #endif
 
-  pigeon->init();
+  if (ignition) {
+    pigeon->init();
+  }
 
   while (!do_exit && panda->connected) {
     std::string recv = pigeon->receive();
     if (recv.length() > 0) {
       if (recv[0] == (char)0x00){
-        LOGW("received invalid ublox message, resetting panda GPS");
-        pigeon->init();
+        if (ignition) {
+          LOGW("received invalid ublox message while onroad, resetting panda GPS");
+          pigeon->init();
+        }
       } else {
         pigeon_publish_raw(pm, recv);
       }
     }
 
+    // init pigeon on rising ignition edge
+    // since it was turned off in low power mode
+    if(ignition && !ignition_last) {
+      pigeon->init();
+    }
+
+    ignition_last = ignition;
+
     // 10ms - 100 Hz
     usleep(10*1000);
   }