[Maffiozzo97]

#avr #elektronika

Chcę napisać program na sucho, bo nie mam jak sprawdzić czy działa. Są 2 przyciski i 4 diody LED. Po wciśnięciu 1 przycisku, zapalają się diody w kolejności L1,L2,L3,L4. Po wciśnięciu 2 przycisku zapalają się w kolejności L4,L3,L2,L1.

Moj program:

#include
#include
int main (void)
{
DDRB = 0xFF;
PORTB = 0x00;
DDRD = 0x00;
PORTD = 0xFF;

while(1)
{
int i;
for (i=0;i<5;i++)
{
PORTB |= BV(1);
delayms(500);
PORTB&=~BV(i);
delayms(500);
}
for (i=4;i>0;i--)
{
PORTB|=BV(1);
delayms(500);
PORTB&=~BV(i);
delayms(500);
}
}
return 0;
}

Dobry tok rozumowania? Dopiero zaczynam z tym i pytam się :D

[keton22]

@Maffiozzo97: nie za dobry. Funkcja __delayms() powinna być zakazana. Nie ucz się złych nawyków. Obsługa przycisków przez przerwaniu GPIO z debouncingiem na timerze a potem sekwencja ledów w przerwaniu timera. Na razie pewnie myślisz że się czepiam ale jak spróbujesz zrobić coś rzeczywistego to mi podziękujesz.

[A.....k]

nie za dobry. Funkcja __delayms() powinna być zakazana

@keton22: Serio, to wszystko co masz do powiedzenia o tym kodziku? Może skoro opowi wystarcza to przestań się czepiać spraw drugorzędnych i skup się na algorytmie?

[keton22]

@Analityk: widać że nie rozumiesz, ja się nie czepiam, ogarnij model IO na przerwaniach. W AVR nie programowałem już kilka lat więc kodu ci nie podam ale blokowo:
1. pętla główna może być pusta albo zająć się czymś innym w tzw. międzyczasie (w nietrywialnym przypadku dzięki temu program pozostanie responsywny)
2. w inicie aktywujesz PCINT dla portu na którym znajduje się pin od przycisku (doczytaj w DS do swojejgo AVRa jak ustawić rejestry)

[zetisdead]

i tyle, dzieki temu nie blokujesz procesora czekajac na przyciski

@keton22: i możesz wywołać funkcję sleep :-)

[keton22]

@zetisdead: no to juz zalezy od modelu procka, jak pcint budzi to oszczedzanie energii zawsze na propsie alternatywnie mozna nie przegapic jakiegos waznego zdarzenia ktore przepadloby w trakcie delaya. Potem sa dziwy ze np przepadaja pomiary z jakiegos sensora bo obsluga wyswietlacza jest zrobiona na delayu.

[zetisdead]

@keton22: to był sarkazm gdybyś nie zauważył

[keton22]

@zetisdead: zauwazylem uwierz, w druga strone troche tez byl. Potem trzeba po takich delayowych specach kod poprawiac.

[zetisdead]

Potem trzeba po takich delayowych specach kod poprawiac.

@keton22: straszne

[A.....k]

@keton22: Nie chodzi mi o zwiechę procesora tylko o to, że on niczego więcej nie potrzebuje tylko żeby kurła te diody zmieniały się tak a nie inaczej. A ty przychodzisz i chrzanisz coś od rzeczy.