Помогите с обработкой кнопок через ADC в CVAVR

[Dimitrij]

Есть код обработки кнопок через ADC:

#define K5 0
#define K4 118
#define K3 160
#define K2 182
#define K1 194  
#define K0 213 // для ненажатого состояния 

/*------ Выводы кнопок -----------------------------------------*/
#define KEYt PINA.6
/*------ -------------------------------------------------------*/
unsigned char read_adc(unsigned char adc_input);
char find = 0;

unsigned char KEYPRESS ()
{
unsigned char Key,KeyTmp;   // нажатая клавиша
delay_ms(35);
if(read_adc(6)‹K0)
{
  delay_ms(15); //дребезг контактов
  KeyTmp=read_adc(6);
  if(KeyTmp‹K0)
    {
    if(KeyTmp‹K1){
        if(KeyTmp‹K2){
            if(KeyTmp‹K3){
              if(KeyTmp‹K4)Key=3;
              else Key=4;}
            else Key=1;}
        else Key=2;}            
    else Key=5;
    }                                       
}
else Key=0;
        delay_ms(75);
    return(Key);    
    } 

В непрерывном цикле:

if(PINA.5==0) find = 1;
else find = 0; 

Все работает нормально. Но мне нужно, чтобы при установленной переменной find==1 и нажатых кнопках K4 или К5, Key=3; или Key=4; отрабатывались непрерывно. А при find==0 - одинарно.
Не могу сообразить, как это сделать?

[Easyrider83]

Мощный код. Особенно обработка дребезга повеселила. Есть лишнее время в цикле, можно хотя бы примитивное усредненение сделать. Да и обработка нажатия должна быть с каким-то допуском +/-.

[Dimitrij]

Мощный код. .

Я прошу помощи, а не критики!

[whoim]

Я прошу помощи

Есть лишнее время в цикле

примитивное усредненение сделать

обработка нажатия должна быть с каким-то допуском +/-

обработка дребезга

не? маловато?

[_guardianangel]

Dimitrij,

KeyboardEventTypeDef keyboard_status(void)
{
    uint16_t *puiKeyBoardData = adc_GetData(KEYBOARD_CHANEL);
    if (puiKeyBoardData)
    {
        if ((*puiKeyBoardData ›= UL_MIN) && (*puiKeyBoardData ‹= UL_MAX)) return (PRESS_LEFT);
        if ((*puiKeyBoardData ›= UD_MIN) && (*puiKeyBoardData ‹= UD_MAX)) return (PRESS_DOWN);
        if ((*puiKeyBoardData ›= UR_MIN) && (*puiKeyBoardData ‹= UR_MAX)) return (PRESS_RIGHT);
        if ((*puiKeyBoardData ›= UU_MIN) && (*puiKeyBoardData ‹= UU_MAX)) return (PRESS_UP);
        if ((*puiKeyBoardData ‹= DEFENCE_OFFSET)) return (NO_PRESS);
    }
    else
    {
        return (UNKNOWN);
    }
    return (ERROR_KBD);
} 

Код рабочий, диапазоны значений АЦП рассчитывал исходя из 10 процентного допуска резисторов делителя. Это не сложно, если, конечно, закон Ома не забыли )))).

[_guardianangel]

P.S. Еще использую усреднение по 8 выборкам. Тупо суммирую 8 отсчетов и сдвигаю влево на 3 разряда. Прямо в прерывании АЦП.

[Dimitrij]

Спасибо, но вопрос был о том, как сделать, чтобы одна и та же кнопка отрабатывалась однократно или многократно, в зависимости от установленного флага. Может, я не так изначально поставил вопрос, но смысл такой:
Кнопки К4 и К5 управляют перестройкой частоты. При одинарном нажатии на кнопку, частота перестраивается +- на один шаг. А мне нужно, чтобы при установленном флаге find и одинарном нажатии на кнопку, частота перестраивалась автоматически до сброса флага (автопоск станции). Даже желательно, чтобы автопоиск включался не сразу, а при удержании кнопки в течении 1-2 секунд.
А код опроса ADC переписать, если нужно, не вопрос.

[Godzilla82]

Можно так:

 
#define K4 118
#define K3 160
#define K2 182
#define K1 194
#define K0 213

unsigned char KEYPRESS(void) 
{ 
unsigned char c;
c = read_adc(6);
if(c ‹ K4)  return 5;
if(c ‹ K3)  return 4;
if(c ‹ K2)  return 3;
if(c ‹ K1)  return 2;
if(c ‹ K0)  return 1;
return 0;
} 

void main(void)
{
unsigned char b = 0;
unsigned char k;

begin:
k = KEYPRESS();
switch(k)
{
case 1:
if(b == 0)      Press_1();      // одиночное срабатывание
if(b == 39)     Press_Find();   // одиночное срабатывание после 2-х сек. удержания
if(b == 40)     Press_1();      // непрерывный повтор с интервалом 50 мс после 2-х сек. удержания 
break;

case 2:
if(b == 0)      Press_2();
break;

case 3:
if(b == 0)      Press_3();
break;

case 4:
if(b == 0)      Press_4();
break;

case 5:
if(b == 0)      Press_5();
break;
}
if(b ‹ 40) b++;
if(k == 0) b = 0;
delay_ms(50);
goto begin;
}

Условие срабатывания по нужным вам флагам добавите сами.

[proba60]

не хотел заводить новую тему из за одного вопроса ,поэтому размещу здесь Почему вот это работает в CV

int tim=50;
delay_ms (tim);

а вот это не работает
int tim=50;
delay_us (tim);

[Godzilla82]

а вот это не работает
int tim=50;
delay_us (tim);

Потому, что delay_ms() построена так:

Имеется процедура задержки на 1 мс. В регистр Y загружается количество раз, которое будет вызываться процедура. И потом в цикле вызывается процедура задержки на 1 мс указанное количество раз.

Процедура задержки на 1 мс - это пустой цикл. Количество раз выполнения которого расчитывается на этапе компиляции в зависимости от тактовой частоты.

_delay_ms:
	adiw r26,0
	breq __delay_ms1
__delay_ms0:
	__DELAY_USW 0x7D0
	wdr
	sbiw r26,1
	brne __delay_ms0
__delay_ms1:
	ret

__DELAY_USW - это как раз макрос пустого цикла, который выполняется 1 мкс.
А 0x7D0 - это количество итераций, равное CLK / 4000.

Функция delay_us() - это всего один цикл. На самом деле всё тот же макрос __DELAY_USW. Параметр которого расчитывается на этапе компиляции.

Если хотите - делайте свой аналог delay_us.
Например:

i = 1;
do {i--;} while(i);

Одна итерация цикла выполнится за 6 тактов.
Общее время выполнения на 2 такта больше.

Соответственно при тактовой частоте 1 Мгц минимальная задержка 8 мкс.

При таком подходе нужно смотреть ассемблерный листинг. Потому, что количество тактов на 1 итерацию цикла может существенно измениться. Например, если компилятор разместит перемнную i в регистрах X-Z количество тактов может стать 4.
И наоборот, при нехватке регистров количество тактов может возрасти до 14 и более (за счёт размещения переменной в памяти).