simply wait loop logic

rust/src/main.rs

@@ -932,11 +932,10 @@ fn wait_infinity(wait_type: WaitType, reboot_now: Arc<AtomicBool>) -> ! {
 
     let mut led_count = 8;
     loop {
+        // TODO implement actually different blink patterns instead of modulating blink duration
         if wait_type == WaitType::MissingConfig {
+            led_count %= 8;
             led_count += 1;
-            if led_count > 8 {
-                led_count = 1;
-            }
         };
         unsafe {
             //do not trigger watchdog
@@ -946,19 +945,16 @@ fn wait_infinity(wait_type: WaitType, reboot_now: Arc<AtomicBool>) -> ! {
             BOARD_ACCESS.lock().unwrap().general_fault(true);
             vTaskDelay(delay);
             BOARD_ACCESS.lock().unwrap().general_fault(false);
+            //TODO move locking outside of loop and drop afterwards
             for i in 0..8 {
                 BOARD_ACCESS.lock().unwrap().fault(i, false);
             }
             vTaskDelay(delay);
-            match wait_type {
+            if wait_type == WaitType::MqttConfig {
-                WaitType::MissingConfig => {}
-                WaitType::ConfigButton => {}
-                WaitType::MqttConfig => {
                 if !STAY_ALIVE.load(std::sync::atomic::Ordering::Relaxed) {
                     reboot_now.store(true, std::sync::atomic::Ordering::Relaxed);
                 }
             }
-            }
             if reboot_now.load(std::sync::atomic::Ordering::Relaxed) {
                 //ensure clean http answer
                 Delay::new_default().delay_ms(500);