Improve some comments

Author: jannic

neotron-bmc-pico/src/main.rs

@@ -471,7 +471,8 @@ mod app {
 						ctx.shared.pin_sys_reset.lock(|pin| pin.set_low().unwrap());
 						// Step 4 - Turn on PSU
 						ctx.shared.pin_dc_on.set_high().unwrap();
-						// Step 5 happens below when power is good
+						// Taking the system out of reset and enabling the IRQ line happens
+						// later, when the power rail is settled
 						defmt::info!("Waiting for power-good");
 					}
 				}
@@ -590,7 +591,7 @@ mod app {
 			}
 			// TODO: Also monitor 5.0V rail
 
-			// Wait for power good
+			// Wait for power-good
 			if ctx.shared.state_dc_power_enabled.lock(|r| *r) == DcPowerState::Starting {
 				let mon_3v3 = ctx.local.adc.read_abs_mv(ctx.local.pin_3v3_monitor);
 				defmt::trace!("3v3 reading: {}", mon_3v3);
@@ -607,13 +608,10 @@ mod app {
 					ctx.shared
 						.state_dc_power_enabled
 						.lock(|r| *r = DcPowerState::On);
-					// Steps 1 - 4 happened above, in the button press handler
-					// Step 5 - Leave it in reset for a while.
-					// TODO: Start monitoring 3.3V and 5.0V rails here
-					// TODO: Take system out of reset when 3.3V and 5.0V are good
+					// Wait a bit before taking system out of reset.
 					// Returns an error if it's already scheduled (but we don't care)
 					let _ = exit_reset::spawn_after(RESET_DURATION_MS.millis());
-					// Set 6 - unmask the IRQ
+					// Unmask the IRQ
 					irq_forced_low = false;
 				}
 			}