From 3b5363fea188f65f7a7d46284d5d06369b0adc35 Mon Sep 17 00:00:00 2001 From: Gergely Nagy Date: Mon, 5 Jun 2017 09:11:46 +0200 Subject: [PATCH] bootAnimation: Just do it A while ago, we added a bit of code to `bootAnimation()` that only did the animation on first boot - this is not a desirable thing anymore, not in `bootAnimation()` itself. These days, one would use `Kaleidoscope-EEPROM-Settings`, and decide whether to do the boot animation there. Since `bootAnimation()` is an optional thing, just do the animation whenever the function is called, and remove the obsolete EEPROM bits. Signed-off-by: Gergely Nagy --- src/BootAnimation.cpp | 8 -------- 1 file changed, 8 deletions(-) diff --git a/src/BootAnimation.cpp b/src/BootAnimation.cpp index af04676d..be4d9cd4 100644 --- a/src/BootAnimation.cpp +++ b/src/BootAnimation.cpp @@ -1,8 +1,5 @@ #include "BootAnimation.h" #include "Kaleidoscope-LEDControl.h" -#include "EEPROM.h" - -#define EEPROM_BOOT_ANIMATION_LOCATION 1 #ifdef ARDUINO_AVR_MODEL01 static void @@ -19,9 +16,6 @@ type_letter(uint8_t letter) { void bootAnimation(void) { #ifdef ARDUINO_AVR_MODEL01 - if (EEPROM.read(EEPROM_BOOT_ANIMATION_LOCATION)) - return; - LEDControl.set_all_leds_to(0, 0, 0); type_letter(LED_K); type_letter(LED_E); @@ -37,7 +31,5 @@ bootAnimation(void) { type_letter(LED_0); type_letter(LED_PERIOD); type_letter(LED_9); - - EEPROM.update(EEPROM_BOOT_ANIMATION_LOCATION, 1); #endif }