ATNEL tech-forum

Re: ATmega88PA i funkcja delay_ms na Timerze 2

2016-07-28T06:08:37+01:00

Takie rozwiązanie też jest OK.

Pozwolę sobie jednak wytłumaczyć, na czym polega różnica między naszymi rozwiązaniami.
W Twoim przypadku akurat prawdopodobnie nie ma to większego znaczenia, ale czasami jest bardzo istotne, aby przerwania blokować na jak najkrótszy okres czasu, szczególnie jeśli korzystamy z większej ilości przerwań i niektóre z nich wymagają natychmiastowej (lub przynajmniej jak najszybszej) reakcji.

Rozwiązanie z dodatkową zmienną ma w takim przypadku przewagę, dlatego że przerwania są zablokowane tylko na czas odczytu/kopiowania zmiennej delay_cnt_ms. W Twoim rozwiązaniu, oprócz odczytu danej z RAM, musi być jeszcze wykonane porównanie zmiennej z wartością 0 i przypisanie wartości do zmiennej exit. To zajmie nieco więcej czasu, a przerwanie, które może w tym czasie nastąpić, będzie musiało zaczekać na obsługę do końca bloku ATOMIC_BLOCK.

Mam nadzieję, że nie potraktujesz tego jako złośliwą krytykę, bo nie to było moim zamiarem

Statystyki: Napisane przez andrews — 28 lip 2016, o 06:08

Re: ATmega88PA i funkcja delay_ms na Timerze 2

2016-07-27T18:01:56+01:00

Ogromne dzięki andrews. Nigdy bym na to nie wpadł.

Rozwiązałem to w następujący sposób:
[c]
void delay_ms_x (uint16_t ms_del) {
uint8_t exit = FALSE;
delay_cnt_ms = ms_del;

Timer2_start();

while(1) {
ATOMIC_BLOCK(ATOMIC_RESTORESTATE) {
if(!delay_cnt_ms) exit = TRUE;
}
if(exit) break;
}

Timer2_stop();
}[/c]

Statystyki: Napisane przez JungleMan — 27 lip 2016, o 18:01

Re: ATmega88PA i funkcja delay_ms na Timerze 2

2016-07-27T05:56:12+01:00

JungleMan napisał(a):

[syntax=c] while(delay_cnt_ms);[/syntax]

Tak naprawdę to ta instrukcja jest równoważna:[syntax=c] while(delay_cnt_ms!=0);[/syntax]
Zwróć uwagę, że w tym momencie mikrokontroler porównuje z wartością 0 zmienną dwubajtową modyfikowaną w przerwaniu. Jako że jest to mikrokontroler 8-bitowy, nie zrobi tego w jednym takcie zegara, a między sprawdzaniem poszczególnych bajtów może wystąpić przerwanie, które wprowadzi nową wartość do tej zmiennej i operacja porównania będzie błędna. Możnaby spróbować wprowadzić zmienną pośrednią i zapewnić atomowy dostęp do zmiennej delay_cnt_ms, coś w stylu:[syntax=c]uint16_t delay_tmp;
do
{
ATOMIC_BLOCK(ATOMIC_RESTORESTATE) {
delay_tmp = delay_cnt_ms;
}
} while(delay_tmp);[/syntax]

Statystyki: Napisane przez andrews — 27 lip 2016, o 05:56

Re: ATmega88PA i funkcja delay_ms na Timerze 2

2016-07-27T00:38:59+01:00

topic7402.html

Statystyki: Napisane przez Jaglarz — 27 lip 2016, o 00:38

ATmega88PA i funkcja delay_ms na Timerze 2

2016-07-27T18:03:39+01:00

Witam,

mam pewien problem, który wydaje mi się banalny, ale nie daje mi spać.

Otóż, utworzyłem sobie funkcję delay_ms_x o ciele:
[c]inline void Timer2_start() {
// Clear timer register.
TCNT2 = 0;

// Start Timer and set prescaling.
TCCR2B |= (1<}

inline void Timer2_stop() {
TCCR2B &= ~((1<}

void delay_ms_x (uint16_t ms_del) {
delay_cnt_ms = ms_del;
Timer2_start();
while(delay_cnt_ms);
Timer2_stop();
}[/c]

Ową funkcję wywołuję w pętli, która odpowiedzialna jest za animację a'la "loading" na LCD:
[c]for(uint8_t i = 0 ; i < LCD_COLS && dev_state == UNARMING; i ++) {
lcd_char(0xFF);
delay_ms_x(arm_bar_dur);
LED_TOG;
}[/c]

Ciało przerwania Timer'a 2:
[c]
ISR(TIMER2_COMPA_vect) {
if(!delay_cnt_ms) return;
delay_cnt_ms--;
}[/c]

Timer2 inicjalizuję na początku funkcji main:
[c]void Timer2_init() {
// Set CTC mode.
TCCR2A |= (1<
#define TIMER2_PRESCALER (8.0)
#define TIMER2_INT_FREQ (1000.0)

// Set ~1ms interrupt.
OCR2A = F_CPU / TIMER2_PRESCALER / TIMER2_INT_FREQ - 1;

TIMER2_INT_EN;
}[/c]

Problem w tym, że raz po raz ten odstęp nie jest równy i co około 3 raz funkcja delay_ms_x kończy swoje instrukcje szybciej niż powinna.
Zmienna arm_bar_dur jest niezmienna. Nawet jak podmieniłem w wywołaniu funkcji argument na wartość liczbową zachowanie jest identyczne.
Proszę, pokażcie mi gdzie popełniam błąd.

Statystyki: Napisane przez JungleMan — 27 lip 2016, o 00:36