Fixed the bug in switching logic, timer is now working properly, LEDs are now disabled when microcontroller is in sleep mode

Author: pcm720

main.c

@@ -22,7 +22,7 @@ HDMI Switcher code for ATmega88PA
 #include "switch.h"
 #include "eeprom.h"
 
-volatile uint8_t selectedInput;
+volatile uint8_t selectedInput = 0x0;
 volatile uint8_t timer = 11;
 
 void init(void);
@@ -38,6 +38,8 @@ int main(void) {
         outputState = 0x0;                 // reading failed, revert to default parameters
         eeprom_write(outputState);         // try to save defaults
     }
+    selectedInput = outputState;
+    outputState = 0x0;
 
     init();
 
@@ -85,7 +87,7 @@ void init(void) {
     PORTC = 0x0;
     DDRD = 0x0;  // set PD to input
     PORTD = 0x0; // disable pull-ups
-    PRR = 0xEB;  // power reduction
+    PRR = 0xE7;  // power reduction
 
     PCMSK2 = 0x3F;          // prepare interrupts for every input pin
     PCICR = 0x4;            // enable PCINT for PORTD
@@ -97,6 +99,15 @@ void init(void) {
 }
 
 void mcu_sleep(void) {
+    PORTB |= 0xC3; // blink all LEDs
+    _delay_ms(1000);
+    PORTB &= ~0xC3;
+    _delay_ms(100);
+    PORTB |= 0xC3;
+    _delay_ms(100);
+    PORTB &= ~0xC3; // disable LEDs
+    _delay_ms(100);
+    
     set_sleep_mode(SLEEP_MODE_PWR_DOWN);
     cli();
     sleep_enable();

switch.c

@@ -25,13 +25,13 @@ void switch_off(void) {
     _delay_ms(10);
     PORTC &= ~SW3_OE;       // disable SW3_OE
     _delay_ms(10);
+    PORTC &= ~Dx_SEL_MASK; // clear Dx_SEL on all switches
 }
 
 void switch_input(uint8_t portConfig, uint8_t coldStart) {
     ATOMIC_BLOCK(ATOMIC_RESTORESTATE) {
         if (coldStart == 0) switch_off();
-        if (portConfig == OFF) PORTC &= ~Dx_SEL_MASK; // clear Dx_SEL on all switches
-        else {
+        if (portConfig != OFF) {
             PORTC |= (portConfig & Dx_SEL_MASK);      // set Dx_SEL on all switches
             _delay_ms(500);
             PORTC |= (portConfig & SW1_SW2_OE);       // enable SWn_OE