Не могу понять, где ошибка в программе

[Valer 5]

Написал часть программы для обработки нажатия кнопки для Attiny2313, при котором должно изменяться значение одной переменной. При коротком нажатии однократном или многократном соответственно изменение значения на 1 или равно количеству нажатий. При долгом нажатии изменение на 10 . Чтобы не вводить весь код, описываю словами:
1. В таймере Т0 в прерывании увеличивается на 1 переменная k, при этом прерывание запрещено.
2. В main, while (1) при обработке нажатия кнопки разрешается прерывание Т0;

     while (1)
 {
      // Place your code here
 if(!PIND.3)
  { TIMSK=0x02,PORTB=k1;}
  if((k1==20)&&(!PIND.3)) {k2=k1, sec++;}

   // if((k1==40)&&(!PIND.3))
   // {TIMSK=0x00,sec+=10-1,k1=0;}
    if(PIND.3)  {TIMSK=0x00,k1=0;}
 }

k1 считается в Т0, k2 показывает выполнение 3ей строки программы,
sec++ должна увеличиваться на единицу, последняя строка останавливает прерывание.
При симуляции в Протеус7 и удержании кнопки на PIND.3 более 4 сек. для прохождения более 20 имп. sec›30. Где я не прав?
Прошу очень не ругать, т.к. программировать учусь.
Пишу и компилирую в CVAVR.

[mike-y-k]

Valer 5, а как связаны T0 и нажатие кнопки?
Может стоит сменить логику обработки?
Например прерывание по нажатию и отпусканию кнопки…
А внутри того прерывания разбираться с длительностью и решать вопрос при отпускании о величине инкремента счетчика…

PS Вы бы логику алгоритма управления кнопками расписали, может там и другие варианты возможны…
Например энкодер, которым можно параметр без заморочек менять в обе стороны - там всего-то код Грея разобрать нужно будет…

[Valer 5]

mike-y-k
При нажатии кнопки запускается таймер Т0. В каждом прерывании увеличивается переменная k1, которая говорит о прошедшем, после нажатия кнопки, времени. В
while при прошествии 20 прерываний (примерно 2 сек) переменная sec должна увеличиваться на 1. Далее до отпускания кнопки ничего не должно меняться. При отпускании кнопки прерывания запрещается, k1 обнуляется и при следующем нажатии всё повторяется. Как мне кажется.

[mike-y-k]

А если таймер всегда тикает с некоторым интервалом и инкрементирует в некоторой переменной время?
При нажатии кнопки по прерыванию от нажатия сохранили отсчет таймера в переменной, при отпускании посчитали разницу по времени, по величине разницы поменяли значение в счетчике нажатий, если времени прошло мало - отбросили как дребезг контактов и ждем следующего отпускания кнопки…
Алгоритм должен быть минимально зависим от размеров кода в основном цикле программы…
И разрешение прерываний принято делать в обработчике, обычно при выходе и с минимумом команд в нем…

PS Вам очень поможет раздел Конечных Автоматов (Finite-State Machine) из Теории Алгоритмов…

[AJScorp]

Valer 5,
Как объявлена переменная k1?
Так как она изменяется в прерывании и проверяется в цикле, то должна быть объявлена с ключевым словом volatile, ибо могут быть чудеса оптимизации.

[akegor]

Как объявлена переменная k1?

Да, это очень важно, когда "кефаль думает за вас"(из рекламы).
На всякий случай, добавлю для Valer 5: никаких "по умолчанию" и неявных преобразований, только всё ручками. Нюансы умолчаний и неявных преобразований могут меняться от версии к версии одного и того же компилятора и при использовании другого.

[Valer 5]

AJScorp, переменная объявлена

volatile unsigned int k1=0;

[mike-y-k]

Valer 5, таки задействуйте 6 или 7 ногу как INT0/INT1 и разбирайтесь с состоянием кнопки по прерыванию при постоянно работающем 8 или 16-битном таймере

[Valer 5]

Valer 5, таки задействуйте 6 или 7 ногу как INT0/INT1 и разбирайтесь с состоянием кнопки по прерыванию при постоянно работающем 8 или 16-битном таймере

mike-y-k, я писал, что при коротком нажатии надо увеличить переменную на 1, а при длинном на 10. При постоянно работающем прерывании трудно нажать на требуемый интервал, потребуется сложная обработка. А так при отпущенной кнопке прерывания запрещены, при нажатии начинается прерывания, если их больше 20 в переменную записывается 1. Дальше у меня в виде комментариев записано условие для прибавления 10.

[mike-y-k]

Valer 5, прерывание по edge все решает - читаем состояние порта и при нужном уровне разбираемся с таймером. При нажатии запоминаем текущий таймер или обнуляем его, при отпускании - вычисляем или читаем длительность.
Алгоритм таки стоит основывать изначально на оптимальном выборе из возможностей аппаратуры…
Это для очень больших и быстрых можно немного расслабиться…
Я выше уже на макал на конечные автоматы - тут именно их случай в чистом виде…
Перед написанием кода стоит изучить все особенности целевого МК и расписать весь алгоритм на основе целевого железа, а уже потом диаграмму состояний оформить кодом…
Чем меньше МК, тем это правило существеннее влияет…
В ограниченных ресурсах памяти каждый байт кода значим, а при ограниченной скорости - каждый такт…
Поэтому часто итоговая прошивка допиливается и вылизывается на ассемблере…
С использованием прерывания от кнопки Вы сможете даже дребезг контактов спокойно обработать…
Время работы обработчика ≈10 тактов максимум или 10µs на 1MHz, 1µs на 10MHz, 500ns на 20MHz