ATNEL tech-forum

Re: Kłopot z przerwaniami

2014-07-11T15:39:36+01:00

Witam. Po przerwie związanej z sesją, chciałbym powrócić do dalszego rozbudowywania menu. Troszkę przebudowałem swoje menu i wygląda ona następująco:
[syntax=c]#define E_IDDLE 0
#define E_Prev 1
#define E_Select 2
#define E_Next 3

void menuiddle( uint8_t event);
void zmianaMENU();
void godzina( uint8_t event);
void up(void);
void down(void);
void ok(void);

const uint8_t MN01[] PROGMEM="1.Godzina";
const uint8_t subMN01_1[] PROGMEM="1_1.Czas";
const uint8_t subMN01_2[] PROGMEM="1_2.Data";
const uint8_t MN02[] PROGMEM="2.Wspol";
const uint8_t MN03[]PROGMEM = "3.COS";

typedef struct
{
uint8_t next_state[4];//przejścia do następnych stanów
void (*callback)(uint8_t event);//funkcja zwrotna
uint8_t* pierwszyWiersz;
uint8_t* drugiWiersz;
} menu_item;

enum {
Hello,
Menu1,
SubMenu1_1,
SubMenu1_1_1,
Submenu1_2,
Menu2,
Menu3
} ;

const menu_item menu[] = {
//Iddle,Prev, Select,Next
{{Hello, Menu1, Menu1, Menu1}, NULL,NULL, NULL},
{{Menu1, Menu1, Menu2, SubMenu1_1}, NULL, MN01,MN02},
{{SubMenu1_1, Menu1, Submenu1_2, SubMenu1_1_1},NULL, subMN01_1, subMN01_2},
{{SubMenu1_1_1, SubMenu1_1, SubMenu1_1_1, SubMenu1_1_1},godzina, NULL, NULL},
{{Submenu1_2, Menu1, SubMenu1_1, Submenu1_2}, NULL, subMN01_2, subMN01_1},
{{Menu2, Menu2, Menu3, Menu2}, NULL, MN02, MN03},
{{Menu3, Menu3, Menu1, Menu3}, NULL, MN03, MN01},

};[/syntax]

Do obsługi wykorzystuje funkcję
[syntax=c]void zmianaMENU() {
current_menu = menu[current_menu].next_state[menu_event];
lcd_cls();
//wyświetl 1-szą linię
if (menu[current_menu].pierwszyWiersz)
{

lcd_locate(0,1);
lcd_str_P((char*)menu[current_menu].pierwszyWiersz);
}
//wyświetl 2-gą linię
if (menu[current_menu].drugiWiersz)
{
lcd_locate(1,1);
lcd_str_P((char*)menu[current_menu].drugiWiersz);
}
//wywolaj funkcje zwrotna
if (menu[current_menu].callback)
menu[current_menu].callback(menu_event);

//skasuj zdarzenie
menu_event = E_IDDLE;
}[/syntax]

Tak prezentuje się w całości plik main.c
[syntax=c]#include
#include
#include
#include
#include

#include "klawisz/klawisz.h"
#include "menu.h"
#include "LCD/lcd44780.h" //biblioteka_LCD

volatile uint8_tcurrent_menu = 0;
volatile uint8_tmenu_event = E_IDDLE;
volatile uint16_t Timer3;

uint8_t key1_lock, key2_lock, key3_lock;

/*POCZATEK_FUNKCJI_MAIN*/
int main(void) {
//DDRD |= (1<//PORTD &=~(1<//DDRC |= (1<
PORTC |= KEY1 | KEY2 | KEY3;

TCCR2 |= (1<TCCR2 |= (1<OCR2 = 108;
TIMSK = (1<
lcd_init();
sei();

/*POCZATEK_PETLI_WHILE*/
while(1) {
key_press( &key1_lock, &PINC, KEY1, up );
key_press( &key2_lock, &PINC, KEY2, down );
key_press( &key3_lock, &PINC, KEY3, ok );

if(menu_event) {
zmianaMENU();
}
if(current_menu == E_IDDLE)
menuiddle(E_IDDLE);

}
/*KONIEC_PETLI_WHILE*/

}
/*KONIEC_FUNKCJI_MAIN*/

void up(void)
{
menu_event=E_Prev;
}
void down(void)
{
menu_event=E_Next;
}
void ok(void)
{
menu_event=E_Select;
}

void zmianaMENU() {
current_menu = menu[current_menu].next_state[menu_event];
lcd_cls();
//wyświetl 1-szą linię
if (menu[current_menu].pierwszyWiersz)
{

lcd_locate(0,1);
lcd_str_P((char*)menu[current_menu].pierwszyWiersz);
}
//wyświetl 2-gą linię
if (menu[current_menu].drugiWiersz)
{
lcd_locate(1,1);
lcd_str_P((char*)menu[current_menu].drugiWiersz);
}
//wywolaj funkcje zwrotna
if (menu[current_menu].callback)
menu[current_menu].callback(menu_event);

//skasuj zdarzenie
menu_event = E_IDDLE;
}

ISR(TIMER2_COMP_vect) {
uint16_t n;

n = Timer3;
if(n)
Timer3 = --n;
}[/syntax]

Mój problem polega dosłownie identycznie jak poprzednio. Czyli wchodząc do 'SubMenu1_1_1' chciałbym otrzymywać przykładowo pomiary czasu_rzeczywistego. Rozwiązanie z poprzedniego menu nie potrafię przenieść na obecne menu. Czy ktoś mógłby naprowadzić jak sobie z tym poradzić?

Statystyki: Napisane przez dawidek034 — 11 lip 2014, o 15:39

Re: Kłopot z przerwaniami

2014-06-12T22:38:45+01:00

Bardzo dziękuje za porady. Mam nadzieje, że teraz się zgłoszę dopiero w celu zaprezentowania działającego menu

Statystyki: Napisane przez dawidek034 — 12 cze 2014, o 22:38

Re: Kłopot z przerwaniami

2014-06-12T21:56:35+01:00

Już wiem czym było spowodowane moje zagubienie Deklaracje funkcji zostały poczynione tylko, że nie zdefiniowałem funkcji i dlatego był błąd wyrzucany z strony kompilatora

Quote:

../main.c:50: error: 'pomiarTempAction' undeclared here (not in a function)

Poniżej daje graf przejść

Miałem zrobiony już wcześniej ten graf, ale niestety nie umiałem go zbytnio przełożyć na program nawet przy tak prostym grafie.

Statystyki: Napisane przez dawidek034 — 12 cze 2014, o 21:56

Re: Kłopot z przerwaniami

2014-06-12T21:28:27+01:00

mokrowski napisał(a):

Kolego, ty chyba jednak nie wziąłeś kartki i nie narysowałeś tych stanów

Ja tylko od siebie dodam - że nawet przy w sumie tak stosunkowo niby prostym programie ... sam (tak dla siebie) nie odważyłbym się marnować czasu - bez uprzedniego ostrego popracowania z kartką i ołówkiem a nawet gumką Na prawdę to nie tylko oszczędność czasu ale i nerwów .... własnych oczywiście.

Statystyki: Napisane przez mirekk36 — 12 cze 2014, o 21:28

Re: Kłopot z przerwaniami

2014-06-12T20:48:24+01:00

Staram się rozwiązać problem, jak mam zaimplementować funkcję do tego. Bo zwraca mi taki komunikat:

Quote:

../main.c:50: error: 'pomiarTempAction' undeclared here (not in a function)

Więc zrobiłem coś takiego:
[syntax=c]#ifndef USED_FOR_TARGET
#define pomiarTempAction 0
#define pomiarWilgotnoscAction 1
#define pomiarWartoscAction 2
#define menuTempAction 3
#define menuWilgotnoscAction 4
#define menuWartoscAction 5
#endif[/syntax]

Tylko, że teraz w konsoli dostaje warning:

Quote:

../main.c:50: warning: initialization makes pointer from integer without a cast

Jeżeli można wytłumaczyć ten fragment bo nie mam pojęcia jak zadeklarować funkcję dla każdego stanu.

Statystyki: Napisane przez dawidek034 — 12 cze 2014, o 20:48

Re: Kłopot z przerwaniami

2014-06-12T00:12:58+01:00

Dokładnie o coś takiego mi właśnie chodzi. Wprowadziłem zmiany niby zachowuje się tak jak chcę, ale to jest bardzo dalekie od rozwiązania uniwersalnego.
[syntax=c]enum pomiary { pomiarTemp, pomiarWilgotnosc, pomiarWartosc };
enum pomiary s_pomiary = 10;[/syntax]

przypisałem s_pomiary wartość aby przy pierwszym uruchomieniu pętli głównej nie wchodził mi automatycznie do pomiarTemp

W pętli głównej
[syntax=c]if(s_pomiary == pomiarTemp) {
lcd_locate(1,4);
lcd_str("Temp");
}
if(s_pomiary == pomiarWilgotnosc) {
lcd_locate(1,4);
lcd_str("wilgot");
if(!Timer2) {
Timer2 = 20;
PORTD ^= LED1;
}
}
if(s_pomiary == pomiarWartosc) {
lcd_locate(1,4);
lcd_str("wartosci");
}([/syntax]

Tylko, że nieudolnie wychodzi mi obsługa key_enter. Zrobiłem coś takiego w funkcji do której się odwołuje klawisz_enter
[syntax=c]void kl2_press(void){
s_pomiary = element_menu;
}[/syntax]

oraz dodałem coś takiego dla klawisza_menu aby przy ponownym wciśnięciu go wychodzić z stanu_pomiaru
[syntax=c]void kl1_press(void){
element_menu++;
lcd_cls();
s_pomiary= 50;
}[/syntax]

tutaj dodam całość kodu, bo pokrętnie to tłumaczone
[syntax=c]#include
#include
#include "LCD/lcd44780.h"
#include "1Wire/ds18x20.h"
#include

#define KEY3 (1<#define KEY2 (1<#define LED (1<#define LED1 (1<

volatile uint16_t Timer1, Timer2, Timer3;

void superDebounce(volatile uint8_t *KPIN,
uint8_t key_mask, uint16_t rep_time, uint16_t rep_wait,
void (*push_proc)(void), void (*rep_proc)(void) );
void kl1_press(void);
void kl1_rep(void);
void kl2_press(void);
void kl2_rep(void);
void show_menu(uint8_t element);
void test();
uint8_t element_menu = 0;

enum menu { menuTemp, menuWilgotnosc, menuWartosci };
enum menu s_menu ;

enum pomiary { pomiarTemp , pomiarWilgotnosc , pomiarWartosc };
enum pomiary s_pomiary = 10;

int main(void) {

DDRD |= LED | LED1;
PORTD &=~ LED | LED1;

PORTC |= KEY3 | KEY2;
TCCR2 |= (1<TCCR2 |= (1<OCR2 = 108;
TIMSK = (1<

lcd_init();
sei();

while(1) {
superDebounce(&PINC, KEY3, 20, 500, kl1_press, kl1_rep );
superDebounce(&PINC, KEY2, 80, 400, kl2_press, kl2_rep );

show_menu(element_menu);
if(element_menu > 2)
element_menu = 0;

if(s_pomiary == pomiarTemp) {
lcd_locate(1,4);
lcd_str("Temp");
}
if(s_pomiary == pomiarWilgotnosc) {
lcd_locate(1,4);
lcd_str("wilgot");
if(!Timer2) {
Timer2 = 20;
PORTD ^= LED1;
}
}
if(s_pomiary == pomiarWartosc) {
lcd_locate(1,4);
lcd_str("wartosci");
}
test();

}
}

void show_menu(uint8_t element) {
switch(element) {
case menuTemp:
lcd_locate(0,4);
lcd_str("Temp");
break;
case menuWilgotnosc:
lcd_locate(0,4);
lcd_str("Wilg");
break;
case menuWartosci:
lcd_locate(0,4);
lcd_str("Wart");
break;
}

}

void superDebounce(volatile uint8_t *KPIN,
uint8_t key_mask, uint16_t rep_time, uint16_t rep_wait,
void (*push_proc)(void), void (*rep_proc)(void) ) {

enum KS {idle, debounce, go_rep, wait_rep, rep};

static enum KS key_state;
static uint8_t last_key;
uint8_t key_press;

// zabezpieczenie przed wykonywaniem tej samej funkcji dla
// dwóch klawiszy wciskanych jednocześnie (zawsze będzie
// wykonywana odpowiednia akcja dla tego, który został
// wciśnięty jako pierwszy
if( last_key && last_key != key_mask ) return;

key_press = !(*KPIN & key_mask);

if( key_press && !key_state ) {
key_state = debounce;
Timer1 = 5;
} else
if( key_state ) {
if( key_press && debounce==key_state && !Timer1 ) {
key_state = go_rep;
Timer1=3;
last_key = key_mask;
} else
if( !key_press && key_state>debounce && key_stateif(push_proc) push_proc();/* KEY_UP */
key_state=idle;
last_key = 0;
} else
if( key_press && go_rep==key_state && !Timer1 ) {
if(!rep_time) rep_time=20;
if(!rep_wait) rep_wait=150;
key_state = wait_rep;
Timer1=rep_wait;
} else
if( key_press && wait_rep==key_state && !Timer1 ) {
key_state = rep;
} else
if( key_press && rep==key_state && !Timer1 ) {
Timer1 = rep_time;
if(rep_proc) rep_proc();/* KEY_REP */
}
}

if( key_state>=wait_rep && !key_press ) {
key_state = idle;
last_key = 0;
}
}

void kl1_press(void){
element_menu++;
lcd_cls();
s_pomiary= 50;

}
void kl1_rep(void){

}
void kl2_press(void){
s_pomiary = element_menu;
}

void kl2_rep(void){

}

void test() {
if(!Timer3) {
Timer3 = 75;
PORTD ^= LED;
}

}

ISR(TIMER2_COMP_vect) {
uint16_t n;

n = Timer1;
if(n)
Timer1 = --n;

n = Timer2;
if(n)
Timer2 = --n;

n = Timer3;
if(n)
Timer3 = --n;
}[/syntax]
Nie mam pomysłu jak to składnie zrobić Jakieś sugestie?

Statystyki: Napisane przez dawidek034 — 12 cze 2014, o 00:12

Re: Kłopot z przerwaniami

2014-06-11T19:33:13+01:00

[syntax=c]#include
#include
#include "LCD/lcd44780.h"
#include "1Wire/ds18x20.h"
#include

#define KEY3 (1<#define KEY2 (1<#define LED (1<#define LED1 (1<volatile uint16_t Timer1, Timer2, Timer3;

void superDebounce(volatile uint8_t *KPIN,
uint8_t key_mask, uint16_t rep_time, uint16_t rep_wait,
void (*push_proc)(void), void (*rep_proc)(void) );
void kl1_press(void);
void kl1_rep(void);
void kl2_press(void);
void kl2_rep(void);
void show_menu();
void test();
uint8_t element_menu = 0;

enum menu { mainMenu, mpomiar, mwartosci };
enum menu s_menu ;

int main(void) {

DDRD |= LED | LED1;
PORTD &=~ LED | LED1;

PORTC |= KEY3 | KEY2;
TCCR2 |= (1<TCCR2 |= (1<OCR2 = 108;
TIMSK = (1<
lcd_init();
sei();

while(1) {
superDebounce(&PINC, KEY3, 20, 500, kl1_press, kl1_rep );
superDebounce(&PINC, KEY2, 80, 400, kl2_press, kl2_rep );

show_menu();
if(element_menu > 2)
element_menu = 0;
test(); // test czy przerwania sie zaklocaja
}
}

void show_menu() {
switch(element_menu) {
case mainMenu:
lcd_locate(0,5);
lcd_str("MENU");
break;
case mpomiar:
lcd_locate(0,2);
lcd_str("Pomiar_temp");
break;
case mwartosci:
lcd_locate(0,4);
lcd_str("Wartosci");
if(!Timer2) {
Timer2 = 20;
PORTD ^= LED1;
}
break;
}

}

void superDebounce(volatile uint8_t *KPIN,
uint8_t key_mask, uint16_t rep_time, uint16_t rep_wait,
void (*push_proc)(void), void (*rep_proc)(void) ) {

enum KS {idle, debounce, go_rep, wait_rep, rep};

static enum KS key_state;
static uint8_t last_key;
uint8_t key_press;

// zabezpieczenie przed wykonywaniem tej samej funkcji dla
// dwóch klawiszy wciskanych jednocześnie (zawsze będzie
// wykonywana odpowiednia akcja dla tego, który został
// wciśnięty jako pierwszy
if( last_key && last_key != key_mask ) return;

key_press = !(*KPIN & key_mask);

if( key_press && !key_state ) {
key_state = debounce;
Timer1 = 5;
} else
if( key_state ) {
if( key_press && debounce==key_state && !Timer1 ) {
key_state = go_rep;
Timer1=3;
last_key = key_mask;
} else
if( !key_press && key_state>debounce && key_stateif(push_proc) push_proc();/* KEY_UP */
key_state=idle;
last_key = 0;
} else
if( key_press && go_rep==key_state && !Timer1 ) {
if(!rep_time) rep_time=20;
if(!rep_wait) rep_wait=150;
key_state = wait_rep;
Timer1=rep_wait;
} else
if( key_press && wait_rep==key_state && !Timer1 ) {
key_state = rep;
} else
if( key_press && rep==key_state && !Timer1 ) {
Timer1 = rep_time;
if(rep_proc) rep_proc();/* KEY_REP */
}
}

if( key_state>=wait_rep && !key_press ) {
key_state = idle;
last_key = 0;
}
}

void kl1_press(void){
lcd_cls();
element_menu++;

}
void kl1_rep(void){

}
void kl2_press(void){

}
void kl2_rep(void){

}

void test() {
if(!Timer3) {
Timer3 = 75;
PORTD ^= LED;
}

}

ISR(TIMER2_COMP_vect) {
uint16_t n;

n = Timer1;
if(n)
Timer1 = --n;

n = Timer2;
if(n)
Timer2 = --n;

n = Timer3;
if(n)
Timer3 = --n;
}[/syntax]

Wprowadziłem typ wyliczeniowy jak zaproponowałeś i działa tak jak chciałem. Miałeś racje źle się do tego zabierałem. Dziękuje za pomoc. Teraz mam pytanie jak powiązać tą zmianę stanów z jakimś prostym menu. Czyli jak ja to widzę przykładowo:
1.poruszam się klawiszem pod opcjach menu(pomiar_temp; godzina; wilgotność)
2.gdy jestem na konkretnej pozycji z menu, drugim klawiszem wybieram daną opcje i przechodzę jak powyżej do pracy ciągłej pomiaru temperatury

Trochę próbowałem jakoś powiązać dwie konstrukcje switch() ze sobą, ale zbytni mi to nie wychodzi. Jak inaczej się do tego zabrać? Powyższy kod różni się od pierwotnego, lecz idea pozostaje ta sama(zastosowałem inne funkcje)

Statystyki: Napisane przez dawidek034 — 11 cze 2014, o 19:33

Kłopot z przerwaniami

2014-06-10T20:01:20+01:00

Problem polega na tym iż chciałbym za pomocą klawisza włączyć daną funkcję tak aby tak działała w sposób ciągły. Przykładowo pomiar temperatury: naciskając KEY3 chciałbym zobaczyć na lcd, wynik pomiaru, ale nie pojedynczego tylko trwającego aż do odwołania przykładowo innym przyciskiem. Następnie takie rozwiązanie chciałbym zaimplementować do jakiegoś prostego menu. Siedzę już nad tym dość sporo ale najwyraźniej czegoś nie rozumiem.
Bardzo proszę o wskazówki i porady dotyczącego tego problemu jak kodu. Co warto w nim zmienić.
[syntax=c]#include
#include
#include "LCD/lcd44780.h"
#include "1Wire/ds18x20.h"
#include

#define KEY1 (1<#define KEY2 (1<#define KEY3 (1<
typedef struct {

volatile uint8_t *KPIN;
uint8_t wait_time_s;
uint8_t key_mask;
void(*kfun1)(void);
void(*kfun2)(void);
uint8_t klock;
uint8_t flag;

}TBUTTON;

volatile uint16_t TIMER1, TIMER2, TIMER3;
volatile uint8_t sekundy, ms10_cnt, s1_flag;
uint8_t czujniki_cnt;
uint8_t subzero, cel, cel_fract_bits;

void pomiar_temp();
void temp_value(void);
void key_press(TBUTTON *btn);
void timer_conf(void);
void button_value();
void display_temp(uint8_t x);
void button_value1();
void moj_klawisz(void);

//Definicja klawisza
TBUTTON button;
TBUTTON button1;

int main(void) {

PORTC |= (KEY1) | (KEY2) | (KEY3);

timer_conf();
button_value();
button_value1();
lcd_init();
sei();

while(1) {

key_press(&button);
key_press(&button1);

} //end while

}
void key_press(TBUTTON * btn) {
register uint16_t key_press = (*btn -> KPIN & btn -> key_mask);
if( !btn->klock && !key_press ) {
btn->klock=1;

// reakcja na PRESS krótkie wcinięcie klawisza
if(btn->kfun1) btn->kfun1();
btn->flag=1;
TIMER1 = (btn->wait_time_s*1000)/10;

}
else if( btn->klock && key_press ) {
(btn->klock)++;
if( !btn->klock ) {
TIMER1 = 0;
btn->flag = 0;
}
} else if( btn->flag && !TIMER1 ) {
// reakcja na dłuższe wcinięcie klawisza
if(btn->kfun2) btn->kfun2();
btn->flag=0;
}
}

void temp_value(void) {

if(DS18X20_OK == DS18X20_read_meas(gSensorIDs[0], &subzero, &cel, &cel_fract_bits)) {
lcd_locate(1,0);
lcd_str(" ");
display_temp(1);
TIMER2 = 100;
}

else {
lcd_locate(1,0);
lcd_str("error");
}
}
void pomiar_temp(void) {
czujniki_cnt = search_sensors();
DS18X20_start_meas( DS18X20_POWER_EXTERN, NULL );

}

void display_temp(uint8_t x) {
x = 1;
lcd_locate(1,x);
if(subzero) lcd_str("-");/* jeśli subzero==1 wyświetla znak minus (temp. ujemna) */
else lcd_str(" ");/* jeśli subzero==0 wyświetl spację zamiast znaku minus (temp. dodatnia) */
lcd_int(cel);/* wyświetl dziesiętne części temperatury */
lcd_str(".");/* wyświetl kropkę */
lcd_int(cel_fract_bits); /* wyświetl dziesiętne części stopnia */

//lcd_str(" \x80""C"); /* wyświetl znak jednostek (C - stopnie Celsiusza) */
}

void timer_conf(void) {
TCCR2 |= (1<TCCR2 |= (1<OCR2 = 108;
TIMSK = (1<}

void button_value() {
button.KPIN = &PINC;
button.key_mask = KEY1;
button.wait_time_s = 2;
button.kfun1 = temp_value ;
button.kfun2 = NULL;
}
void button_value1() {
button1.KPIN = &PINC;
button1.key_mask = KEY2;
button1.wait_time_s = 2;
button1.kfun1 = NULL;
button1.kfun2 = NULL;
}

/**Przerwania_TIMER2**/
ISR(TIMER2_COMP_vect) {

uint16_t n;

n = TIMER1;
if(n)
TIMER1 = --n; //100Hz

n = TIMER2;
if(n)
TIMER2 = --n; //100Hz

n = TIMER3;
if(n)
TIMER3 = --n;//100Hz

}[/syntax]

Statystyki: Napisane przez dawidek034 — 10 cze 2014, o 20:01