ATNEL tech-forum

Re: Problem ze zrozumieniem Timera

2020-03-29T20:18:49+01:00

mars10 napisał(a):

A własnie, że zmienna Timer1 musi być static.

Ależ co kolega opowiada że MUSI bez sensu - dla tego przykładu nie ma to ŻADNEGO znaczenia a ja podejrzewam, że kolega tak pisze bo albo niestety nie rozumie jak działa ten mechanizm "timera programowego" albo nie do końca rozumie działania specyfikatora static dla zmiennych globalnych. Więc jak coś to dopytaj to chętnie podpowiem - zamiast wprowadzać w błąd kogoś - że tutaj musi być static. Ja tylko podpowiem, że w tym wypadku specyfikator static dla Timer1 może być ale nie musi. A ty jeśli wiesz dlaczego to spróbuj napisać jaka będzie różnica - wtedy może lepiej zrozumiesz jak to działa ok?

Statystyki: Napisane przez mirekk36 — 29 mar 2020, o 20:18

Re: Problem ze zrozumieniem Timera

2020-03-29T19:20:18+01:00

mars10 napisał(a):

Mirek pisał "mars10 --> tak, ale nie żadne static......"
A własnie, że zmienna Timer1 musi być static. Lepiej podczas deklaracji od razu zainicjować ją na 50.
Sprawdź, co się stanie jak nie będzie static.

Tak, jeżeli zmienna Timer1 jest zdefiniowana w przerwaniu to musi to być static.
Natomiast jeżeli, tak jak to było wcześniej, czyli jako zmienna globalna to zamiast static musi być użyty specyfikator volatile.

Statystyki: Napisane przez auers — 29 mar 2020, o 19:20

Re: Problem ze zrozumieniem Timera

2020-03-29T17:45:48+01:00

Statystyki: Napisane przez mars10 — 29 mar 2020, o 17:45

Re: Problem ze zrozumieniem Timera

2020-03-29T15:28:33+01:00

Szofer napisał(a):

Tu wywalenie obsługi diody z przerwania nie ma nic do rzeczy.
Możesz to zrobić w przerwaniu i też musi działać.

MA do rzeczy bo mryganie diodą było i w petli (potem zakomentowane bo nie swieciła dioda, zapewne) i w przerwaniu także bardziej pwm się robił 50hz,

No w tym sensie, że trzeba się zdecydować w którym miejscu mrygamy, to tak ma to znaczenie.
Albo mrugamy w przerwaniu, albo w pętli głównej.
Natomiast samo mryganie w przerwaniu też jest dopuszczalne i musi działać.

------------------------ [ Dodano po: 31 minutach ]

AKSELINEK

Poniżej masz poprawiony kod twojego programu z mruganiem.
Sterownie diodą masz w przerwaniu.
Uwagi:
Jakie masz taktowanie 1Mhz? Przelicz jeszcze raz OCR2.

[syntax=c]#include
#include

#define LEDN (1<
int main( void ) {

DDRB|=LEDN;

/* timer2 - konfiguracja silnika timerów programowych */
TCCR2 |= (1< TCCR2 |= (1< OCR2 = 8; // przerwanie porównania co 10ms (100Hz) 78
TIMSK = (1<
sei(); // globalne odblokowanie przerwania
while(1) {};
}

ISR(TIMER2_COMP_vect) {
static int Timer1;
if (Timer1) (--Timer1);
else {
Timer1=50;
PORTB ^= LEDN;
}

}[/syntax]

Statystyki: Napisane przez auers — 29 mar 2020, o 15:28

Re: Problem ze zrozumieniem Timera

2020-03-29T15:18:07+01:00

[/quote]
Tu wywalenie obsługi diody z przerwania nie ma nic do rzeczy.
Możesz to zrobić w przerwaniu i też musi działać.[/quote]

MA do rzeczy bo mryganie diodą było i w petli (potem zakomentowane bo nie swieciła dioda, zapewne) i w przerwaniu także bardziej pwm się robił 50hz,

Statystyki: Napisane przez Szofer — 29 mar 2020, o 15:18

Re: Problem ze zrozumieniem Timera

2020-03-29T15:00:25+01:00

^ to operator xor
w tym przypadku:
sw^=1
to nic innego jak negacja.
Cała ta zabawa z sw jest niepotrzebna można to zrobić prościej od razu na porcie:
PORTB ^= LEDN;
Czyli negujesz jeden bit na który wskazuje LEDN;

------------------------ [ Dodano po: 6 minutach ]

AKSELINEK napisał(a):

Kolega Szofer słusznie stwierdził żeby wywalić obsługę diody z przerwania - tak też zrobiłem, wrzuciłem warunek do pętli głównej też nie poszło.

Tu wywalenie obsługi diody z przerwania nie ma nic do rzeczy.
Możesz to zrobić w przerwaniu i też musi działać.

Statystyki: Napisane przez auers — 29 mar 2020, o 15:00

Re: Problem ze zrozumieniem Timera

2020-03-29T12:07:09+01:00

Dziękuje za zainteresowanie przykładem, wzorowałem się na przykładach z książki więc tam ustawienia zmiennych były poprawne.
Kolega Szofer słusznie stwierdził żeby wywalić obsługę diody z przerwania - tak też zrobiłem, wrzuciłem warunek do pętli głównej też nie poszło.
Jednak nie dokładnie skopiowałem warunek z książki i tu poprawiłem:

[syntax=c]if (!Timer1) {

if( !sw ) PORTB |= LEDN;
else PORTB &= ~LEDN;
sw^=1;

Timer1 =40;
}[/syntax]
mylę że zbagatelizowałem w przykładzie działanie zmiennej sw i chyba do końca nie odrobiłem lekcji.

------------------------ [ Dodano po: 54 minutyach ]

Proszę przeanalizujcie czy dobrze rozumuje:
Wykonuje się przerwanie co 100 Hz w wykonanym przerwaniu jest stworzona zmienna lokalna n, która zanika po wykonaniu przerwania .
Wartość zmiennej globalnej Timer1 jest wpisana do n.
Jeśli n jest dodatnie to Timer 1 się zmniejsza o 1 , ale gdy po 40 razach dojdzie do 0 to spełnia się warunek w pętli if(!Timer1) i w pętli jeśli sw=0 to (!sw)=1 to zapala diodę - czy tak??
nie rozumiem natomiast co znaczy sw^=1 ??? ale następnie Timer1 otrzymuje zwiększenie wartości do 40 i sytuacja się cyklicznie powtarza.

Podpowiedzcie czy dobrze rozważam czy nie ?????

Statystyki: Napisane przez AKSELINEK — 29 mar 2020, o 12:07

Re: Problem ze zrozumieniem Timera

2020-03-28T23:52:18+01:00

Podstawowe zasady dobrego programu:
1. Jeżeli jakaś zmienna jest używana tylko lokalnie wewnątrz funkcji, to nie używaj w tym celu zmiennej globalnej.
2. Idealna liczba zmiennych globalnych to zero.

Pozdrawiam,

Statystyki: Napisane przez mars10 — 28 mar 2020, o 23:52

Re: Problem ze zrozumieniem Timera

2020-03-28T23:47:36+01:00

AKSELINEK
Nie widzę definicji zmiennej Timer1. Czy masz przy niej specyfikator volatile?
Coś pokombinowałeś też z nawiasami {}.
ISR(TIMER2_COMP_vect) powinien być poza main().

Statystyki: Napisane przez auers — 28 mar 2020, o 23:47

Re: Problem ze zrozumieniem Timera

2020-03-28T23:34:51+01:00

mars10 napisał(a):

Ale do dobrego stylu programowania należy unikać zmiennych globalnych, które są dostępne dla wszystkich funkcji, a to może prowadzić do błędów.

No tak ale co to ma wspólnego ze zmienną static uint16_t n; ? w tej procedurze obsługi przerwania - skoro nie jest ona globalna? ... zachowuje się tylko jako globalna w zakresie inicjalizacji właśnie.

Statystyki: Napisane przez mirekk36 — 28 mar 2020, o 23:34

Re: Problem ze zrozumieniem Timera

2020-03-28T22:53:31+01:00

Masz rację Mirku. Zmienne globalne są inicjowane na zero.
Ale do dobrego stylu programowania należy unikać zmiennych globalnych, które są dostępne dla wszystkich funkcji, a to może prowadzić do błędów.

Statystyki: Napisane przez mars10 — 28 mar 2020, o 22:53

Re: Problem ze zrozumieniem Timera

2020-03-28T22:30:09+01:00

mars10 --> tak ale nie żadne static, a gdybyś już nawet dawał specyfikator static to powiedz mi po co przypisujesz wtedy zero ? toż static powoduje, że masz tę zmienną tak jak globalną czyli inicjalizowaną zerem automatycznie. Tylko zmienne lokalne się inicjalizuje - o ile pierwszą kolejną na niej operacją nie jest właśnie inicializaja jak tutaj

[syntax=c]ISR(TIMER2_COMP_vect) {

uint16_t n;

n = Timer1; /* 100Hz Timer */
if (n) Timer1 = --n;[/syntax]

więc tak ma pozostać i tak jest NAJLEPIEJ

Statystyki: Napisane przez mirekk36 — 28 mar 2020, o 22:30

Re: Problem ze zrozumieniem Timera

2020-03-28T22:17:09+01:00

W ISR oczywiście
static uint16_t n = 0;

...sorry

Statystyki: Napisane przez mars10 — 28 mar 2020, o 22:17

Re: Problem ze zrozumieniem Timera

2020-03-28T22:12:42+01:00

Panowie mars10 i Szofer - zerknijcie do Bluebooka to zrozumiecie jak działa ten mechanizm w przerwaniu - bo na razie to niestety obaj wymyślacie niestworzone historie na temat tej zmiennej "n" ... a tu akurat ma być tak jak jest

Statystyki: Napisane przez mirekk36 — 28 mar 2020, o 22:12

Re: Problem ze zrozumieniem Timera

2020-03-28T22:12:20+01:00

Zrób tak:
[syntax=c]#define TIMER140

volatile uint8_t flaga = 0;

int main(void)
{
DDRB |= PB3;
init_timer();
sei();
while (1)
{
if (flaga)
{
flaga = 0;
PORTB ^= (1 << PB3);
}
}

}
//----------------------------------------
ISR( TIMER2_COMP_vect)
{

uint16_t n = 0;

if (n++ > TIMER1)
{
flaga = 1;
n = 0;
}
}[/syntax]

Statystyki: Napisane przez mars10 — 28 mar 2020, o 22:12

Re: Problem ze zrozumieniem Timera

2020-03-28T21:56:39+01:00

Jak n zadeklarujesz w przerwaniu jakąś wartością np: n=50; to w petli zmieniając timer1=40; jaka wartość n będzie po wykonaniu if-a I ustawieniu timer1=40; i wejsciu w przerwanie?

Statystyki: Napisane przez Szofer — 28 mar 2020, o 21:56

Re: Problem ze zrozumieniem Timera

2020-03-28T21:15:59+01:00

Zmienna int16_t powinna być klasy static oraz zainicjowana, a więc static uint16_t n=40;

------------------------ [ Dodano po: 10 minutach ]

Zmienna static istnieje między wywołaniami funkcji. To znaczy, że stanowi prywatną pamięć funkcji.
U ciebie deklaracja uint_16 n; nie wiadomo jaką ma wartość zmienna n i przy kolejnych wywołaniach obsługi przerwania posiada przypadkową wartość.

Statystyki: Napisane przez mars10 — 28 mar 2020, o 21:15

Re: Problem ze zrozumieniem Timera

2020-03-28T21:09:11+01:00

Wywaliłem obsługę diody z przerwania ...
potem:
wrzuciłem do pętli głównej
[syntax=c]if (!Timer1) {
PORTB ^= LEDN;
Timer1 =40;
}[/syntax]

niestety dalej nie miga ??????

Statystyki: Napisane przez AKSELINEK — 28 mar 2020, o 21:09

Re: Problem ze zrozumieniem Timera

2020-03-28T20:36:13+01:00

Wywal obsługę diody z przerwania

Statystyki: Napisane przez Szofer — 28 mar 2020, o 20:36

Problem ze zrozumieniem Timera

2020-03-28T19:48:27+01:00

Witam chciałem w pewnym programie migać diodą za pomocą Timera . W założeniu chciałem by dioda migała co 1 sek .
Nie wiem co zrobiłem żele ale dioda mignie parę razy i koniec??

Atmega 32 1 MZ trochę ślimak

[syntax=c]#define LEDN (1<
int main( void ) {

/* timer2 - konfiguracja silnika timerów programowych */
TCCR2 |= (1< TCCR2 |= (1< OCR2 = 255; // przerwanie porównania co 10ms (100Hz) 78
TIMSK = (1<
sei(); // globalne odblokowanie przerwania
while(1) {
// ************************************************************
if (!Timer1) {
//PORTB ^= LEDN;
Timer1 =40;
}
{
}
ISR(TIMER2_COMP_vect) {

uint16_t n;
PORTB ^= LEDN;

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

}[/syntax]

Statystyki: Napisane przez AKSELINEK — 28 mar 2020, o 19:48