From 126d55ed0654313a0e30341148c7478b4feec49be260b9e6ed7c19fa1315ba3b Mon Sep 17 00:00:00 2001
From: localhorst <localhorst@mosad.xyz>
Date: Sun, 9 Nov 2025 17:11:54 +0100
Subject: [PATCH 1/3] Sleep during delay instead of busy-wait

---
 main.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/main.c b/main.c
index e77d1f8..ad8f14a 100644
--- a/main.c
+++ b/main.c
@@ -136,7 +136,10 @@ int main(void)
             }
         }
 
+        // Sleep during delay instead of busy-wait
+        sleep_enable();
         _delay_ms(MAIN_LOOP_SLEEP);
+        sleep_disable();
     }
 }
 

From 39b2eb11e292676458a0d0279eab7b3aaac711f82997610a24ba8e6a7b44ec6e Mon Sep 17 00:00:00 2001
From: localhorst <localhorst@mosad.xyz>
Date: Sun, 9 Nov 2025 17:18:43 +0100
Subject: [PATCH 2/3] cleanup

---
 main.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/main.c b/main.c
index ad8f14a..6084781 100644
--- a/main.c
+++ b/main.c
@@ -24,6 +24,8 @@
 #define BUTTON_IGNORE_DURATION_MS 2000U     // Time button ignored after long press
 #define BUTTON_CONFIRMATION_BLINK_LOOPS 10U // Blink animation for confirmation
 #define BUTTON_CONFIRMATION_BLINK_DURATION_MS 50U
+#define GLOW_BRIGHTNESS_MIN 10U
+#define GLOW_BRIGHTNESS_MAX 100U
 
 /** Convert milliseconds to system ticks */
 #define MS_TO_TICKS(ms) ((ms) / MAIN_LOOP_SLEEP)
@@ -111,7 +113,6 @@ int main(void)
             set_sleep_mode(SLEEP_MODE_STANDBY); // Deepest sleep mode (standby)
             sleep_enable();                     // Enable sleep
 
-            cli();       // Disable global interrupts
             sei();       // Re-enable interrupts
             sleep_cpu(); // MCU sleeps here
         }
@@ -428,14 +429,14 @@ void ledAnimationGlow(void)
     brightness += direction;
 
     // Reverse direction at limits
-    if (brightness >= 100U)
+    if (brightness >= GLOW_BRIGHTNESS_MAX)
     {
-        brightness = 100U;
+        brightness = GLOW_BRIGHTNESS_MAX;
         direction = -1;
     }
-    else if (brightness == 10U)
+    else if (brightness == GLOW_BRIGHTNESS_MIN)
     {
-        brightness = 10U;
+        brightness = GLOW_BRIGHTNESS_MIN;
         direction = 1;
     }
 

From 3214f658137768237c1651a83c65facaed484a2ebbabcda38d34aba654d71d33 Mon Sep 17 00:00:00 2001
From: localhorst <localhorst@mosad.xyz>
Date: Sun, 9 Nov 2025 17:33:26 +0100
Subject: [PATCH 3/3] Disable unused peripherals for power saving

---
 main.c | 53 +++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 53 insertions(+)

diff --git a/main.c b/main.c
index 6084781..d7f6aee 100644
--- a/main.c
+++ b/main.c
@@ -46,6 +46,7 @@ volatile eMode eModeCurrent = ANIMATION_BLINK;
 // Forward declarations
 ISR(PORTA_PORT_vect);
 static void software_reset(void);
+static void configureLowPower(void);
 void initPWM(void);
 void setPWM_PA2(uint8_t duty);
 static inline void leds_off(void);
@@ -62,6 +63,10 @@ void ledStaticFull(void);
  */
 int main(void)
 {
+
+    // Disable unused peripherals for power saving
+    configureLowPower();
+
     // Configure LED pins as outputs
     VPORTA.DIR = (PA1_SET_MASK | PA2_SET_MASK | PA3_SET_MASK | PA6_SET_MASK | PA7_SET_MASK);
     initPWM();
@@ -152,6 +157,54 @@ static inline void switchMode(void)
     eModeCurrent = (eModeCurrent + 1) % MAX_COUNT;
 }
 
+/**
+ * @brief Configure for lowest power consumption
+ */
+static void configureLowPower(void)
+{
+    // Set unused pins as outputs LOW to prevent floating inputs
+    // Floating inputs can cause extra current consumption
+    PORTA.DIRSET = PIN4_bm | PIN5_bm; // Set PA4, PA5 as outputs if unused
+    PORTA.OUTCLR = PIN4_bm | PIN5_bm; // Drive them LOW
+
+    // Configure sleep mode
+    set_sleep_mode(SLEEP_MODE_IDLE); // Use IDLE when TCA0 PWM needs to run
+    // Use SLEEP_MODE_STANDBY for deep sleep when LEDs are off
+
+    // Disable unused timers
+    TCB0.CTRLA = 0; // Disable TCB0 if not used
+    // TCA0 is used for PWM, keep it enabled
+
+    // Disable ADC (Analog-to-Digital Converter)
+    ADC0.CTRLA &= ~ADC_ENABLE_bm;
+
+    // Disable AC (Analog Comparator)
+    AC0.CTRLA &= ~AC_ENABLE_bm;
+
+    // Disable unused USART
+    USART0.CTRLB = 0;
+
+    // Disable TWI (I2C) if not used
+    TWI0.MCTRLA = 0;
+    TWI0.SCTRLA = 0;
+
+    // Disable SPI
+    SPI0.CTRLA = 0;
+
+    // Disable Watchdog Timer (if not needed)
+    // Note: WDT can only be disabled during first 4 clock cycles after reset
+    // CCP = CCP_IOREG_gc;
+    // WDT.CTRLA = 0;
+
+    // Disable BOD (Brown-Out Detection) in sleep modes for lower power
+    // This is done via fuses, not runtime configurable
+
+    // Disable digital input buffers on unused pins to save power
+    // Only needed if pins are truly unused (floating)
+    // PORTA.PIN4CTRL = PORT_ISC_INPUT_DISABLE_gc;  // PA4 if unused
+    // PORTA.PIN5CTRL = PORT_ISC_INPUT_DISABLE_gc;  // PA5 if unused
+}
+
 /**
  * @brief Init PWM for PA2
  */