ATNEL tech-forum 2011-10-26T21:50:08+01:00 https://forum.atnel.pl/feed.php?f=4&t=35&mode 2011-10-26T21:50:08+01:00 2011-10-26T21:50:08+01:00 https://forum.atnel.pl/viewtopic.php?t=35&p=149#p149 <![CDATA[Re: Multipleksowanie LED na Atmega8]]> ATmega8 "zmarnować" na taki cel aż 16-bitowy timer sprzętowy (Timer1)
???? Szkoda bo 8-bitowy wystarczyłby w zupełności. No ale jeśli to
tylko dla testów to rozumiem ;)

Generalnie zabiera Pan się za przesiadkę na inny procek i na inny
timer tymczasem, pierwsza linia w prezentowanym kodzie, nadesłanym
przez Pana to:

[syntax=cpp]// ustawienie TIMER0[/syntax]

a zwracałem w książce uwagę, że prawidłowe komentarze w programie są
nie mniej ważne niż sam kod, a szczególnie gdy pracujemy w zespole.
Wprawdzie tu nie pracuje Pan w zespole - jednak .... podsyła Pan kod
komuś innemu do sprawdzenia (mowa tutaj o mnie) .... czyli poniekąd
tworzymy mini zespół. No i co ja widzę? komentarz, że jednak dokonuje
Pan ustawień dla Timer0 a nie Timer1.

hmmm to że chodziło Panu o Timer1 wywnioskowałem na szybko (ale być
może błędnie chociażby z nazwy wektora przerwania jakiego Pan użył)

Code:
ISR(TIMER1_COMPA_vect)


Niestety w inicjalizacji jest już totalny "MISZ-MASZ" jeśli chodzi o
próbę dotarcia co Pan miał na myśli, bo:

wprawdzie ustawia Pan tryb CTC dla Timera1 (patrząc na nazwę bitu
WGM12) ale czy spojrzał Pan w jakim rejestrze Pan ten bit ustawia ????
dlaczego w TCCR0 ???? przecież nawet nie zaglądając do noty PDF widać
że to rejestr konfiguracyjny Timer0 a nie Timer1

Zatem po takiej linijce to ja już do końca nie wiem czy chodziło Panu
o Timer0 czy Timer1.

Patrząc jednak na koleją linie - jeszcze bardziej się upewniam, że
może chodziło Panu o Timer0 gdyż preksaler bity CS02 i CS00 ustawia
Pan w rejerze TCCR0 znowu od Timera0 i to jeszcze nazwy bitów też
pochodzą z Timera0. Bo dla Timera1 miałyby one nazwy CS12 i CS10

Widzi Pan różnicę ???

Jeśli nie to bardzo jednak proszę otworzyć sobie notę PDF procka
ATmega8 (ja właśnie to zrobiłem) i zajrzeć w jakich rejestrach są bity
konfiguracyjne dotyczące Timer1 i jakie tam są nazwy tych bitów.
Niestety bez noty daleko nie zajedziemy - musi ona być wciąż pod ręką
o czym wciąż piszę w książce ;)

Co ciekawe w kolejnej linii już Pan sprawdził że rejestr porównania
dla Timera1 to OCR1A a nie OCR0 ;) więc tu chyba zajrzał Pan do noty
PDF ???

Dobrze to na razie tyle moich podpowiedzi ;) nie chcę od razu podawać
gotowego rozwiązania a jedynie naprowadzić Pana.

Jednak jeśli dalej się nie uda to proszę śmiało pisać podpowiem dalej
OK?

Statystyki: Napisane przez mirekk36 — 26 paź 2011, o 21:50

]]> 2011-10-26T20:19:51+01:00 2011-10-26T20:19:51+01:00 https://forum.atnel.pl/viewtopic.php?t=35&p=147#p147 <![CDATA[Multipleksowanie LED na Atmega8]]> Po przeczytaniu (tzn. dokładnym przejrzeniu- nie znam j. angielskiego)noty PDF, przerobiłem kod książkowy i nie działa
mi timer w trybie CTC.Co zrobiłem źle? :?:

[syntax=cpp]// ustawienie TIMER0
TCCR0 |= (1<<WGM12);// tryb CTC
TCCR0 |= (1<<CS02)|(1<<CS00);// preskaler = 1024
OCR1A = 39;// dodatkowy podział przez 39 (rej. przepełnienia)
TIMSK |= (1<<OCIE1A);// zezwolenie na przerwanie CompareMatch
}


// ================= PROCEDURA OBSŁUGI PRZERWANIA – COMPARE MATCH
ISR(TIMER1_COMPA_vect)
{
static uint8_t licznik=1;// zmienna do przełączania kolejno anod wyrwietlacza

LED_ANODY = (LED_ANODY & 0xF8);


if(licznik==1) LED_DATA = pgm_read_byte( &cyfry[cy1] );// gdy zapalony wyśw.1 podaj stan zmiennej c1
else if(licznik==2) LED_DATA = pgm_read_byte( &cyfry[cy2] );// gdy zapalony wyśw.2 podaj stan zmiennej c2
else if(licznik==4) LED_DATA = pgm_read_byte( &cyfry[cy3] );// gdy zapalony wyśw.3 podaj stan zmiennej c3
//else if(licznik==8) LED_DATA = pgm_read_byte( &cyfry[cy4] );
//else if(licznik==16) LED_DATA = pgm_read_byte( &cyfry[cy5] );
//else if(licznik==32) LED_DATA = pgm_read_byte( &cyfry[cy6] );

LED_ANODY = (LED_ANODY & 0xF8) | (~licznik & 0x7);// cykliczne przełączanie kolejnej anody w każdym przerwaniu

// operacje cyklicznego przesuwania bitu zapalającego anody w zmiennej licznik
licznik <<= 1;// przesunięcie zawartości bitów licznika o 1 w lewo
if(licznik>4) licznik = 1;// jeśli licznik większy niż 8 to ustaw na 1
}[/syntax]

Statystyki: Napisane przez szymwoj — 26 paź 2011, o 20:19

]]>