Do not log an error when NVS namespace is missing on read

kivancsikert · Dec 13, 2024 · decd29d · decd29d
1 parent 3ec45f2
commit decd29d
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/main/kernel/NvsStore.hpp b/main/kernel/NvsStore.hpp
@@ -133,9 +133,18 @@ class NvsStore {
 
         nvs_handle_t handle;
         esp_err_t err = nvs_open(name.c_str(), readOnly ? NVS_READONLY : NVS_READWRITE, &handle);
-        if (err != ESP_OK) {
-            LOGTE("nvs", "failed to %s '%s'", readOnly ? "read" : "write", name.c_str());
-            return false;
+        switch (err) {
+            case ESP_OK:
+                break;
+            case ESP_ERR_NVS_NOT_FOUND:
+                LOGTV("nvs", "namespace '%s' does not exist yet, nothing to read",
+                    name.c_str());
+                return ESP_ERR_NOT_FOUND;
+                break;
+            default:
+                LOGTW("nvs", "failed to open NVS to %s '%s': %s",
+                    readOnly ? "read" : "write", name.c_str(), esp_err_to_name(err));
+                break;
         }
 
         esp_err_t result = action(handle);