Revert "Fix CommandHandler semaphore initialization (#135)"

This reverts commit e5ca8bf.
OpenShock · Jan 22, 2024 · e0bda5f · e0bda5f
1 parent 62e855c
commit e0bda5f
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/src/CommandHandler.cpp b/src/CommandHandler.cpp
@@ -157,12 +157,6 @@ bool CommandHandler::Init() {
     return true;
   }
 
-  // Initialize semaphores
-  s_rfTransmitterSemaphore = xSemaphoreCreateBinary();
-  xSemaphoreGive(s_rfTransmitterSemaphore);
-  s_keepAliveSemaphore = xSemaphoreCreateBinary();
-  xSemaphoreGive(s_keepAliveSemaphore);
-
   auto& rfConfig = Config::GetRFConfig();
 
   std::uint8_t txPin = rfConfig.txPin;
@@ -172,13 +166,17 @@ bool CommandHandler::Init() {
     return false;
   }
 
+  s_rfTransmitterSemaphore = xSemaphoreCreateBinary();
+  xSemaphoreGive(s_rfTransmitterSemaphore);
   s_rfTransmitter = std::make_unique<RFTransmitter>(txPin, 32);
   if (!s_rfTransmitter->ok()) {
     ESP_LOGE(TAG, "Failed to initialize RF Transmitter");
     s_rfTransmitter = nullptr;
     return false;
   }
 
+  s_keepAliveSemaphore = xSemaphoreCreateBinary();
+  xSemaphoreGive(s_keepAliveSemaphore);
   if (rfConfig.keepAliveEnabled) {
     _internalSetKeepAliveEnabled(true);
   }