Witam wszystkich,

Po oglądnięciu poradników video Pana Mirka na temat pamięci EEPROM staram się wykorzystać tą wiedzę na kolorowej diodzie RGB. Już od dłuższego czasu staram się napisać program na zestawie uruchomieniowym ATB wraz z ATB SHIELD IO. Założenie działania programu jest takie:
Przyciskiem zmieniać kolory diody RGB oraz wraz z każdą zmianą koloru procesor powinien zapisać dany kolor do pamięci EEPROM, aby po resecie włączył się zapamiętany kolor diody.

Częściowo się to udało. Zmieniam kolory i zapisuje je do pamięci EEPROM, ale podczas resetu nie wszystkie kolory są prawidłowo zapamiętane. Problem jest z kolorem białym, czerwonym i żółtym.

Poniżej zamieszczam kod programu (Nie uporządkowany bo w fazie testów). Jakby ktoś mógł sprawdzić na swoim zestawie i może będzie wiedział o co chodzi:

main.c


common.c


common.h

Zacznijmy od tego:


Nie analizowałem dokładnie całego kodu, ale to co zauważyłem "na pierwszy rzut oka". Masz wewnątrz funkcji niezainicjowany wskaźnik wsk i zapisujesz go do EEPROM? Raczej to nie zadziała tak jak zakładasz. Wskaźnik będzie zawierał przypadkowe dane. Kompilator nie zgłaszał zastrzeżeń?

W ogóle nie bardzo rozumiem co chcesz zrobić. Po co używasz funkcji atoi()? Po co te warunki? Co tak naprawdę chcesz wpisać do tej struktury? Jeśli chcesz zapisać do EEPROM aktualną zawartość struktury ram_cfg wywal zupełnie tę funkcję RBG() i w jej miejscu użyj od razu funkcji copy_ram_to_eem(), przykładowo:

...a nawet można jej użyć tylko raz, po zakończeniu tych wszystkich instrukcji if.

W pętli głównej, zamiast tego szeregu if'ów lepiej użyć switch. Tak jak jest też będzie działać, ale ze switch będzie czytelniej.


EDIT:
Jeszcze nie zauważyłem jednej rzeczy. Przykładowo makro DLEDR_ON masz zdefiniowane jako PORTB &= ~LEDR_PIN.
Później używasz zapisu:
Co zostanie rozwinięte przez preprocesor (tak w uproszczeniu) do:
Niewykluczone, że to jakoś zadziała, ale nie sądzisz, że to dość dziwny zapis?
Do ustawienia koloru w strukturze używaj raczej wartości liczbowych (0 lub 1, bo zakładam, że nie sterujesz jasnością poszczególnych kolorów) np.:
i dopiero później na podstawie tej wartości zmieniaj stan pinu, tak jak to masz na początku funkcji main().

Dziękuję Ci bardzo andrews, wprowadziłem poprawki według Twoich wskazówek i teraz kod działa tak jak sobie założyłem.

Funkcją atoi() chciałem zamienić parametr na liczbę i wpisać ją do struktury, widać musze poczytać na temat tej funkcji, widziałem ją w poradniku i też próbowałem z niej skorzystać. Jednak na działaniach liczbowych nie bardzo :p