More small cleanups in prep for BLE work

Author: davidmweber

README.md

@@ -19,7 +19,7 @@ I started the project using the Espressif IDF (a powerful tool indeed) but switc
   ecosystem.
 - I chose the `#[no_std]` (i.e. no Espressif IDF) option, mostly to feel out the ecosystem in a pure Rust world. This
   is an art piece after all.
-- Async has some significant advantages, particularly with stack management and efficient concurrency so I have
+- Async has some significant advantages, particularly with stack management and efficient concurrency, so I have
   made some effort to use async interfaces wherever possible.
 
 ## Hardware Requirements
@@ -53,7 +53,6 @@ compilers for your targeted hardware. The tools you will need are:
   Wokwi setup in the repo. You will need to get an account and set up your IDE to use it. There are plugins for VSCode
   and Jetbrains.
 
-
 ## Building and running
 Builds are mostly managed by cargo, but we use the awesome [just](https://github.com/casey/just) tool to automate
 some of the builds. Running `just --list` will show all the available tasks.
@@ -66,8 +65,12 @@ some of the builds. Running `just --list` will show all the available tasks.
 - [Rust on ESP book](https://docs.esp-rs.org/book/)
 - 
 
-
 # TODO
 
 - [ ] Set up hardware targets for the ESP32-S3 and ESP32-H2 targets
-- [ ] Fix the Wokwi emulator
+- [ ] Fix the Wokwi emulator
+
+
+# Learnings
+ - The Rust embedded ecosystem is potent but immature. That being said, it is actually really nice to work with and
+   is rapidly evolving.

src/display_task.rs

@@ -10,14 +10,19 @@ use smart_leds::RGB8;
 /// this is one of the many reasons
 pub enum DisplayState {
     /// Suspends animation update
+    #[allow(unused)]
     Stop,
     /// Restart animation update
+    #[allow(unused)]
     Start,
     /// Set the pixel colour. It is always the 1st pixel. Boring but....
+    #[allow(unused)]
     Colour(RGB8),
     /// Switch of all the LEDs
+    #[allow(unused)]
     Off,
-    /// Sets the led to torch mode. This disables the animation
+    /// Sets the LED to torch mode. This disables the animation
+    #[allow(unused)]
     Torch(u8),
 }
 

src/main.rs

@@ -80,11 +80,10 @@ async fn main(spawner: Spawner) {
     let display_channel = DISPLAY_CHANNEL.init(Channel::new());
     let led_driver: &'static mut LedDriver =
         LED_DRIVER.init(LedDriver::new(peripherals.RMT, peripherals.GPIO6));
-
+    // Start the display manager task
     spawner
         .spawn(display_task(display_channel, led_driver))
         .expect("Failed to spawn display task");
-    Timer::after(Duration::from_secs(1)).await;
 
     // Simple example that exercises the display task
     loop {