Lift storage reset into the storage base class in preparation for bei…

…ng able to do something smarter on platforms with a filesystem

Signed-off-by: Jesse Vincent <[email protected]>

plugins/Kaleidoscope-EEPROM-Settings/src/kaleidoscope/plugin/EEPROM-Settings.cpp

@@ -231,10 +231,8 @@ EventHandlerResult FocusEEPROMCommand::onFocusEvent(const char *input) {
   } else if (::Focus.inputMatchesCommand(input, cmd_free)) {
     ::Focus.send(Runtime.storage().length() - ::EEPROMSettings.used());
   } else if (::Focus.inputMatchesCommand(input, cmd_erase)) {
-    for (uint16_t i = 0; i < Runtime.storage().length(); i++) {
-      Runtime.storage().update(i, EEPROMSettings::EEPROM_UNINITIALIZED_BYTE);
-    }
-    Runtime.storage().commit();
+
+    Runtime.storage().erase();
     Runtime.device().rebootBootloader();
   } else {
     return EventHandlerResult::OK;

src/kaleidoscope/driver/storage/Base.h

@@ -59,6 +59,13 @@ class Base {
 
   void setup() {}
   void commit() {}
+
+  void erase() {
+    for (uint16_t i = 0; i < length(); i++) {
+      update(i, _StorageProps::uninitialized_byte);
+    }
+    commit();
+  }
 };
 
 }  // namespace storage

src/kaleidoscope/driver/storage/GD32Flash.h

@@ -46,6 +46,12 @@ class GD32Flash : public EEPROMClass<_StorageProps::length> {
     }
     return true;
   }
+  void erase() {
+    for (uint16_t i = 0; i < this->length(); i++) {
+      this->update(i, _StorageProps::uninitialized_byte);
+    }
+    this->commit();
+  }
 };
 
 }  // namespace storage