summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/audio/fatfs_source.cpp4
-rw-r--r--src/system_fsm/include/system_events.hpp5
-rw-r--r--src/system_fsm/include/system_fsm.hpp1
-rw-r--r--src/system_fsm/running.cpp8
4 files changed, 16 insertions, 2 deletions
diff --git a/src/audio/fatfs_source.cpp b/src/audio/fatfs_source.cpp
index 4fc5adbb..72c3940d 100644
--- a/src/audio/fatfs_source.cpp
+++ b/src/audio/fatfs_source.cpp
@@ -12,11 +12,13 @@
#include <memory>
#include "esp_log.h"
+#include "event_queue.hpp"
#include "ff.h"
#include "audio_source.hpp"
#include "codec.hpp"
#include "spi.hpp"
+#include "system_events.hpp"
#include "types.hpp"
namespace audio {
@@ -39,7 +41,7 @@ auto FatfsSource::Read(cpp::span<std::byte> dest) -> ssize_t {
UINT bytes_read = 0;
FRESULT res = f_read(file_.get(), dest.data(), dest.size(), &bytes_read);
if (res != FR_OK) {
- ESP_LOGE(kTag, "error reading from file");
+ events::System().Dispatch(system_fsm::StorageError{.error = res});
return -1;
}
return bytes_read;
diff --git a/src/system_fsm/include/system_events.hpp b/src/system_fsm/include/system_events.hpp
index f9ab9e11..22e3b6bd 100644
--- a/src/system_fsm/include/system_events.hpp
+++ b/src/system_fsm/include/system_events.hpp
@@ -11,6 +11,7 @@
#include "battery.hpp"
#include "bluetooth_types.hpp"
#include "database.hpp"
+#include "ff.h"
#include "haptics.hpp"
#include "samd.hpp"
#include "service_locator.hpp"
@@ -42,7 +43,9 @@ struct OnIdle : tinyfsm::Event {};
*/
struct StorageMounted : tinyfsm::Event {};
-struct StorageError : tinyfsm::Event {};
+struct StorageError : tinyfsm::Event {
+ FRESULT error;
+};
struct KeyLockChanged : tinyfsm::Event {
bool locking;
diff --git a/src/system_fsm/include/system_fsm.hpp b/src/system_fsm/include/system_fsm.hpp
index a129829e..e0a0ac7a 100644
--- a/src/system_fsm/include/system_fsm.hpp
+++ b/src/system_fsm/include/system_fsm.hpp
@@ -105,6 +105,7 @@ class Running : public SystemState {
void react(const database::event::UpdateFinished&) override;
void react(const SamdUsbMscChanged&) override;
void react(const internal::UnmountTimeout&) override;
+ void react(const StorageError&) override;
using SystemState::react;
diff --git a/src/system_fsm/running.cpp b/src/system_fsm/running.cpp
index 8625ac66..796c96dc 100644
--- a/src/system_fsm/running.cpp
+++ b/src/system_fsm/running.cpp
@@ -8,6 +8,7 @@
#include "audio_events.hpp"
#include "database.hpp"
#include "db_events.hpp"
+#include "ff.h"
#include "file_gatherer.hpp"
#include "freertos/portmacro.h"
#include "freertos/projdefs.h"
@@ -119,6 +120,13 @@ void Running::react(const SamdUsbMscChanged& ev) {
}
}
+void Running::react(const StorageError& ev) {
+ ESP_LOGE(kTag, "storage error %u", ev.error);
+ if (ev.error == FR_DISK_ERR || ev.error == FR_INVALID_OBJECT) {
+ unmountStorage();
+ }
+}
+
auto Running::checkIdle() -> void {
xTimerStop(sUnmountTimer, portMAX_DELAY);
if (IdleCondition()) {