[preferences] Shorten log messages (#8982)

This commit is contained in:
Keith Burzinski 2025-05-31 14:23:47 -05:00 committed by GitHub
parent 9ba9674437
commit 5771bb4907
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
4 changed files with 17 additions and 18 deletions

View File

@ -111,7 +111,7 @@ class ESP32Preferences : public ESPPreferences {
if (s_pending_save.empty())
return true;
ESP_LOGD(TAG, "Saving %d preferences to flash...", s_pending_save.size());
ESP_LOGV(TAG, "Saving %d items...", s_pending_save.size());
// goal try write all pending saves even if one fails
int cached = 0, written = 0, failed = 0;
esp_err_t last_err = ESP_OK;
@ -139,10 +139,10 @@ class ESP32Preferences : public ESPPreferences {
}
s_pending_save.erase(s_pending_save.begin() + i);
}
ESP_LOGD(TAG, "Saving %d preferences to flash: %d cached, %d written, %d failed", cached + written + failed, cached,
written, failed);
ESP_LOGD(TAG, "Writing %d items: %d cached, %d written, %d failed", cached + written + failed, cached, written,
failed);
if (failed > 0) {
ESP_LOGE(TAG, "Error saving %d preferences to flash. Last error=%s for key=%s", failed, esp_err_to_name(last_err),
ESP_LOGE(TAG, "Writing %d items failed. Last error=%s for key=%s", failed, esp_err_to_name(last_err),
last_key.c_str());
}
@ -173,7 +173,7 @@ class ESP32Preferences : public ESPPreferences {
}
bool reset() override {
ESP_LOGD(TAG, "Cleaning up preferences in flash...");
ESP_LOGD(TAG, "Erasing storage");
s_pending_save.clear();
nvs_flash_deinit();

View File

@ -235,7 +235,7 @@ class ESP8266Preferences : public ESPPreferences {
if (s_prevent_write)
return false;
ESP_LOGD(TAG, "Saving preferences to flash...");
ESP_LOGD(TAG, "Saving");
SpiFlashOpResult erase_res, write_res = SPI_FLASH_RESULT_OK;
{
InterruptLock lock;
@ -245,11 +245,11 @@ class ESP8266Preferences : public ESPPreferences {
}
}
if (erase_res != SPI_FLASH_RESULT_OK) {
ESP_LOGE(TAG, "Erase ESP8266 flash failed!");
ESP_LOGE(TAG, "Erasing failed");
return false;
}
if (write_res != SPI_FLASH_RESULT_OK) {
ESP_LOGE(TAG, "Write ESP8266 flash failed!");
ESP_LOGE(TAG, "Writing failed");
return false;
}
@ -258,14 +258,14 @@ class ESP8266Preferences : public ESPPreferences {
}
bool reset() override {
ESP_LOGD(TAG, "Cleaning up preferences in flash...");
ESP_LOGD(TAG, "Erasing storage");
SpiFlashOpResult erase_res;
{
InterruptLock lock;
erase_res = spi_flash_erase_sector(get_esp8266_flash_sector());
}
if (erase_res != SPI_FLASH_RESULT_OK) {
ESP_LOGE(TAG, "Erase ESP8266 flash failed!");
ESP_LOGE(TAG, "Erasing failed");
return false;
}

View File

@ -101,7 +101,7 @@ class LibreTinyPreferences : public ESPPreferences {
if (s_pending_save.empty())
return true;
ESP_LOGD(TAG, "Saving %d preferences to flash...", s_pending_save.size());
ESP_LOGV(TAG, "Saving %d items...", s_pending_save.size());
// goal try write all pending saves even if one fails
int cached = 0, written = 0, failed = 0;
fdb_err_t last_err = FDB_NO_ERR;
@ -129,11 +129,10 @@ class LibreTinyPreferences : public ESPPreferences {
}
s_pending_save.erase(s_pending_save.begin() + i);
}
ESP_LOGD(TAG, "Saving %d preferences to flash: %d cached, %d written, %d failed", cached + written + failed, cached,
written, failed);
ESP_LOGD(TAG, "Writing %d items: %d cached, %d written, %d failed", cached + written + failed, cached, written,
failed);
if (failed > 0) {
ESP_LOGE(TAG, "Error saving %d preferences to flash. Last error=%d for key=%s", failed, last_err,
last_key.c_str());
ESP_LOGE(TAG, "Writing %d items failed. Last error=%d for key=%s", failed, last_err, last_key.c_str());
}
return failed == 0;
@ -158,7 +157,7 @@ class LibreTinyPreferences : public ESPPreferences {
}
bool reset() override {
ESP_LOGD(TAG, "Cleaning up preferences in flash...");
ESP_LOGD(TAG, "Erasing storage");
s_pending_save.clear();
fdb_kv_set_default(&db);

View File

@ -114,7 +114,7 @@ class RP2040Preferences : public ESPPreferences {
if (s_prevent_write)
return false;
ESP_LOGD(TAG, "Saving preferences to flash...");
ESP_LOGD(TAG, "Saving");
{
InterruptLock lock;
@ -129,7 +129,7 @@ class RP2040Preferences : public ESPPreferences {
}
bool reset() override {
ESP_LOGD(TAG, "Cleaning up preferences in flash...");
ESP_LOGD(TAG, "Erasing storage");
{
InterruptLock lock;
::rp2040.idleOtherCore();