From f7b8586424b92affe24ae2d91e1ab4e89036cb43 Mon Sep 17 00:00:00 2001
From: Reed Krantz <programming@krantz.one>
Date: Mon, 17 Aug 2026 21:41:01 -0500
Subject: [PATCH] refactor: move storage initialization to the start_config
 function

---
 src/config.rs | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/config.rs b/src/config.rs
index 0fffdd5..f7c70b3 100644
--- a/src/config.rs
+++ b/src/config.rs
@@ -6,7 +6,7 @@ use esp_hal::peripherals::FLASH;
 use esp_storage::FlashStorage;
 use int_enum::IntEnum;
 use sequential_storage::{
-    cache::Cache,
+    cache::{Cache, Uncached},
     map::{Key, MapConfig, MapStorage, SerializationError},
 };
 
@@ -21,18 +21,18 @@ impl ConfigCommand {
 pub static CONFIG_SERVICE: RpcService<CriticalSectionRawMutex, ConfigCommand, ()> =
     RpcService::new();
 
-pub fn start_config(spawner: Spawner, flash: FLASH<'static>) {
-    spawner.spawn(config_handler(flash).unwrap());
-}
-
-#[embassy_executor::task]
-async fn config_handler(flash: FLASH<'static>) {
-    let mut storage = MapStorage::<ConfigKey, _, _>::new(
+pub async fn start_config(spawner: Spawner, flash: FLASH<'static>) {
+    let storage = MapStorage::<ConfigKey, _, _>::new(
         BlockingAsync::new(FlashStorage::new(flash)),
         const { MapConfig::new(0x3F_0000..0x40_0000) },
         Cache::new_uncached(),
     );
 
+    spawner.spawn(config_handler(storage).unwrap());
+}
+
+#[embassy_executor::task]
+async fn config_handler(mut storage: MapStorage<ConfigKey, BlockingAsync<FlashStorage<'static>>, Cache<Uncached, Uncached, Uncached, ConfigKey>>) {
     let mut data_buffer = [0u8; 256];
 
     loop {