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:


Ową funkcję wywołuję w pętli, która odpowiedzialna jest za animację a'la "loading" na LCD:


Ciało przerwania Timer'a 2:


Timer2 inicjalizuję na początku funkcji main:



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.

_________________
http://www.jaglarz.info

Tak naprawdę to ta instrukcja jest równoważna:
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:

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

Rozwiązałem to w następujący sposób:

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