Merge branch 'main' into feature/energy-saving

main.c

@@ -1,7 +1,6 @@
 /**
  * @file main.c
- * @brief Bike rear light implementation for ATTINY202.
+ * @brief Bike rear light implementation for ATTINY202 with low-power standby.
- *
  */
 
 #include <stdint.h>
@@ -18,72 +17,70 @@
 #define PA6_SET_MASK 0x40    ///< Green LED pin
 #define PA7_SET_MASK 0x80    ///< Red LED
 
-#define MAIN_LOOP_SLEEP 10U                 // Main loop delay in ms
+#define MAIN_LOOP_SLEEP 50U                 // Loop period in ms
 #define BUTTON_LONG_PRESS_DURATION_MS 1000U // Long press threshold
-#define BUTTON_SHORT_PRESS_DURATION_MS 50U  // Short press threshold
 #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)
 
-typedef enum _Mode
-{
-    ANIMATION_BLINK,
-    ANIMATION_GLOW,
-    STATIC_FULL,
-    MAX_COUNT,
-} eMode;
-
 /** Global flags */
 volatile bool bLedEnabled = true;
 volatile bool bBtnPressed = false;
-volatile eMode eModeCurrent = ANIMATION_BLINK;
 
 // Forward declarations
-ISR(PORTA_PORT_vect);
+void blinkLed(bool resetCounters);
-static void software_reset(void);
-static void configureLowPower(void);
-void initPWM(void);
-void setPWM_PA2(uint8_t duty);
 static inline void leds_off(void);
 static inline void leds_on(void);
 static void battery_level_indicator(void);
-static bool handleSwitch(void);
+static void handleSwitch(void);
-static inline void switchMode(void);
+static void software_reset(void);
-void ledAnimationBlink(bool resetCounters);
+ISR(PORTA_PORT_vect);
-void ledAnimationGlow(void);
-void ledStaticFull(void);
 
-/**
+/* --- Timer init: Use RTC PIT for periodic wake-up --- */
- * @brief Main entry point
+void init_timer(void)
- */
+{
+    RTC.CLKSEL = RTC_CLKSEL_INT1K_gc; // 1 kHz ULP clock for RTC
+    while (RTC.STATUS > 0)
+    {
+    } // Wait for sync
+
+    RTC.PITINTCTRL = RTC_PI_bm;        // Enable PIT interrupt
+    RTC.PITCTRLA = RTC_PERIOD_CYC64_gc // ≈64 ms wake-up (~50 ms)
+                   | RTC_PITEN_bm;     // Enable PIT
+}
+
+ISR(RTC_PIT_vect)
+{
+    RTC.PITINTFLAGS = RTC_PI_bm; // Clear interrupt flag
+}
+
+/* --- MAIN --- */
 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();
 
     // Configure PA0 as input with pull-up
     VPORTA.DIR &= ~BUTTON_PIN_MASK;    // Input
     PORTA.PIN0CTRL = PORT_PULLUPEN_bm; // Pull-up enabled
 
-    leds_off();                // Ensure all LEDs off at startup
+    // Ensure all LEDs off at startup
+    leds_off();
     battery_level_indicator(); // TODO: Implement
 
     bool bLedEnabledOld = bLedEnabled;
-    eModeCurrent = ANIMATION_BLINK; // Set the mode to start with
 
-    while (true)
+    cli();
+    init_timer();
+    sei();
+
+    set_sleep_mode(SLEEP_MODE_STANDBY);
+
+    while (1)
     {
-        bBtnPressed = handleSwitch(); // Check switch state
+        handleSwitch(); // Check switch state
 
         // Light LEDs while button is pressed
         if (bBtnPressed)
@@ -118,6 +115,7 @@ 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
         }
@@ -125,118 +123,22 @@ int main(void)
         {
             if (bLedEnabled && !bBtnPressed)
             {
-                switch (eModeCurrent)
+                blinkLed(false); // run normal blink
-                {
-                case ANIMATION_BLINK:
-                    ledAnimationBlink(false); // run normal blink
-                    break;
-                case ANIMATION_GLOW:
-                    ledAnimationGlow();
-                    break;
-                case STATIC_FULL:
-                    ledStaticFull();
-                    break;
-                default:
-                    break;
-                }
             }
         }
 
-        // Sleep during delay instead of busy-wait
         sleep_enable();
-        _delay_ms(MAIN_LOOP_SLEEP);
+        sleep_cpu(); // Sleep until PIT wakes
         sleep_disable();
     }
 }
 
-/**
- * @brief Move to next mode
- */
-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
- */
-void initPWM(void)
-{
-    // No PORTMUX needed - PA2 is WO2 by default
-
-    // TCA0 in normal mode (single slope PWM)
-    TCA0.SINGLE.CTRLB = TCA_SINGLE_CMP2EN_bm | TCA_SINGLE_WGMODE_SINGLESLOPE_gc;
-
-    // Set period for ~19.5kHz PWM (5MHz / 256 = ~19.5kHz)
-    TCA0.SINGLE.PER = 0xFF;
-
-    // Start timer with DIV1 (no prescaler)
-    TCA0.SINGLE.CTRLA = TCA_SINGLE_CLKSEL_DIV1_gc | TCA_SINGLE_ENABLE_bm;
-}
-
-/**
- * @brief Set PWM duty cycle for PA2
- */
-void setPWM_PA2(uint8_t duty)
-{
-    TCA0.SINGLE.CMP2 = duty;
-}
-
 /**
  * @brief Turn off all controlled LEDs (PA1, PA2, PA3)
  */
 static inline void leds_off(void)
 {
-    VPORTA.OUT &= (uint8_t) ~(PA1_SET_MASK | PA3_SET_MASK);
+    VPORTA.OUT &= (uint8_t) ~(PA1_SET_MASK | PA2_SET_MASK | PA3_SET_MASK);
-    setPWM_PA2(0U);
 }
 
 /**
@@ -244,8 +146,7 @@ static inline void leds_off(void)
  */
 static inline void leds_on(void)
 {
-    VPORTA.OUT |= (PA1_SET_MASK | PA3_SET_MASK);
+    VPORTA.OUT |= (PA1_SET_MASK | PA2_SET_MASK | PA3_SET_MASK);
-    setPWM_PA2(255U);
 }
 
 /**
@@ -260,10 +161,9 @@ static void battery_level_indicator(void)
 /**
  * @brief Handle momentary switch input on PA0
  *
- * A long press toggles bLedEnabled.
+ * A long press toggles ::bLedEnabled.
- * A short press swiches the mode.
  */
-static bool handleSwitch(void)
+static void handleSwitch(void)
 {
     static uint16_t pressTicks = 0;  ///< Press duration counter
     static bool prevPressed = false; ///< Previous button state
@@ -272,32 +172,22 @@ static bool handleSwitch(void)
 
     if (pressed)
     {
+        bBtnPressed = true;
         if (pressTicks < 0xFFFF)
             pressTicks++; // Prevent overflow
     }
     else
     {
-        // Button released
+        bBtnPressed = false;
-        if (prevPressed)
-        {
-            // Check if it was a short press (not a long press)
-            if (pressTicks >= MS_TO_TICKS(BUTTON_SHORT_PRESS_DURATION_MS) &&
-                pressTicks < MS_TO_TICKS(BUTTON_LONG_PRESS_DURATION_MS))
-            {
-                switchMode();
-            }
-        }
-
         pressTicks = 0;
     }
 
-    if (pressed && pressTicks >= MS_TO_TICKS(BUTTON_LONG_PRESS_DURATION_MS))
+    if (prevPressed && pressTicks >= MS_TO_TICKS(BUTTON_LONG_PRESS_DURATION_MS))
     {
         bLedEnabled = !bLedEnabled; // Toggle LED blinking
     }
 
     prevPressed = pressed;
-    return pressed;
 }
 
 /**
@@ -341,7 +231,7 @@ ISR(PORTA_PORT_vect)
  *   18: all off, wait 250 ms → restart
  * Special: every 3rd cycle, all LEDs ON for 250 ms
  */
-void ledAnimationBlink(bool resetCounters)
+void blinkLed(bool resetCounters)
 {
     const uint8_t T50 = MS_TO_TICKS(50);
     const uint8_t T100 = MS_TO_TICKS(100);
@@ -408,7 +298,7 @@ void ledAnimationBlink(bool resetCounters)
         break;
 
     case 12: // LED 3–6 on, wait 50 ms
-        setPWM_PA2(255U);
+        VPORTA.OUT |= PA2_SET_MASK;
         if (counter >= T50)
         {
             counter = 0;
@@ -426,7 +316,7 @@ void ledAnimationBlink(bool resetCounters)
         break;
 
     case 16: // LED 3–6 on, wait 50 ms
-        setPWM_PA2(255U);
+        VPORTA.OUT |= PA2_SET_MASK;
         if (counter >= T50)
         {
             counter = 0;
@@ -452,47 +342,12 @@ void ledAnimationBlink(bool resetCounters)
         break;
 
     case 20: // special: all LEDs on for 250 ms
-        leds_on();
+        VPORTA.OUT |= (PA1_SET_MASK | PA2_SET_MASK | PA3_SET_MASK);
         if (counter >= T250)
         {
             counter = 0;
-            state = 0; // restart normal sequence
+            state = 0;
         }
         break;
     }
 }
-
-/**
- * @brief All LEDs with static full power
- */
-void ledStaticFull(void)
-{
-    leds_on();
-}
-
-/**
- * @brief Inner LEDs with glow animation
- */
-void ledAnimationGlow(void)
-{
-    static uint8_t brightness = 0;
-    static int8_t direction = 1;
-
-    // Update brightness level every call (10ms)
-    brightness += direction;
-
-    // Reverse direction at limits
-    if (brightness >= GLOW_BRIGHTNESS_MAX)
-    {
-        brightness = GLOW_BRIGHTNESS_MAX;
-        direction = -1;
-    }
-    else if (brightness == GLOW_BRIGHTNESS_MIN)
-    {
-        brightness = GLOW_BRIGHTNESS_MIN;
-        direction = 1;
-    }
-
-    // Apply PWM brightness to LED 3-6
-    setPWM_PA2(brightness * 255 / 100); // Scale 0-100 to 0-255
-}