Nie wiedzieć czemu program mimo poprawnej kompilacji nie działa jak powinien. W przerwaniu chciałem obsługiwać klawisze, które by zmieniały zmienną do obsługi menu. Niestety wyświetlanie menu "show_menu()" wygląda jakby tylko raz się wykonywało. Testowałem (do testów użyłem lcd_puts(buf) w przerwaniu) i przerwanie zmienia wartość menu_pos ale funkcja, która by miała wyświetlać menu nie reaguje na te zmiany. Poniżej kod:
keyboard.c
[syntax=c]#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#include <stdio.h>
#include "keyboard.h"
#include "menu.h"
#include "hd44780.h"
enum t_key {left, enter, right};
enum t_key sw_check(void);
void set_var(void);
void keyboard_init(void)
{
KEYS_DDR &= ~KEY_LEFT | ~KEY_ENTER | ~KEY_RIGHT;
KEYS_PORT |= KEY_LEFT | KEY_ENTER | KEY_RIGHT;
TCCR0 |= (1<<WGM01);
TCCR0 |= (1<<CS02) | (1<<CS00);
//OCR0 |= 38;
TIMSK |= (1<<OCIE0);
}
ISR(TIMER0_COMP_vect)
{
set_var();
}
enum t_key sw_check(void)
{
_delay_ms(10);
if( ! (KEYS_PIN & KEY_LEFT) ) return left;
if( ! (KEYS_PIN & KEY_ENTER) ) return enter;
if( ! (KEYS_PIN & KEY_RIGHT) ) return right;
else return -1; //musi byc jakis return inaczej jesli nie nacisniemy zadnego klawisza funkcja domyslnie zwroci 0 co odpowiada za przycisk lewo
}
void set_var(void)
{
switch( sw_check() )
{
case left:
if (menu_pos <= 0)
{
menu_pos = 0;
break;
}
else menu_pos--;
break;
case enter:
lcd_clrscr();
lcd_puts("enter");
break;
case right:
if(menu_pos >= MENU_OPTIONS_NUMBER - 1)
{
menu_pos = MENU_OPTIONS_NUMBER;
break;
}
else menu_pos++;
break;
}
}[/syntax]
menu.c
[syntax=c]#include <avr/io.h>
#include "hd44780.h"
#include <stdio.h>
#include "menu.h"
#include <util/delay.h>
/**********************************************************************
Cursor positioning
Input: Row,Position
Returns: None
**********************************************************************/
void lcd_gotoRP(uint8_t row,uint8_t pos)
{
switch(row)
{
case 1: lcd_goto(pos); break;
case 2: lcd_goto(0x40+pos); break;
}
}
extern volatile int menu_pos;
void show_menu(t_menu_item *ptr_menu_tab)
{
char buf[50];
sprintf(buf,"%d",menu_pos);
if(menu_pos == 0)
{
/*lcd_clrscr();
lcd_gotoRP(1,6);
lcd_puts(ptr_menu_tab[menu_pos].m_name);
lcd_gotoRP(1,12);
lcd_puts(ptr_menu_tab[menu_pos + 1].m_name);
_delay_ms(10);*/
lcd_clrscr();
lcd_puts(buf);
}
if(menu_pos == MENU_OPTIONS_NUMBER -1)
{
/*lcd_clrscr();
lcd_puts(ptr_menu_tab[menu_pos - 1].m_name);
lcd_gotoRP(1,6);
lcd_puts(ptr_menu_tab[menu_pos].m_name);
_delay_ms(10);*/
lcd_clrscr();
lcd_puts(buf);
}
else
{
/*lcd_clrscr();
lcd_puts(ptr_menu_tab[menu_pos - 1].m_name);
lcd_gotoRP(1,6);
lcd_puts(ptr_menu_tab[menu_pos].m_name);
lcd_gotoRP(1,12);
lcd_puts(ptr_menu_tab[menu_pos + 1].m_name);
_delay_ms(10);*/
lcd_clrscr();
lcd_puts(buf);
}
}[/syntax]
main.c
[syntax=c]/*
*/
#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include "hd44780.h"
#include "hd44780_settings.h"
#include "menu.h"
#include "keyboard.h"
t_menu_item main_menu[MENU_OPTIONS_NUMBER] =
{
{"tes1", 1},
{"tes2", 2},
{"tes3", 3},
{"tes4", 4},
{"tes5", 5},
};
volatile int menu_pos = 0;
int main(void)
{
lcd_init();
sei();
keyboard_init();
while(1)
{
show_menu(main_menu);
}
return 0;
}[/syntax]
Jeśli ręcznie zmienię menu_pos na inną wartość to show_menu wyświetli zgodnie z "if'ami" jak należy jeśli jednak manipuluję tą zmienną z przerwania niestety nic się nie dzieje. Co ciekawe gdy wcisnę klawisze "enter" wyświetla sie napis "enter" (tak jak jest ustawione w przerwaniu i już tak zostaje nie znika.. jakby funkcja wyswietlająca menu wykonywała się tylko raz na samym początku. Czy ktoś może widzi jakiś błąd? Może to jakaś oczywista oczywistość a ja nie potrafię tego zauważyć.
Co ciekawe funkcje "debugujące" lcd_puts(buf) w funkcji "show_menu" na ekranie nie zwracają nic - pusty ekran aż do naciśnięcia "enter". W przypadku obecnie wykomentowanego kodu, pokazują menu dla pozycji zależnej od menu_pos ustawionego w main.c.. niestety nie reaguje jak wspomniałem na zmianę.
EDIT:
Spróbowałem jeszcze taki test w main.c:
[syntax=c]...
while(1)
{
sprintf(buf,"%d",menu_pos);
lcd_clrscr();
lcd_puts(buf);
_delay_ms(100);
//show_menu(main_menu);
}
...[/syntax]
Dzieje się to samo. Jakby pętla while wykonywała się tylko raz.. przy naciśnięciu lewo/prawo nic się nie dzieje cały czas jest na wyświetlaczu "0" gdy nacisnę "enter" pojawia się napis "enter" i już nie znika.
EDIT2:
Próbowałem również zablokować przerwania na czas wykonywania "set_var":
[syntax=c]...
ISR(TIMER0_COMP_vect)
{
ATOMIC_BLOCK(ATOMIC_RESTORESTATE)
{
set_var();
}
}
...[/syntax]
Nie pomoglo :/
EDIT 3 - PROBLEM ROZWIAZANY
Problemem okazała się funkcja delay w przerwaniu. Przerwanie wykonywało się tak długo, że nie było czasu na główną funkcję.. wystarczyło odpalić przewodnik z przerwaniam Mirka
. Jednak teraz mam nowy problem gdyż sprawdzanie wciśniętego klawisza odbywa się tak szybko, że przeskakuje od 0 - 5 przy jednym naciśnięciu. Jak to teraz rozwiązać?EDIT 4 - PROBLEM CALKOWICIE ROZWIAZANY
Udało mi się rozwiązać drgania styków bez użycia "delay" za pomocą następującego przerwania (co przerwanie sprawdzam czy dany klawisz jest wciśnięty i zwiększam zmienną kontrolną o jeden). W ten sposób idzie idealnie wysterować switch bez przerywania głównej funkcji programu:
[syntax=c]...
ISR(TIMER0_COMP_vect)
{
ATOMIC_BLOCK(ATOMIC_RESTORESTATE)
{
static uint8_t left_ck = 0;
static uint8_t enter_ck = 0;
static uint8_t right_ck = 0;
if( ! (KEYS_PIN & KEY_LEFT) ) left_ck++;
if( ! (KEYS_PIN & KEY_ENTER) ) enter_ck++;
if( ! (KEYS_PIN & KEY_RIGHT) ) right_ck++;
if(left_ck >= 50){
if(menu_pos == 0) left_ck = 0;
else{
menu_pos--;
left_ck=0;
}
}
if(enter_ck >= 50) enter_ck = 0;
if(right_ck >= 50){
if(menu_pos >= MENU_OPTIONS_NUMBER -1){
menu_pos = MENU_OPTIONS_NUMBER -1;
right_ck = 0;
}
else{
menu_pos++;
right_ck = 0;
}
}
}
}
..[/syntax]
Pozdrawiam
//holo
Statystyki: Napisane przez holo — 26 gru 2014, o 12:47
]]>