From deaa9ee60b601b099584cb56e498cde4f43da908 Mon Sep 17 00:00:00 2001 From: Oliver Kleinecke Date: Wed, 19 Feb 2025 20:52:16 +0100 Subject: [PATCH] fix: increase timer capacity from 64 to 256 in DynamicLampComponent --- .../components/dynamic_lamp/dynamic_lamp.cpp | 18 +++++++++--------- esphome/components/dynamic_lamp/dynamic_lamp.h | 2 +- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/esphome/components/dynamic_lamp/dynamic_lamp.cpp b/esphome/components/dynamic_lamp/dynamic_lamp.cpp index c1ba4e5dc6..b93068dec1 100644 --- a/esphome/components/dynamic_lamp/dynamic_lamp.cpp +++ b/esphome/components/dynamic_lamp/dynamic_lamp.cpp @@ -259,13 +259,13 @@ bool DynamicLampComponent::add_timer(std::string timer_desc, std::string lamp_li new_timer.timer_desc, lamp_list_str.c_str(), new_timer.active, new_timer.action, new_timer.hour, new_timer.minute, new_timer.monday, new_timer.tuesday, new_timer.wednesday, new_timer.thursday, new_timer.friday, new_timer.saturday, new_timer.sunday); uint8_t save_slot; - for (save_slot = 0; save_slot < 64; save_slot++) { + for (save_slot = 0; save_slot < 256; save_slot++) { if (this->timers_[save_slot].in_use != true) { break; } } - if (save_slot == 64) { - ESP_LOGW(TAG, "No more timer slots available, max 64 timers supported!"); + if (save_slot == 256) { + ESP_LOGW(TAG, "No more timer slots available, max 256 timers supported!"); this->status_set_warning(); return false; } @@ -299,7 +299,7 @@ std::vector DynamicLampComponent::build_lamp_list_from_list_str_(std::stri void DynamicLampComponent::read_fram_timers_to_log() { DynamicLampTimer timer; - for (uint8_t i = 0; i < 64; i++) { + for (uint8_t i = 0; i < 256; i++) { this->fram_->read((0x4000 + (i * 64)), reinterpret_cast(&timer), 64); if (timer.validation_bytes[0] == 'V' && timer.validation_bytes[1] == 'D' && timer.validation_bytes[2] == 'L' && timer.validation_bytes[3] == 'T' && timer.in_use == true) { std::string lamp_names_str = ""; @@ -329,7 +329,7 @@ void DynamicLampComponent::read_fram_timers_to_log() { void DynamicLampComponent::read_initialized_timers_to_log() { DynamicLampTimer timer; - for (uint8_t i = 0; i < 64; i++) { + for (uint8_t i = 0; i < 256; i++) { if (this->timers_[i].in_use == true) { timer = this->timers_[i]; std::string lamp_names_str = ""; @@ -397,7 +397,7 @@ void DynamicLampComponent::restore_lamp_settings_() { void DynamicLampComponent::restore_timers_() { switch (this->save_mode_) { case SupportedSaveModes::SAVE_MODE_NONE: - for (uint8_t i = 0; i < 64; i++) { + for (uint8_t i = 0; i < 256; i++) { this->timers_[i] = DynamicLampTimer(); this->timers_[i].in_use = false; } @@ -409,20 +409,20 @@ void DynamicLampComponent::restore_timers_() { break; case SupportedSaveModes::SAVE_MODE_FRAM: DynamicLampTimer timer; - for (uint8_t i = 0; i < 64; i++) { + for (uint8_t i = 0; i < 256; i++) { timer = DynamicLampTimer(); this->timers_[i] = timer; this->timers_[i].in_use = false; } std::string lamp_names_str; - for (uint8_t i = 0; i < 64; i++) { + for (uint8_t i = 0; i < 256; i++) { this->fram_->read((0x4000 + (i * 64)), reinterpret_cast(&timer), 64); if (timer.validation_bytes[0] == 'V' && timer.validation_bytes[1] == 'D' && timer.validation_bytes[2] == 'L' && timer.validation_bytes[3] == 'T' && timer.in_use == true) { this->timers_[i] = timer; } else { this->timers_[i] = DynamicLampTimer(); this->timers_[i].in_use = false; - ESP_LOGVV(TAG, "Timer save slot %" PRIu8 " did not contain valid record, initializing unused empty timer slot", i); + //ESP_LOGVV(TAG, "Timer save slot %" PRIu8 " did not contain valid record, initializing unused empty timer slot", i); } } break; diff --git a/esphome/components/dynamic_lamp/dynamic_lamp.h b/esphome/components/dynamic_lamp/dynamic_lamp.h index c0ac4f5253..7db941890d 100644 --- a/esphome/components/dynamic_lamp/dynamic_lamp.h +++ b/esphome/components/dynamic_lamp/dynamic_lamp.h @@ -128,7 +128,7 @@ class DynamicLampComponent : public Component { CombinedLamp active_lamps_[16]; LinkedOutput available_outputs_[16]; - DynamicLampTimer timers_[64]; + DynamicLampTimer timers_[256]; uint8_t save_mode_; uint8_t lamp_count_ = 0; };