ATNEL tech-forum 2018-04-26T09:12:05+01:00 https://forum.atnel.pl/feed.php?f=58&t=19592&mode 2018-04-26T09:12:05+01:00 2018-04-26T09:12:05+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=207155#p207155 <![CDATA[Re: Obsługa klawiszy]]>
Wydaje mi się, że było tak jak mówiłem czyli Timery sobie tak jakby przeszkadzały. Nie wiem czy dobrze myślę, ale właśnie tak myśląc podszedłem do tego inaczej.

Zainicjowałem timer wewnątrz funkcji "void change_color(void);" i teraz nie ma żadnych mignięć diody, tylko normalnie świeci z zadanym wypełnieniem zmiennych pwm_r, pwm_g, pwm_b 0-255. Przy krótkim wciśnięciu przycisku zmieniają się kolory oraz przy długim wciśnięciu zeruję zmienne pwm_r, pwm_g, pwm_b tak jak zaproponował rskup.

Wszystko działa tak jak chciałem. Dziękuję za zaangażowanie w temat :)

Statystyki: Napisane przez Tom92 — 26 kwi 2018, o 09:12

]]> 2018-04-22T23:12:03+01:00 2018-04-22T23:12:03+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=207061#p207061 <![CDATA[Re: Obsługa klawiszy]]> Jak chcesz wyłączać LEDy to wyzeruj wartości w zmiennych pwm_r, pwm_g, pwm_b lub dodaj dodatkową zmienną pwm_on_off (lub zamień istniejącą statyczną zmiennej flaga) i ją zmieniaj w momencie zmiany stanu ON/OFF oraz sprawdzaj na jej podstawie w przerwaniu Timer1 czy możesz zmieniać stan LEDów.

--
Pozdrawiam,
Robert

Statystyki: Napisane przez rskup — 22 kwi 2018, o 23:12

]]> 2018-04-20T09:24:56+01:00 2018-04-20T09:24:56+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=206986#p206986 <![CDATA[Re: Obsługa klawiszy]]> Statystyki: Napisane przez Tom92 — 20 kwi 2018, o 09:24

]]> 2018-04-19T08:41:20+01:00 2018-04-19T08:41:20+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=206962#p206962 <![CDATA[Re: Obsługa klawiszy]]> rskup napisał(a):

Zrób nowy projekt i wrzuć w nim obsługę Timer1 i miganie jedna diodą. Dzięki temu sprawdzisz sobie czy na pewno dobrze inicjalizujesz Timer1

Sorki, ale nie mam takiego doświadczenia dlatego nie przyszło mi na myśl, żeby tak sobie to sprawdzić. Dobrze, więc zrobiłem jak zasugerowałeś i faktycznie, źle inicjowałem Timer1. Powinno być tak:

[syntax=c]/* Timer1 - PWM */
TCCR1B |= (1<<WGM12);// tryb CTC
TCCR1B |= (1<<CS10);// preskaler = 1
OCR1A = 40000;// przerwanie co ok. 5 ms (200Hz)
TIMSK = (1<<OCIE1A);// Odblokowanie przerwania CompareMatch[/syntax]

oraz wektor A, a nie B:

[syntax=c]/* przerwanie Timer0 CompareM */
ISR(TIMER1_COMPA_vect) {

// przykładowy kod

}[/syntax]

Teraz wrzuciłem ten Timer do mojego projektu i kolory się wyświetlają, mogę przyciskiem je przełączać, ALE zauważyłem, że co ok. 1 sekunde dioda RGB lekko mrugnie i jest brak reakcji na dłuższe przytrzymanie klawisza.

Mam wrażenie jakby Timery przeszkadzały sobie na wzajem? Nie wiem jak to rozwiązać. Poniżej zamieszczam aktualny kod jakby ktoś był miły zerknąć.

[syntax=c]#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>

/* konfiguracja pinów podłączonej diody LED RGB */
#define LEDR (1<<PB2)// definicja pinu R
#define LEDG (1<<PB0)// definicja pinu G
#define LEDB (1<<PB1)// definicja pinu B
#define LEDR_OFF PORTB |= LEDR// makrodefinicja - wyłączenie pinu R
#define LEDG_OFF PORTB |= LEDG// makrodefinicja - wyłączenie pinu G
#define LEDB_OFF PORTB |= LEDB// makrodefinicja - wyłączenie pinu B
#define LEDR_ON PORTB &= ~LEDR// makrodefinicja - załączenie pinu R
#define LEDG_ON PORTB &= ~LEDG// makrodefinicja - załączenie pinu G
#define LEDB_ON PORTB &= ~LEDB// makrodefinicja - załączenie pinu B
#define LEDS_OFF PORTB |= (LEDR|LEDG|LEDB)// makrodefinicja - wyłączenie całej taśmy
#define LEDS_ON PORTB &= ~(LEDR|LEDG|LEDB)// makrodefinicja - załączenie całej taśmy

/* konfiguracja pinu podłączonego klawisza */
#define KEY1 (1<<PD5)
// **********************************************************

/* struktura danych */
typedef struct {
volatile uint8_t *KPIN;
uint8_t key_mask;
uint8_t wait_time_s;
void (*kfun1)(void);
void (*kfun2)(void);
uint16_t klock;
uint8_t flag;
} TBUTTON;

TBUTTON button; // definicja KLAWISZA

/* timer programowy */
volatile uint16_t Timer0;

volatile uint8_t pwm_r, pwm_g, pwm_b;

/* definicje funckcji */
void change_color(void);// funkcja zmieniająca kolor taśmy LED RGB
void change_ON_OFF(void);// funkcja załączająca i wyłączająca taśmę LED
void key_press( TBUTTON * btn );// funkcja obsługi pojedynczych klawiszy
void set_color(uint8_t red, uint8_t green, uint8_t blue);

//------------------------------------------------------------------

int main(void) {

PORTD |= KEY1; // podciągamy linie klawiszy do VCC
DDRB |= LEDR|LEDG|LEDB; // piny LED jako WYjścia

/* Timer0 – konfigurcajca silnika timerów programowych */
TCCR0A |= (1<<WGM01); // tryb pracy CTC
TCCR0B |= (1<<CS02)|(1<<CS00); // preskaler = 1024
OCR0A = 78; // przerwanie porównania co ok. 10ms (100,16Hz)
TIMSK = (1<<OCIE0A); // Odblokowanie przerwania CompareMatch

/* Timer1 - PWM */
TCCR1B |= (1<<WGM12);// tryb CTC
TCCR1B |= (1<<CS10);// preskaler = 1
OCR1A = 40000;// przerwanie co ok. 5 ms (200Hz)
TIMSK = (1<<OCIE1A);// Odblokowanie przerwania CompareMatch

button.KPIN = &PIND;
button.key_mask = KEY1;
button.wait_time_s = 1;
button.kfun1 = change_color;
button.kfun2 = change_ON_OFF;

sei();

while(1) {

key_press( &button );

}
}
//------------------------------------------------------------------



/* przerwanie Timer0 CompareM */
ISR(TIMER0_COMPA_vect) {

uint16_t n;

n = Timer0; /* 100,16Hz Timer1 */
if (n) Timer0 = --n;// odliczanie czasu wciśniętego klawisza

}



/* przerwanie Timer1 CompareM */
ISR(TIMER1_COMPA_vect) {

static uint8_t cnt;// definicja zmiennej statycznej licznika PWM

// bezpośrednie sterowanie wyjściami kanałów PWM
if(cnt>=pwm_r) LEDR_OFF; else LEDR_ON;
if(cnt>=pwm_g) LEDG_OFF; else LEDG_ON;
if(cnt>=pwm_b) LEDB_OFF; else LEDB_ON;

cnt++;// zwiększanie licznika o 1

}



/* funkcja do obsługi pojedynczych klawiszy */
void key_press( TBUTTON * btn )
{
register uint8_t key_press = (*btn->KPIN & btn->key_mask);

if( !btn->klock && !key_press )
{
btn->klock=50000;
btn->flag=1;
Timer0 = (btn->wait_time_s*1000)/10;
}
else if( btn->klock && key_press )
{
(btn->klock)++;
if( !btn->klock && btn->flag )
{
// reakcja na krótkie wcinięcie klawisza - reakcja po zwolnieniu przycisku
if(btn->kfun1) btn->kfun1();
Timer0=0;
btn->flag=0;
}
} else if( btn->flag && !Timer0 )
{
// reakcja na dłuższe wciniecie klawisza
if(btn->kfun2) btn->kfun2();
btn->flag=0;
}
}



/* własna funkcja - zmiana koloru taśmy */
void change_color(void) {
// stan=0, zmienna statyczna czyli zachowuje się jak zmienna globalna, ale dostępna jest tylko w tej funkcji
static uint8_t stan;// zmienna statyczna
stan++;// zwiększanie o 1
if (stan>6) stan=0;// zmienna stan może przyjąc wortości od 0 do 6

if (stan==0) {// biały
set_color( 255, 255, 255 );
}
else if (stan==1) {// czerwony

set_color( 255, 0, 0 );
}
else if (stan==2) {// zielony
set_color( 0, 255, 0 );
}
else if (stan==3) {// niebieski
set_color( 0, 0, 255 );
}
else if (stan==4) {// fioletowy
set_color( 255, 0, 255 );
}
else if (stan==5) {// żółty
set_color( 255, 255, 0 );
}
else {// błękitny
set_color( 0, 255, 255 );
}
}



/* własna funkcja - wyłączenie/załączenie taśmy */
void change_ON_OFF(void) {
// flaga=0, zmienna statyczna czyli zachowuje się jak zmienna globalna, ale dostępna jest tylko w tej funkcji
static uint8_t flaga;
if (flaga) {// jeżeli flaga jest równa 1 (lub różna od 0) to włącz diody i wyzeruj tą flagę (flaga=0)
LEDS_ON;
flaga=0;
}
else {// natomiast jeżeli flaga jest równa 0 to wyłącz diody i ustaw flagę (flaga=1)
LEDS_OFF;
flaga=1;
}
}



/* funkcja ustawiająca wybrany kolor diody RGB */
void set_color(uint8_t red, uint8_t green, uint8_t blue)
{
if ( pwm_r > red ) while (pwm_r > red) {pwm_r--;}
else if ( pwm_r < red ) while (pwm_r < red) {pwm_r++;}

if ( pwm_g > green ) while (pwm_g > green) {pwm_g--;}
else if ( pwm_g < green ) while (pwm_g < green) {pwm_g++;}

if ( pwm_b > blue ) while (pwm_b > blue) {pwm_b--;}
else if ( pwm_b < blue ) while (pwm_b < blue) {pwm_b++;}
}[/syntax]

Statystyki: Napisane przez Tom92 — 19 kwi 2018, o 08:41

]]> 2018-04-18T14:49:14+01:00 2018-04-18T14:49:14+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=206936#p206936 <![CDATA[Re: Obsługa klawiszy]]> rskup napisał(a):

Zrób nowy projekt i wrzuć w nim obsługę Timer1 i miganie jedna diodą. Dzięki temu sprawdzisz sobie czy na pewno dobrze inicjalizujesz Timer1 (używasz dobre rejestry i wartości w nich) i korzystasz z jego przerwań. Jak Ci zadziała to należy szukać dalej w kolejnych częściach kodu.

I to trzeba powtarzać młodym adeptom programowania do znudzenia ... ;)

Statystyki: Napisane przez mirekk36 — 18 kwi 2018, o 14:49

]]> 2018-04-18T14:45:01+01:00 2018-04-18T14:45:01+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=206935#p206935 <![CDATA[Re: Obsługa klawiszy]]> Tom92 napisał(a):

Timerze1 też mi nic nie chciało działać
Jest taki prosty sprawdzony sposób do rozwiązywania takich problemów. O nim Mirek też ciągle wspomina ...
Zrób nowy projekt i wrzuć w nim obsługę Timer1 i miganie jedna diodą. Dzięki temu sprawdzisz sobie czy na pewno dobrze inicjalizujesz Timer1 (używasz dobre rejestry i wartości w nich) i korzystasz z jego przerwań. Jak Ci zadziała to należy szukać dalej w kolejnych częściach kodu.

--
Pozdrawiam,
Robert

Statystyki: Napisane przez rskup — 18 kwi 2018, o 14:45

]]> 2018-04-18T09:53:30+01:00 2018-04-18T09:53:30+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=206930#p206930 <![CDATA[Re: Obsługa klawiszy]]> Wróciłem do tego projektu, ponieważ chciałem go trochę ulepszyć o sterowanie PWM diodą RGB. Gdy do przerwania dodałem sterowanie wyjściami tak:

[syntax=c]/* przerwanie Timer0 CompareM */
ISR(TIMER0_COMPA_vect) {

uint16_t n;

n = Timer0; /* 100,16Hz Timer1 */
if (n) Timer0 = --n;// odliczanie czasu wciśniętego klawisza

static uint8_t cnt;// definicja zmiennej statycznej licznika PWM

// bezpośrednie sterowanie wyjściami kanałów PWM
if(cnt>=pwm_r) LEDR_OFF; else LEDR_ON;
if(cnt>=pwm_g) LEDG_OFF; else LEDG_ON;
if(cnt>=pwm_b) LEDB_OFF; else LEDB_ON;

cnt++;// zwiększanie licznika o 1

}[/syntax]

to jest brak reakcji na dłuższe przytrzymanie klawisza oraz dioda co jakiś czas mignie. Natomiast jak dodałem kolejny Timera1 to teraz nic nie działa. Możliwe, że może mieć to związek z tym, że wcześniej w tym temacie na Timerze1 też mi nic nie chciało działać? Ale dlaczego? Może ktoś będzie miał pomysł dlaczego tak jest? Poniżej zamieszczam cały kod:

[syntax=c]#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>

/* konfiguracja pinów podłączonej diody LED RGB */
#define LEDR (1<<PB2)// definicja pinu R
#define LEDG (1<<PB0)// definicja pinu G
#define LEDB (1<<PB1)// definicja pinu B
#define LEDR_OFF PORTB |= LEDR// makrodefinicja - wyłączenie pinu R
#define LEDG_OFF PORTB |= LEDG// makrodefinicja - wyłączenie pinu G
#define LEDB_OFF PORTB |= LEDB// makrodefinicja - wyłączenie pinu B
#define LEDR_ON PORTB &= ~LEDR// makrodefinicja - załączenie pinu R
#define LEDG_ON PORTB &= ~LEDG// makrodefinicja - załączenie pinu G
#define LEDB_ON PORTB &= ~LEDB// makrodefinicja - załączenie pinu B
#define LEDS_OFF PORTB |= (LEDR|LEDG|LEDB)// makrodefinicja - wyłączenie całej taśmy
#define LEDS_ON PORTB &= ~(LEDR|LEDG|LEDB)// makrodefinicja - załączenie całej taśmy

/* konfiguracja pinu podłączonego klawisza */
#define KEY1 (1<<PD5)
// **********************************************************

/* struktura danych */
typedef struct {
volatile uint8_t *KPIN;
uint8_t key_mask;
uint8_t wait_time_s;
void (*kfun1)(void);
void (*kfun2)(void);
uint16_t klock;
uint8_t flag;
} TBUTTON;

TBUTTON button; // definicja KLAWISZA

/* timer programowy */
volatile uint16_t Timer0;

volatile uint8_t pwm_r, pwm_g, pwm_b;

/* definicje funckcji */
void change_color(void);// funkcja zmieniająca kolor taśmy LED RGB
void change_ON_OFF(void);// funkcja załączająca i wyłączająca taśmę LED
void key_press( TBUTTON * btn );// funkcja obsługi pojedynczych klawiszy
void set_color(uint8_t red, uint8_t green, uint8_t blue);

//------------------------------------------------------------------

int main(void) {

PORTD |= KEY1; // podciągamy linie klawiszy do VCC
DDRB |= LEDR|LEDG|LEDB; // piny LED jako WYjścia

/* Timer0 – konfigurcajca silnika timerów programowych */
TCCR0A |= (1<<WGM01); // tryb pracy CTC
TCCR0B |= (1<<CS02)|(1<<CS00); // preskaler = 1024
OCR0A = 78; // przerwanie porównania co ok. 10ms (100,16Hz)
TIMSK = (1<<OCIE0A); // Odblokowanie przerwania CompareMatch

/* Timer1 - PWM */
TCCR1B |= (1<<WGM12);// tryb CTC
TCCR1B |= (1<<CS10);// preskaler = 1
OCR1BL = 200;
TIMSK = (1<<OCIE1B);// Odblokowanie przerwania CompareMatch

button.KPIN = &PIND;
button.key_mask = KEY1;
button.wait_time_s = 1;
button.kfun1 = change_color;
button.kfun2 = change_ON_OFF;

sei();

while(1) {

key_press( &button );

// przykładowy niezależny i nieblokujący proces
// if( !Timer1 ) {
// Timer1=50;
// PORTC ^= LED4;
// }
}
}
//------------------------------------------------------------------



/* przerwanie Timer0 CompareM */
ISR(TIMER0_COMPA_vect) {

uint16_t n;

n = Timer0; /* 100,16Hz Timer1 */
if (n) Timer0 = --n;// odliczanie czasu wciśniętego klawisza

}



/* przerwanie Timer0 CompareM */
ISR(TIMER1_COMPB_vect) {

static uint8_t cnt;// definicja zmiennej statycznej licznika PWM

// bezpośrednie sterowanie wyjściami kanałów PWM
if(cnt>=pwm_r) LEDR_OFF; else LEDR_ON;
if(cnt>=pwm_g) LEDG_OFF; else LEDG_ON;
if(cnt>=pwm_b) LEDB_OFF; else LEDB_ON;

cnt++;// zwiększanie licznika o 1

}



/* funkcja do obsługi pojedynczych klawiszy */
void key_press( TBUTTON * btn )
{
register uint8_t key_press = (*btn->KPIN & btn->key_mask);

if( !btn->klock && !key_press )
{
btn->klock=50000;
btn->flag=1;
Timer0 = (btn->wait_time_s*1000)/10;
}
else if( btn->klock && key_press )
{
(btn->klock)++;
if( !btn->klock && btn->flag )
{
// reakcja na krótkie wcinięcie klawisza - reakcja po zwolnieniu przycisku
if(btn->kfun1) btn->kfun1();
Timer0=0;
btn->flag=0;
}
} else if( btn->flag && !Timer0 )
{
// reakcja na dłuższe wciniecie klawisza
if(btn->kfun2) btn->kfun2();
btn->flag=0;
}
}



/* własna funkcja - zmiana koloru taśmy */
void change_color(void) {
// stan=0, zmienna statyczna czyli zachowuje się jak zmienna globalna, ale dostępna jest tylko w tej funkcji
static uint8_t stan;// zmienna statyczna
stan++;// zwiększanie o 1
if (stan>6) stan=0;// zmienna stan może przyjąc wortości od 0 do 6

if (stan==0) {// biały
set_color( 255, 255, 255 );
}
else if (stan==1) {// czerwony

set_color( 255, 0, 0 );
}
else if (stan==2) {// zielony
set_color( 0, 255, 0 );
}
else if (stan==3) {// niebieski
set_color( 0, 0, 255 );
}
else if (stan==4) {// fioletowy
set_color( 255, 0, 255 );
}
else if (stan==5) {// żółty
set_color( 255, 255, 0 );
}
else {// błękitny
set_color( 0, 255, 255 );
}
}



/* własna funkcja - wyłączenie/załączenie taśmy */
void change_ON_OFF(void) {
// flaga=0, zmienna statyczna czyli zachowuje się jak zmienna globalna, ale dostępna jest tylko w tej funkcji
static uint8_t flaga;
if (flaga) {// jeżeli flaga jest równa 1 (lub różna od 0) to włącz diody i wyzeruj tą flagę (flaga=0)
LEDS_ON;
flaga=0;
}
else {// natomiast jeżeli flaga jest równa 0 to wyłącz diody i ustaw flagę (flaga=1)
LEDS_OFF;
flaga=1;
}
}



/* funkcja ustawiająca wybrany kolor diody RGB */
void set_color(uint8_t red, uint8_t green, uint8_t blue)
{
if ( pwm_r > red ) while (pwm_r > red) {pwm_r--;}
else if ( pwm_r < red ) while (pwm_r < red) {pwm_r++;}

if ( pwm_g > green ) while (pwm_g > green) {pwm_g--;}
else if ( pwm_g < green ) while (pwm_g < green) {pwm_g++;}

if ( pwm_b > blue ) while (pwm_b > blue) {pwm_b--;}
else if ( pwm_b < blue ) while (pwm_b < blue) {pwm_b++;}
}[/syntax]

Statystyki: Napisane przez Tom92 — 18 kwi 2018, o 09:53

]]> 2017-11-13T20:22:50+01:00 2017-11-13T20:22:50+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=198884#p198884 <![CDATA[Re: Obsługa klawiszy]]> [syntax=c]uint8_t klock;[/syntax]
na
[syntax=c]uint16_t klock;[/syntax]
skoro inicjujesz ją w ten sposób:
[syntax=c]btn->klock=50000;[/syntax]

Wartość 50000 raczej się nie zmieści w zmiennej 8 bitowej. ;)

Statystyki: Napisane przez anshar — 13 lis 2017, o 20:22

]]> 2017-11-13T20:16:22+01:00 2017-11-13T20:16:22+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=198883#p198883 <![CDATA[Re: Obsługa klawiszy]]>
[syntax=c]/* Timer0 – konfigurcajca silnika timerów programowych */
TCCR0A |= (1<<WGM01); // tryb pracy CTC
TCCR0B |= (1<<CS02)|(1<<CS00); // preskaler = 1024
OCR0A = 78; // przerwanie porównania co ok. 10ms (100,16Hz)
TIMSK = (1<<OCIE0A); // Odblokowanie przerwania CompareMatch[/syntax]

Teraz działa prawidłowo. Ale jestem ciekawy dlaczego na Timerze1 nie chciało działać? Źle skonfigurowałem czy coś innego miało na to wpływ np. pin do którego podłączyłem klawisz?

Statystyki: Napisane przez Tom92 — 13 lis 2017, o 20:16

]]> 2017-11-13T17:37:59+01:00 2017-11-13T17:37:59+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=198876#p198876 <![CDATA[Re: Obsługa klawiszy]]>

Nie rozumiem, ten sam kod na atmedze32 działa normalnie a natomiast na attiny2313 nie chce działać, dziwne rzeczy się dzieją. Jedyne co zmieniałem to Timer1:

[syntax=c]#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>


//// ************* definicje dla preprocesora
// dioda LED RGB
#define LED1 (1<<PB0)// definicja pinu do którego podłączona jest dioda R
#define LED2 (1<<PB1)// definicja pinu do którego podłączona jest dioda G
#define LED3 (1<<PB2)// definicja pinu do którego podłączona jest dioda B
#define LED1_OFF PORTB |= LED1// makrodefinicja - wyłączenie diody
#define LED2_OFF PORTB |= LED2
#define LED3_OFF PORTB |= LED3
#define LED1_ON PORTB &= ~LED1// makrodefinicja - załączenie diody
#define LED2_ON PORTB &= ~LED2
#define LED3_ON PORTB &= ~LED3
#define LEDS_OFF PORTB |= (LED1|LED2|LED3)// makrodefinicja - wyłączenie całej taśmy
#define LEDS_ON PORTB &= ~(LED1|LED2|LED3)// makrodefinicja - załączenie całej taśmy

// klawisz K1
#define KEY1 (1<<PD5)

typedef struct {
volatile uint8_t *KPIN;
uint8_t key_mask;
uint8_t wait_time_s;
void (*kfun1)(void);
void (*kfun2)(void);
uint8_t klock;
uint8_t flag;
} TBUTTON;

// timery programowe
volatile uint16_t Timer1;

// własne funkcje użytkownika
void change_led1(void) {
// flaga=0, zmienna statyczna czyli zachowuje się jak zmienna globalna, ale dostępna jest tylko w tej funkcji
static uint8_t stan;
stan++;
if (stan>3) stan=0;
if (stan==0) {
LED1_ON;
LED2_OFF;
LED3_OFF;
}
else if (stan==1) {
LED1_OFF;
LED2_ON;
LED3_OFF;
}
else if (stan==2) {
LED1_OFF;
LED2_OFF;
LED3_ON;
}
else if (stan==3) {
LED1_ON;
LED2_ON;
LED3_ON;
}
}

void change_led2(void) {
// flaga=0, zmienna statyczna czyli zachowuje się jak zmienna globalna, ale dostępna jest tylko w tej funkcji
static uint8_t flaga;

if (flaga) {// jeżeli flaga jest równa 1 (lub różna od 0) to włącz diody i wyzeruj tą flagę (flaga=0)
LEDS_ON;
flaga=0;
}
else {// natomiast jeżeli jest równa 0 to wyłącz diody i ustaw flagę (flaga=1)
LEDS_OFF;
flaga=1;
}
}
// LUB TAK //
//static uint8_t flaga;
//
//if(flaga) LEDS_ON;
//else LEDS_OFF;
//
//flaga = !flaga;


// funkcja obsługi pojedynczych klawiszy
void key_press( TBUTTON * btn );


TBUTTON button; // definicja KLAWISZA

//------------------------------------------------------------------
int main(void) {

PORTD |= KEY1; // podciągamy linie klawiszy do VCC
DDRB |= LED1|LED2|LED3; // piny LED jako WYjścia

/* Timer1 – konfigurcajca silnika timerów programowych */
TCCR1B |= (1<<WGM12); // tryb pracy CTC
TCCR1B |= (1<<CS12)|(1<<CS10); // preskaler = 1024
OCR1B = 78; // przerwanie porównania co ok. 10ms (100,16Hz)
TIMSK = (1<<OCIE1A); // Odblokowanie przerwania CompareMatch

button.KPIN = &PIND;
button.key_mask = KEY1;
button.wait_time_s = 5;
button.kfun1 = change_led1;
button.kfun2 = change_led2;

sei();

while(1) {

key_press( &button );

// przykładowy niezależny i nieblokujący proces
// if( !Timer1 ) {
// Timer1=50;
// PORTC ^= LED4;
// }

}

}
//------------------------------------------------------------------

void key_press( TBUTTON * btn )
{
register uint8_t key_press = (*btn->KPIN & btn->key_mask);

if( !btn->klock && !key_press )
{
btn->klock=50000;
btn->flag=1;
Timer1 = (btn->wait_time_s*1000)/10;
}
else if( btn->klock && key_press )
{
(btn->klock)++;
if( !btn->klock && btn->flag )
{
// reakcja na krótkie wcinięcie klawisza - reakcja po zwolnieniu przycisku
if(btn->kfun1) btn->kfun1();
Timer1=0;
btn->flag=0;
}
} else if( btn->flag && !Timer1 )
{
// reakcja na dłuższe wciniecie klawisza
if(btn->kfun2) btn->kfun2();
btn->flag=0;
}
}

//*** przerwanie Timer1 CompareM
ISR(TIMER1_COMPA_vect) {

uint16_t n;

n = Timer1; /* 100,16Hz Timer1 */
if (n) Timer1 = --n;
// n = Timer2; /* 100Hz Timer2 */
// if (n) Timer2 = --n;
}[/syntax]

------------------------ [ Dodano po: 53 minutyach ]

Czy powodem może być rezonator kwarcowy?

Teraz mam ustawione wewnętrzne taktowanie 8MHz, mam wlutować rezonator 11,0592MHz i tak ustawić timer na przerwania co równe 100Hz tak jak w kodzie superdebounce na blogu Pana Mirka? Bo teraz jak liczyłem to mam 100,16Hz. Ale to chyba znaczenia nie ma? Bo jak wgrywałem kod do Atmegi32 to mam ustawione 8MHz i działa. A jak przeniose do Attiny2313 to głupoty się dzieją.

Nie mam już pomysłów :(

Statystyki: Napisane przez Tom92 — 13 lis 2017, o 17:37

]]> 2017-11-13T17:02:40+01:00 2017-11-13T17:02:40+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=198872#p198872 <![CDATA[Re: Obsługa klawiszy]]> Statystyki: Napisane przez Marhef — 13 lis 2017, o 17:02

]]> 2017-11-13T16:41:55+01:00 2017-11-13T16:41:55+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=198870#p198870 <![CDATA[Re: Obsługa klawiszy]]>
[syntax=c]#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>


//// ************* definicje dla preprocesora
// dioda LED RGB
#define LED1 (1<<PB0)// definicja pinu do którego podłączona jest dioda R
#define LED2 (1<<PB1)// definicja pinu do którego podłączona jest dioda G
#define LED3 (1<<PB2)// definicja pinu do którego podłączona jest dioda B
#define LED1_OFF PORTB |= LED1// makrodefinicja - wyłączenie diody
#define LED2_OFF PORTB |= LED2
#define LED3_OFF PORTB |= LED3
#define LED1_ON PORTB &= ~LED1// makrodefinicja - załączenie diody
#define LED2_ON PORTB &= ~LED2
#define LED3_ON PORTB &= ~LED3
#define LEDS_OFF PORTB |= (LED1|LED2|LED3)// makrodefinicja - wyłączenie całej taśmy
#define LEDS_ON PORTB &= ~(LED1|LED2|LED3)// makrodefinicja - załączenie całej taśmy

// klawisz K1
#define KEY1 (1<<PD5)

typedef struct {
volatile uint8_t *KPIN;
uint8_t key_mask;
uint8_t wait_time_s;
void (*kfun1)(void);
void (*kfun2)(void);
uint8_t klock;
uint8_t flag;
} TBUTTON;

// timer programowy
volatile uint16_t Timer1;

// własne funkcje użytkownika
void change_led1(void) {
// stan=0, zmienna statyczna czyli zachowuje się jak zmienna globalna, ale dostępna jest tylko w tej funkcji
static uint8_t stan;
stan++;
if (stan>3) stan=0;
if (stan==0) {
LED1_ON;
LED2_OFF;
LED3_OFF;
}
else if (stan==1) {
LED1_OFF;
LED2_ON;
LED3_OFF;
}
else if (stan==2) {
LED1_OFF;
LED2_OFF;
LED3_ON;
}
else if (stan==3) {
LED1_ON;
LED2_ON;
LED3_ON;
}
}

void change_led2(void) {
// flaga=0, zmienna statyczna czyli zachowuje się jak zmienna globalna, ale dostępna jest tylko w tej funkcji
static uint8_t flaga;

if (flaga) {// jeżeli flaga jest równa 1 (lub różna od 0) to włącz diody i wyzeruj tą flagę (flaga=0)
LEDS_ON;
flaga=1;
}
else {// natomiast jeżeli jest równa 0 to wyłącz diody i ustaw flagę (flaga=1)
LEDS_OFF;
flaga=0;
}
}

// funkcja obsługi pojedynczych klawiszy
void key_press( TBUTTON * btn );


TBUTTON button; // definicja KLAWISZA

//------------------------------------------------------------------
int main(void) {

PORTD |= KEY1; // podciągamy linie klawiszy do VCC
DDRB |= LED1|LED2|LED3; // piny LED jako WYjścia

/* Timer1 – konfigurcajca silnika timerów programowych */
TCCR1A |= (1<<WGM12); // tryb pracy CTC
TCCR1B |= (1<<CS12)|(1<<CS10); // preskaler = 1024
OCR0B = 78; // przerwanie porównania co ok. 10ms (100,16Hz)
TIMSK = (1<<OCIE1A); // Odblokowanie przerwania CompareMatch

button.KPIN = &PIND;
button.key_mask = KEY1;
button.wait_time_s = 1;
button.kfun1 = change_led1;
button.kfun2 = change_led2;

sei();

while(1) {

key_press( &button );


}

}
//------------------------------------------------------------------

void key_press( TBUTTON * btn )
{
register uint8_t key_press = (*btn->KPIN & btn->key_mask);

if( !btn->klock && !key_press )
{
btn->klock=500;
btn->flag=1;
Timer1 = (btn->wait_time_s*1000)/10;
}
else if( btn->klock && key_press )
{
(btn->klock)++;
if( !btn->klock && btn->flag )
{
// reakcja na krótkie wcinięcie klawisza - reakcja po zwolnieniu przycisku
if(btn->kfun1) btn->kfun1();
Timer1=0;
btn->flag=0;
}
} else if( btn->flag && !Timer1 )
{
// reakcja na dłuższe wciniecie klawisza
if(btn->kfun2) btn->kfun2();
btn->flag=0;
}
}

//*** przerwanie Timer1 CompareM
ISR(TIMER1_COMPA_vect) {

uint16_t n;

n = Timer1; /* 100,16Hz Timer1 */
if (n) Timer1 = --n;

}[/syntax]

Statystyki: Napisane przez Tom92 — 13 lis 2017, o 16:41

]]> 2017-11-13T14:22:38+01:00 2017-11-13T14:22:38+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=198860#p198860 <![CDATA[Re: Obsługa klawiszy]]> anshar napisał(a):

Można to rozwiązać inaczej, na przykład do tego celu powołac dodatkową zmienną i na niej operować.
Jeżeli flaga jest równa 1 (lub różna od 0) to włącz diody i wyzeruj tą flagę (flaga=0).
Natomiast jeżeli jest równa 0 to wyłącz diody i ustaw flagę (flaga=1).


Dzięki za dobre wytłumaczenie, zrobiłem tak i działa :) Dzięki temu podobnie zrobiłem w drugiej funkcji, powołałem sobie zmienną statyczną o nazwie "stan", zwiększam go o jeden "stan++" i sprawdzam sobie jaki jest aktualnie stan i na tej podstawie zapalam sobie odpowiednie diody.

Dzięki za podpowiedzi :)

Statystyki: Napisane przez Tom92 — 13 lis 2017, o 14:22

]]> 2017-11-13T12:35:44+01:00 2017-11-13T12:35:44+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=198848#p198848 <![CDATA[Re: Obsługa klawiszy]]> Statystyki: Napisane przez Marhef — 13 lis 2017, o 12:35

]]> 2017-11-12T18:50:12+01:00 2017-11-12T18:50:12+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=198804#p198804 <![CDATA[Re: Obsługa klawiszy]]>

Statystyki: Napisane przez SylwekK — 12 lis 2017, o 18:50

]]> 2017-11-12T18:36:10+01:00 2017-11-12T18:36:10+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=198799#p198799 <![CDATA[Re: Obsługa klawiszy]]> SylwekK napisał(a):

Równie dobrze zamiast flagi (chociaż umiejętność ich stosowania jest jak najbardziej pożądana) wystarczy zrobić nowe makro, np. LEDS_TOGGLE :)

No nie dokońca...
Powiedzmy, że zapaliliśmy sobie diodę LED1 i LED3 czyli LED2 będzie zgaszona.
Teraz chcemy zgasić wszystkie diody.
Proponujesz:
[syntax=c]#define LEDS_TOGLE PORTC ^= (LED1 | LED2 | LED3)[/syntax]

Gdy teraz nasza funkcja będzie wyglądała tak:
[syntax=c]void change_led2(void) {
LEDS_TOGLE;
}[/syntax]
i zostanie wywołana, to jak myślisz czy wszystkie diody zostaną zgaszone? ;)

Statystyki: Napisane przez anshar — 12 lis 2017, o 18:36

]]> 2017-11-12T18:22:04+01:00 2017-11-12T18:22:04+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=198797#p198797 <![CDATA[Re: Obsługa klawiszy]]>

Statystyki: Napisane przez SylwekK — 12 lis 2017, o 18:22

]]> 2017-11-12T17:08:27+01:00 2017-11-12T17:08:27+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=198793#p198793 <![CDATA[Re: Obsługa klawiszy]]> Quote:

Mam zagwozdkę jak je teraz zaświecić po ponownym długotrwałym wciśnięciu klawisza bo już kombinuje długo i narazie nie wychodzi mi to, nie rozumiem czemu nie działa mi taki zapis:

[syntax=c]void change_led2(void) {
if( LEDS_ON ) LEDS_OFF;
else if( LEDS_OFF ) LEDS_ON;
}[/syntax]

Diody zgasną, ale już się nie zaświecą...


Dzieje się tak dla tego, że sprawdzasz czy LEDS_ON jest różne od zera, prawda?
Jest to makro, coś w tym stylu:
#define LEDS_ON PORTC |= LED1 | LED2 | LED3
lub
#define LEDS_ON PORTC &= ~(LED1 | LED2 | LED3)
Czyli sprawdzasz czy PORTC |= LED | LED2 | LED3; jest różne od zera.
A kiedy będzie równe 0? Pomyśl nad tym.

Można to rozwiązać inaczej, na przykład do tego celu powołac dodatkową zmienną i na niej operować.
Jeżeli flaga jest równa 1 (lub różna od 0) to włącz diody i wyzeruj tą flagę (flaga=0).
Natomiast jeżeli jest równa 0 to wyłącz diody i ustaw flagę (flaga=1).


[syntax=c]void change_led2(void) {
//flaga = 0, zmienna statyczna czyli zachowuje się jak zmienna globalna, ale dostępna jest tylko w tej funkcji
static uint8_t flaga;

if(flaga)
{
LEDS_ON;
flaga=0;
}
else
{
LEDS_OFF;
flaga = 1;
}
}[/syntax]

lub tak:
[syntax=c]void change_led2(void) {

static uint8_t flaga;

if(flaga) LEDS_ON;
else LEDS_OFF;

flaga = !flaga;
}[/syntax]

Statystyki: Napisane przez anshar — 12 lis 2017, o 17:08

]]> 2017-11-12T13:34:00+01:00 2017-11-12T13:34:00+01:00 https://forum.atnel.pl/viewtopic.php?t=19592&p=198776#p198776 <![CDATA[Obsługa klawiszy]]>
Mam problem z obsługą klawiszy z bloga Pana Mirka. Chodzi o to, że chciałbym sobie napisać własne funcje do obsługi jednego klawisza, ale gdy robię zmiany w funkcji:
[syntax=c]void change_led2(void) {
PORTC ^= LED2;
}[/syntax]
to niestety nie działa tak jak bym chciał.

Mam trzy diody i niezależnie jakie teraz świecą to chcę żeby po dłuższym przytrzymaniu klawisza wszystkie zgasły, a po ponownym przytrzymaniu klawisza wszystkie się zaświeciły. I krótkotrwałym wciskaniem zmieniać świecące diody (ale to już druga funkcja). Zrobiłem sobie tak:
[syntax=c]void change_led2(void) {
LEDS_OFF;
}[/syntax]
Mam zagwozdkę jak je teraz zaświecić po ponownym długotrwałym wciśnięciu klawisza bo już kombinuje długo i narazie nie wychodzi mi to, nie rozumiem czemu nie działa mi taki zapis:
[syntax=c]void change_led2(void) {
if( LEDS_ON ) LEDS_OFF;
else if( LEDS_OFF ) LEDS_ON;
}[/syntax]
Diody zgasną, ale już się nie zaświecą...

Statystyki: Napisane przez Tom92 — 12 lis 2017, o 13:34

]]>