Как отследить короткое и продолжительное нажатие кнопки в фонаре?

[poligon_85]

Весь код целиком в прерывании. В while (1) ничего нет. Должно отрабатываться кратковременное нажатие без проблем. Но четкого срабатывания нет. Нажимаешь - светодиод горит, отпускаешь - гаснет. Срабатывает через раз.

#define button PINB.0 

volatile bit long_push = 0;
volatile unsigned int cnt = 0;
volatile bit state = 0;

// Timer 0 overflow interrupt service routine
interrupt [TIM0_OVF] void timer0_ovf_isr(void)
{
                       // Reinitialize Timer 0 value
  TCNT0=0x06;          // срабатывание каждые 2 мс
  
  if (button == 0) {
    cnt++;
    if (cnt›65534) {
      cnt=201;
      }
    }
  else {
    cnt=0;
    long_push=0;
    }

  if ((cnt › 25) && (cnt ‹ 200)) {        // подавляем дребезг 50 мс
    if (state) {                          // и проверяем условие короткого нажатия
      OCR2=0x00;
      TCCR2=0x00;    // выключили ШИМ, остановили таймер   (выкл)
      state = 0;
      }
    else {
      OCR2=0xFF;
      TCCR2=0x6A;    // включили ШИМ на полную, запустили таймер    (вкл)
      state = 1;
      }
    }
    
  if (cnt › 200) {   // длинное нажатие
    long_push = 1;
    } 
}

...

// Timer/Counter 0 initialization
// Clock source: System Clock
// Clock value: 125,000 kHz
TCCR0=0x03;
TCNT0=0x06;

// Timer/Counter 2 initialization
// Clock source: System Clock
// Clock value: 1000,000 kHz
// Mode: Fast PWM top=FFh
// OC2 output: Non-Inverted PWM
ASSR=0x00;
TCCR2=0x00;        // при старте всё выключено
TCNT2=0x00;
OCR2=0x00;         //

// Timer(s)/Counter(s) Interrupt(s) initialization
TIMSK=0x01;

// Global enable interrupts
#asm("sei")

[omercury]

Жирненький МК для такой задачки - хватило бы и младшей "Тиньки" с головой...
Начну, как всегда, с вопросов:
- Где устанавливается флаг state до условия

if (state) {                          // и проверяем усл

?

Что Вы хотите получить - понятно. Давайте решим как...

Длинное нажатие определяется по переполнению счётчика.
Короткое нажатие определяется в момент отпускания кнопки.
Определяемся с переменными. Необходимо: счетчик для определения времени нажатия кнопки(в Вашем случае достаточно байта). Бит для хранения предыдущего состояния кнопки(для короткого нажатия). Его же можно использовать как флаг состояния. Счётчик циклов задержки на изменение компаратора ШИМ. В общем пока всё.

Алгоритм обработки таков: Структура обработчика прерывания
Если кнопка нажата
...Если флаг не установлен -› установить флаг. выход.
...Если флаг установлен -› инкремент счетчика.
......Если счётчик досчитал до длинного нажатия -› начинаем менять компаратор ШИМ. Декремент счётчика(увеличивать-то его больше никчему). выход.
Если кнопка не нажата -› проверяем флаг состояния кнопки
...Если флаг установлен - › проверяем счётчик времени нажатия кнопки
......Если меньше длинного
.........Если больше минимального времени -› включаем/выключаем фонарь. сброс переменных. выход.
.........Иначе - это был "дребезг". сброс переменных. выход.
...Если флаг не установлен -› сброс переменных. выход.

Где-то так.

[кротоВ]

 
    if (button == 0) {
         if (cnt ‹65534) cnt++;
         if (cnt›200) mode = 2// добавил переменную mode(режим или состояние) =2 это длинного нажатия дождались...
          if ((cnt‹200) &&(cnt›25)) mode = 1// короткое нажатие
    }
    else {
         cnt=0;
         short_push = 0;
         long_push = 0;
    }
// здесь закончили с определением состояния кнопки
  if (mode == 1) {    
     short_push = 1;// ещё добавил переменную для ясности..здесь может быть и другой код
// дальше сбрасываем все переменные
     mode = 0;//здесь косяк)))
     cnt = 0;//и здесь косяк)))
// что бы потребовалось новое нажатие
  }
    
  if (mode == 2)  long_push = 1;
  else long_push = 0;
}

В общем типа того мне видится, но проверить код не могу...обычно разрисовываю для себя на бумаге как конечный автомат...а сейчас недосуг
ЗЫ. всё равно накосячил )))) в общем идея такая...но вряд ли сегодня у меня получится)))

[poligon_85]

Жирненький МК для такой задачки - хватило бы и младшей "Тиньки" с головой...

Во-первых есть в наличии. Во-вторых это только начало

- Где устанавливается флаг state до условия

Код:

if (state) { // и проверяем усл

// глобальная переменная
volatile bit state = 0;

Длинное нажатие определяется по переполнению...

Сделал вроде то же самое (см. код выше). Напрочь запутался.

[omercury]

Код:

// глобальная переменная volatile bit state = 0;

Где сброшена вижу, где устанавливается, чтобы выполнилось то условие?

Сделал вроде то же самое (см. код выше). Напрочь запутался.

Очень быстро меняется значение компаратора. Нужно с задержечкой. Но не nop-ами внутри прерывания, а через второй счётчик.

upd. наверху подправил.

[кротоВ]

Наши не сдаются

    if (button == 0) {
         if (cnt ‹65534) cnt++;
         if (cnt›200) mode = 2// добавил переменную mode(режим или состояние) =2 это длинного нажатия дождались...
          if ((cnt‹200) &&(cnt›25)) mode = 1// короткое нажатие
    }
    else {
         cnt=0;
         long_push = 0;
         If (mode =1) short_push = 1;// cбрасываем её при переключении режимаshort_push = 0 там же сбрасываем и mode... в 0;
    }
//-----------------------------    
  if (mode == 2)  long_push = 1;
  else long_push = 0;
}

[poligon_85]

Где сброшена вижу, где устанавливается, чтобы выполнилось то условие?

if ((cnt › 25) && (cnt ‹ 200)) {        // подавляем дребезг 50 мс
    if (state) {                        // и проверяем условие короткого нажатия
      OCR2=0x00;
      TCCR2=0x00;    // выключили ШИМ, остановили таймер   (выкл)
      state = 0;
      }
    else {
      OCR2=0xFF;
      TCCR2=0x6A;    // включили ШИМ на полную, запустили таймер    (вкл)
      state = 1;
      }
    }

[omercury]

Вот именно. Пока статус не установлен, изменить статус невозможно. Замкнутый круг.
Компилятор, скорее всего, это условие вообще выкинул, как невыполнимое в принцыпе...

дополню:

Код:

if ((cnt › 25) && (cnt ‹ 200)) { // подавляем дребезг 50 мс if (state) { // и проверяем условие короткого нажатия OCR2=0x00; TCCR2=0x00; // выключили ШИМ, остановили таймер (выкл) state = 0; } else { OCR2=0xFF; TCCR2=0x6A; // включили ШИМ на полную, запустили таймер (вкл) state = 1; } }

[igel_izh2126]

где то я не понял идею
а зачем для кнопки компаратор?

[omercury]

где то я не понял идею
а зачем для кнопки компаратор?

Имелся в виду цифровой компаратор сравнения внутри модуля таймера в режиме PWM.