Witam,

do tej pory pisałem swoje programy w main. Przyszedł czas na zoptymalizowanie swojego kodu, żeby jakoś to wyglądało. Wiem, że ten program można by napisać w kilku linijkach w mainie, ale chciałem się nauczyć na najprostszym przykładzie Program działa, co mnie bardzo cieszy, ale chciałbym poznać waszą opinię. Domyślam się, że coś tam jeszcze jest nie tak. Tylko pytanie.. co?

main.c


led_mig.c


led_mig.h

_________________
+++++[>++++<-]>[>++++++<-]>.---------.+++.

konfiguracje w funkcji mig_led - wydzielić do innej funkcji???

No tak, ustawienie trybu, odblokowanie przerwania, a i w przważającejliczbie przypadków (np. w przedstawionym programie) ustawienie preskalera i rejestru OCRxy wystarczy zrobić RAZ, przed wejściem do pętli głównej.

_________________
+++++[>++++<-]>[>++++++<-]>.---------.+++.

A ja bym jeszcze wartość Timer1 zrobił jako parametr funkcji mig_led(x,y) i drugi parametr (y) który by zmieniał wartość rejestru OCRx i dawał możliwość wybrania czy ma mrugać co x sekund, milisekund itp...

_________________
Podpis...

Witajcie,
zmagam się właśnie z tym samym zagadnieniem i dioda nie mruga.
Możecie powiedzieć mi co robię źle?

dioda.h


main.c


dioda.c


Przy wklejaniu te dwa pierwsze include mi się tutaj na forum zjadają, a są one następujące:

#include <avr/io.h>
#include <util/delay.h>

Wszystko pomotane ale szczególnie popatrz na linie 18 pliku dioda.c

_________________
Podpis...

Brak pętli while(1)

Namazane ze smarta.

_________________
Usługi druku przestrzennego - www.drumik.pl

nie wiem dlaczego przy kopiowaniu wiele znaków się nie przeniosło. W każdym bądź razie sytuację opanowałem, dzięki za uwagi.
Teraz borykam się z problemem LCD, jeśli możecie proszę zaglądnijcie tutaj topic13375.html
pzdr