pandad: add log_once_ to prevent repeated logging (commaai#33505)

add log_once_ flag
Edison-CBS · Sep 7, 2024 · ee6d7ae · ee6d7ae
1 parent e3f05db
commit ee6d7ae
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/selfdrive/pandad/panda_safety.cc b/selfdrive/pandad/panda_safety.cc
@@ -17,6 +17,7 @@ void PandaSafety::configureSafetyMode() {
   } else if (!is_onroad) {
     initialized_ = false;
     safety_configured_ = false;
+    log_once_ = false;
   }
 }
 
@@ -46,9 +47,12 @@ std::string PandaSafety::fetchCarParams() {
   if (!params_.getBool("FirmwareQueryDone")) {
     return {};
   }
-  LOGW("Finished FW query");
 
-  LOGW("Waiting for params to set safety model");
+  if (!log_once_) {
+    LOGW("Finished FW query, Waiting for params to set safety model");
+    log_once_ = true;
+  }
+
   if (!params_.getBool("ControlsReady")) {
     return {};
   }

diff --git a/selfdrive/pandad/pandad.h b/selfdrive/pandad/pandad.h
@@ -19,6 +19,7 @@ class PandaSafety {
   void setSafetyMode(const std::string &params_string);
 
   bool initialized_ = false;
+  bool log_once_ = false;
   bool safety_configured_ = false;
   bool prev_obd_multiplexing_ = false;
   std::vector<Panda *> pandas_;