[Atmega8][C][PCF8583P] - Wewnętrzne przerwania asynchroniczne. Czy potrzebne?

Opracowałem zegar elektroniczny oparty na mikrokontrolerze Atmega8 i zegarze PCF8583P.

Działanie programu napisanego w C:
-w nieskończonej pętli głównej wyświetlam na wyświetlaczu LED godziny i minuty poprzez multipleksowanie
-za każdym wyświetleniem wyświetlacza pobierany i zapisywany jest czas z PCF8583P
-posiadam dwa przyciski, którymi ustawiam czas

Możnaby powiedzieć że zegar jest skończony i działa poprawnie, ale zastanawiam się czy takie nieustanne pobieranie czasu z PCF-a jest dobrym rozwiązaniem? Jakie są jego wady? Jak mogę udoskonalić projekt? A może nie muszę go udoskonalać?

Jest to dopiero mój drugi poważny projekt i aby nie wynosić z niego złych nawyków proszę o radę.

Marcin1828 napisał:

zastanawiam się czy takie nieustanne pobieranie czasu z PCF-a jest dobrym rozwiązaniem?

Nie widzę żadnych negatywów ani wad takiego rozwiązania. PCF ani atmega się z tego powodu nie zużyją. Może pokaż schemat, kod. Ciężko bez tego powiedzieć co możesz poprawić. Zegarek zasilany z..?

Marcin1828 napisał:

-w nieskończonej pętli głównej wyświetlam na wyświetlaczu LED godziny i minuty poprzez multipleksowanie

Lepiej by było upchnąć to do przerwania.

Marcin1828 napisał:

-za każdym wyświetleniem wyświetlacza pobierany i zapisywany jest czas z PCF8583P

Nie zapisuj czasu, jeśli nie ma takiej potrzeby.

Pokaż program i schemat to Ci podpowiemy co i jak.

excray napisał:

Lepiej by było upchnąć to do przerwania.

Wrzuciłem wyświetlanie godziny na led do przerwania ale przez to ustawianie godziny przestało działać. Po uruchomieniu układu nie działa poprawnie także test wyświetlacza. Da się jakoś zablokować przerwanie jeżeli jestem w trybie ustawiania godziny?

Oto kod:

Marcin1828 napisał:

Da się jakoś zablokować przerwanie jeżeli jestem w trybie ustawiania godziny?

Lepiej zamiast tego zrobić całość POPRAWNIE.

tj:
1. Bez instrukcji opóźniających
2. Z eliminacją drgań styków za pomocą przerwania timera (najlepiej tego samego, co od obsługi LEDów)
3. Odczyt z PCF też wykonywać w przerwaniu (układ ten generuje impulsy o częstotliwości 1Hz - idealne do podłączenia na przerwanie zewnętrzne procesora)
4. Zapis godziny tylko jeśli trzeba

Zacznij od poprawnej obsługi LEDów. Przykład zanjdziesz np. http://and-tech.pl/zestaw-evb-5-1/ (przykłady C)

Obsługę LEDów zrobiłem zgodnie z przykładem, ale musiałem zmodyfikować przerwanie bo były tam ustawiane inne rejestry niż w Atmedze8.

Proszę o wyjaśnienie punktu nr 2.

Ad3
Zrobiłem przerwanie zewnętrzne INT0, ale koliduje mi ono z tym od LEDów i w rezultacie godzina nie jest pobierana.

Oto nowy schemat:

Pokaż program - musiałeś coś ostro namieszać

Pobierasz czas przy kazdym przejsciu petli,
wiec sprawdz jeszcze co sie stanie gdy wyjmiesz pcf8583 z podstawki i uruchomisz układ ... (symulacja braku polaczenia)

a) delay de facto w pętli przerwania (bo w showTimeOnLEDS() ), to może/musi miec głębokie skutki
b) volatile jest użyte w sposób który myli, to nie działa tak, jak myślisz.

Przerwanie TIMER0_COMP_vect, które było użyte w przykładzie od piotrva zastąpiłem TIMER2_COMP_vect.
Taktowanie procesora w fusebitach ustawiłem na 4MHz- wystarczające?
Mruganie dwukropka zrobiłem jako zwyczajną pętlę(pewnie powiecie że trzeba też wrzucić to w przerwanie).
Nie wiem co z ustawianiem godziny. Może zostać w tym miejscu, w którym jest? Wszystkie delay_ms() zastąpić przerwaniami? W jaki sposób to zrobić?

Bez przerwania zewnętrznego INT0 wszystko działa poprawnie. Po jego "odkomentowaniu" dwukropek mruga częściej i nie jest pobierana godzina z PCF8583P(ciągle wyświetla 00:00).

Wrzucam kod, zobaczcie proszę w czym problem.

Umieść kod w znacznikach syntax - będzie szybciej i łatwiej ocenić.

main.c:



7SEG.c:

1. nie definiuj F_CPU w kodzie!!!!!!!!!!
2. nie stosuj zapisów typu: PORTC = 0x0c; - zamiast tego użyj etykiet oraz operacji bitowych - tak jak w zapisie do MCUCR np.
3. mruganie zdecydowanie na przerwanie timera - masz nawet LED_timer
4. aktualizacja zawartości wyświetlacza - wywołanie funkcji LED_dec(liczba,0); - powinno odbywać się tylko po odczytaniu czasu z zegara - czyli nie częściej niż raz na sekundę - w przerwaniu INT0 - ustawiaj sobie jakąś flagę.
5. Kopiując spore kawałki kodu - warto zachować informacje skąd je masz

Ad 1. Definiowałem bo bez definicji Atmel Studio pokazuje mi warninga. Właściwie dlaczego tego nie można robić?
Ad 2. Zmienione.
Ad 3. Dobrze to zrobiłem?
Ad 4. Zrobiłem jakąś flagę, ale chyba źle... Mam straszny problem z tym przerwaniem zewnętrznym. Zaburza cały program. Po odłączeniu PCF-a od INT0 działa ładnie.

Ad. 1. To definiuje się w opcjach projektu. Dlaczego? Bo tylko wtedy definicja będzie widoczna dla wszystkich plików. W przeciwnym wypadku np. plik od obsługi uartu będzie miał inną F_CPU, plik delay.h jeszcze inną a main - trzecią watość.
Ad. 3. Trochę dobrze, ale skąd naczytałeś się, aby dawać takiego "nop"? Taki blok należy umieścić w bloku atomowym - poczytaj o tej kwestii.
Ad. 4. Co ta flaga ma robić? Bo u Ciebie coś dziwnego.
Ona powinna być: po każdym odczycie z zegarka ustawiana na 1, a w main sprawdzana i kasowana:


Co z Ad. 5.?

Ad 1. Zdefiniowałem F_CPU jako symbol w ustawieniach Toolchain->Compiler->Symbols i działa poprawnie. Dziękuję za wyjaśnienie.
Ad 5. Źle Cię zrozumiałem, już dodane oczywiście. Oryginalnie było samo imię i usunąłem
Reszta poprawiona.

Nadal wszystko komplikuje przerwanie INT0. Nie wykonują się instrukcje wewnątrz przerwania. W tej sytuacji nie pojawia się nic na wyświetlaczu i dwukropek zapala się nieregularnie. Co jest nie tak z tym przerwaniem?

Oto kod:

Skąd wiesz, że przerwanie się nie wykonuje?
Dalej - włącz podciąganie na linii przerwania.

piotrva napisał:

Skąd wiesz, że przerwanie się nie wykonuje?

Wnioskuję po tym, że wrzucając do przerwania instrukcję RTC_Read_Time() wyświetla się godzina 00:00, a jeżeli wrzucę ustawianie flagi to nic się nie wyświetla.

piotrva napisał:

Dalej - włącz podciąganie na linii przerwania.

Znalazłem jedno miejsce w kodzie gdzie ten rezystor mógł być przypadkowo odłączany i poprawiłem, ale nie przyniosło to poprawy...


Nie wiem co dalej nie gra. Nadal jest coś nie tak z INT0, ale nie potrafię stwierdzić co...


Na tę chwilę dwukropek miga bardzo nieregularnie a wyświetlacz jest zgaszony.
Oto kod:

Linijka z ustawianiem portów może wywoływać przerwanie. Musisz trochę nad tym popracować.

Ponadto, czy masz analizator stanów logicznych? Jeśli nie to kupuj szybko