Don't sync state of leds when powering down, only when power save

Author: M65649

src/examples/power_down/power_down.ino

@@ -1,15 +1,15 @@
-#include <Wire.h>
 #include <avr/io.h>
 #include <led_ctrl.h>
 #include <log.h>
 #include <low_power.h>
 #include <lte.h>
-#include <sequans_controller.h>
 
 #define SW0 PIN_PD2
 
 ISR(PORTD_PORT_vect) {
     if (PORTD.INTFLAGS & PIN2_bm) {
+        // Reset the interupt flag so that we can process the next incoming
+        // interrupt
         PORTD.INTFLAGS = PIN2_bm;
     }
 }
@@ -44,6 +44,8 @@ void setup() {
 void loop() {
 
     Log.info("Powering down...");
+    // Allow some time for the log message to be transmitted before we power
+    // down
     delay(100);
 
     // Power down for 60 seconds

src/low_power.cpp

@@ -331,9 +331,6 @@ static void disablePIT(void) {
 static void powerDownPeripherals(void) {
 
     // LEDs
-    cell_led_state = digitalRead(LedCtrl.getLedPin(Led::CELL));
-    con_led_state = digitalRead(LedCtrl.getLedPin(Led::CON));
-
     pinConfigure(LedCtrl.getLedPin(Led::CELL),
                  PIN_DIR_INPUT | PIN_PULLUP_ON | PIN_INPUT_DISABLE);
     pinConfigure(LedCtrl.getLedPin(Led::CON),
@@ -510,6 +507,8 @@ bool LowPowerClass::configurePeriodicPowerSave(
 }
 
 void LowPowerClass::powerSave(void) {
+    cell_led_state = digitalRead(LedCtrl.getLedPin(Led::CELL));
+    con_led_state = digitalRead(LedCtrl.getLedPin(Led::CON));
 
     powerDownPeripherals();
     SLPCTRL.CTRLA |= SLPCTRL_SMODE_PDOWN_gc | SLPCTRL_SEN_bm;
@@ -556,6 +555,9 @@ void LowPowerClass::powerSave(void) {
 
     SLPCTRL.CTRLA &= ~SLPCTRL_SEN_bm;
     powerUpPeripherals();
+
+    digitalWrite(LedCtrl.getLedPin(Led::CELL), cell_led_state);
+    digitalWrite(LedCtrl.getLedPin(Led::CON), con_led_state);
 }
 
 void LowPowerClass::powerDown(const uint32_t power_down_time_seconds) {