Прерывание смены контакта — внешнее прерывание с внутренним прерыванием

Внешние прерывания в архитектуре AVR сбивают с толку, но не невозможны. Я обнаружил, что лучшим ресурсом для меня является страница AVR libc по прерываниям. Я думаю, вы сделали код слишком сложным для того, что вы хотите. Начнем с нуля:

#include <avr/io.h>
#include <stdint.h>
#include <avr/interrupt.h>

void main() {
    sei();
    while(1) {};
}

AVR libc фактически делает обработку прерываний довольно безболезненной. На странице, на которую я ссылался выше, есть список всех поддерживаемых векторов прерываний на каждом чипе AVR. Предположим, вы используете Mega32 и теперь хотите, чтобы светодиоды мигали с помощью прерывания таймера. Добавим в программу:

uint8_t led_state;

ISR(TIMER0_COMP_vect) {
    led_state = ~led_state;
    PORTB = led_state;
}

void setup_timer_interrupt() {
    TCCR0B = 0x03;
    TIMSK0 = 0x01;
    TCNT0 = 0x00;
}

При этом светодиоды на PORTB должны мигать каждый раз, когда происходит прерывание таймера. Обратите внимание, что его нужно настроить с помощью макроса ISR(...); вызовы __vector_..., которые вы используете, устарели и немного сбивают с толку.

Наконец, вы хотите использовать набор переключателей для включения светодиода, если я правильно понимаю ваш вопрос. На самом деле я бы не стал использовать для этого внешнее прерывание, просто прочитал значение переключателей, используя PINK во время ISR(TIMER0_COMP_vect), но мы можем использовать его, если хотите. Нам нужно добавить следующий код:

uint8_t switch_state;

void setup_switch_interrupt() {
    // I'm assuming this code to enable external interrupts works.
    DDRK = 0x00;
    PORTK = 0xff;
    PCMSK2 = 0xff; // set to all 1s
    PCICR = 0x04;
    PCIFR = 0x04;
}

ISR(INT0_vect) {
    switch_state = PINK;
}

Что это делает? Мы будем хранить состояние переключателей в switch_state, считывая каждый раз, когда срабатывает внешнее прерывание (я предполагаю, что вы установили это для обоих переходов 0-> 1 и 1-> 0). Осталось только сделать выход светодиода зависимым от значения switch_state. Мы сделаем это в прерывании таймера, так как именно там мы переключали светодиоды до сих пор. Новая версия выглядит так:

ISR(TIMER0_COMP_vect) {
    led_state = ~led_state | switch_state;
    PORTB = led_state;
}

И это должно сделать это!

Сноска: ранее я говорил, что использование внешнего прерывания для чтения переключателей на самом деле не обязательно. Это потому, что вы можете просто прочитать значения переключателя во время прерывания таймера, используя PINK. Вы можете избавиться от switch_state, setup_switch_interrupt() и ISR(INT0_vect) и просто изменить прерывание таймера следующим образом:

ISR(TIMER0_COMP_vect) {
    led_state = ~led_state | PINK;
    PORTB = led_state;
}

Это должно сделать программу немного проще.

Поэтому, несмотря ни на что, каждый раз, когда выполняется ваш __vector_23, вы циклически переключаете светодиоды, назначенные PORTB, увеличивая i. Если я понимаю, что вы пытаетесь сделать, то вы должны увеличивать i только в __vector_25, когда переключатель нажат.