Witam wszystkich

Mam problem z (najprawdopodobniej) zapisem EEPROMu.
Kod ten ma trafić do procka w latarce LED (włącznik jedynie załącza i odcina baterie).

main.c


setup.h


Klikanie włącznikiem (tzn. odcinanie zasilania) nie powoduje zmiany jasności...
Obecnie jasność zmienia się tylko jeżeli w definicji stałej struktury ładowanej do PROGMEM zmienię wartość u08 ktory_tryb.
Kompilacja w Eclipse przechodzi bez żadnych ostrzeżeń.
Aby mieć dostęp do EEPROMu trzeba ustawić jakieś FUSY?

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Panie, Panie .... haaaalo stop bo wybiegłeś na pole ale jak to się mówi pługa zapomniałeś .... a gdzie u ciebie pętla główna (czyli ten pług) ???

w tył zwrot i idź go zabrać i używać

bo teraz to w tym programie jakimś klawiszem to może MATRIX by zdążył

to nie jest w ogóle program

_________________
zapraszam na blog: http://www.mirekk36.blogspot.com (mój nick Skype: mirekk36 ) [ obejrzyj Kurs EAGLE ] [ mój kanał YT TV www.youtube.com/mirekk36 ]

Chyba się nie zrozumieliśmy...
PCB wygląda mniej-więcej tak: http://www.swiatelka.pl/upload_img/obra ... 5d6012.jpg
Włącznik podłącza i odłącza baterie od całego układu (razem z uC).

Cały więc myk polega na tym, aby program ten analizując odpowiednie sekwencje kliknięć (przy czym kliknięciem jest tu odcięcie zasilania od uC) wrzucił odpowiednie wypełnienie na ster AMC7135.

Chyba że ja czegoś nie zrozumiałem i to:

int main (void)
{
epr_to_ram();
(...)
return 0;
}
Nie jest głównym programem...
Pętli w stylu while (1)... Nie ma bo program ma (bardzo) szybko wykonać jakieś kombinacje po tej strukturze, zapisać ją, ustawić odpowiednie wypełnienie i zacząć idlować.

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

ok popatrzę wieczorkiem

_________________
zapraszam na blog: http://www.mirekk36.blogspot.com (mój nick Skype: mirekk36 ) [ obejrzyj Kurs EAGLE ] [ mój kanał YT TV www.youtube.com/mirekk36 ]

Ta linia jest podejrzana:

PS.
niepotrzebe jest sei();

_________________
Dragonus Cracovus: Biomagia

No faktycznie ta linia

jest dosyć radosną konstrukcją, ale powinna działać.
O ile dobrze pamiętam z http://mirekk36.blogspot.com/2012/11/avr-eeprom-struktury-poradnik-do-ksiazek.html to wyzerowany EEPROM przyjmuje wartość 0xFF, więc ten zapis miał w sposób "garażowy" sprawdzać poprawność EEPROMu.
Jeżeli starczy mi miejsca to pewnie wrzucę sprawdzanie tego typu:

ale to zajmuje ~100B FLASHa, a jest go tutaj cały 1KB więc nie mam za bardzo czym szastać.
Nie wiem czemu, ale przy takiej konstrukcji układ działa trochę inaczej.
Przy starej od razu po zaprogramowaniu dioda zaczynała świecić zaprogramowana jasnością, a teraz dopiero po jednym restarcie (po zaprogramowaniu zachowuje się jakby nie mógł w ogóle nic sczytać) ...
Przy obydwu oczywiście nie można tej jasności normalnie zmienić...

A sei(); będzie potrzebny w przyszłości (ochrona ogniwa, itp.), ale faktycznie obecnie jest niepotrzebny.

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Tam ma być != (wykrzyknik i równa się, czyli operator porównania różne) a nie |= (pionowa kreska i równa się)

Po wyjściu z main jest cli(), a potem pętla nieskończona. Sprawdź sobie w pliku wynikowym. Dlatego to sei() na nic się zda. Wstaw jednak to while(1); to jeden rozkaz.

_________________
Dragonus Cracovus: Biomagia

Wielkie dzięki

W obydwu kodach ten sam błąd
Pierwszy zadziałał od kopa, drugi dopiero po przypisaniu wartości początkowych do EEPROMu, ale ważne, że już działa

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Dlaczego taki zapis działa jak powinien:

natomiast zapisany już w ten sposób:

nie działa prawidłowo? Obie zmienne są oczywiście u08. Konstrukcja ta ma wykonywać 2 różne operacje w zależności od stanu najmłodszego bitu (a pozostałe pomijać). Przy drugim zapisie jeżeli cfg_ram.ktory_tryb jest większy od 0b00000001 zawsze wychodzi z warunku 1. Dokonuje się niejawne rzutowanie? Dodam, że taki zapis:

nie rozwiązuje problemu

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Zgadza się. Przy operacjach bitowych następuje promocja do typu int o długości 16-bit. Zastosowałeś rzutowanie, ale ono ma wyższy priorytet niż przesunięcie i w efekcie rzutowałeś uint8_t na uint8_t, a nie wynik przesunięcia. Tak powinno być w tym przykładzie:

osobiście sądzę że lepszy jest taki zapis:

_________________
Dragonus Cracovus: Biomagia

Jeżeli ktoś jest ciekaw, co się narodziło z projektu to zapraszam do wątku:
http://www.swiatelka.pl/viewtopic.php?t=10224

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

No to skoro uzyskałeś TUTAJ pomoc to może wypadałoby opublikować to TUTAJ w dziale DIY?

_________________
http://www.jaglarz.info

dokładnie - sam bym z chęcią sobie o tym u nas poczytał ....

_________________
zapraszam na blog: http://www.mirekk36.blogspot.com (mój nick Skype: mirekk36 ) [ obejrzyj Kurs EAGLE ] [ mój kanał YT TV www.youtube.com/mirekk36 ]

Hmm... no faktycznie, macie rację
Nie wiem tylko czy będę w stanie dokładnie wytłumaczyć jak ten kod działa (zdolności dydaktycznych nie mam wcale, a konstrukcje są trochę zakręcone).

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Postaraj się wytłumaczyć w ogóle co to za urządzenie, jak działa - jego zalety, przewaga nad podobnymi rozwiązaniami - o takie rzeczy szczególnie - żeby mogło być to też inspiracją dla innych

_________________
zapraszam na blog: http://www.mirekk36.blogspot.com (mój nick Skype: mirekk36 ) [ obejrzyj Kurs EAGLE ] [ mój kanał YT TV www.youtube.com/mirekk36 ]