Witam.
Proszę o sprawdzenie i wskazanie błędów w moim pierwszym kodzie w C.
Układ ma za zadanie monitorowanie napięcia na akumulatorze i sygnalizację dwóch progów, tj. 11V i 10,5V.
Powyżej 11, wszystko jest ok. Pomiędzy 10,5, a 11 powinien się załączyć brzęczek, a poniżej 10,5 led.
Schemat naszkicowany na szybko, mam nadzieję że czytelny.

Zawsze spełniony warunek, dioda świeci po wieczność.

Mogliby mnie Koledzy trochę naprowadzić na swoje spostrzeżenia?

Zobacz co to jest AKU u Ciebie i do czego porównujesz to.

Chodzi o dodanie konfiguracji AKU jako wejścia?

AKU to jest u Ciebie:

czyli porównującco jest bez sensu, bo ten warunek i drugi też jest zawsze spełniony

W takim razie pytanko...
Jaka komenda służy do odczytu wartości na wejściu ADC, aby później przyrównać ją do 717?

Moje spostrzeżenie - kolega musi jednak zacząć konkretnie od PODSTAW języka C ... bo na razie to działasz totalnie po omacku - na zasadzie "albo się uda albo się nie uda" ..... "a jak się nie uda to może uda się jak napiszę odwrotnie?"

wiesz ile czasu zmarnujesz na takie podejście ?

może warto zacząć od książki: http://atnel.pl/mikrokontrolery-avr-jezyk-c.html

a na dokładkę obejrzeć sobie poradnik o ADC: https://www.youtube.com/playlist?list=P ... 5hRnig_f23

bo z takimi kwiatkami jak z tym swoim AKU - nic nie zrobisz

_________________
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 ]

Witam.
Za poradą Mirka wziąłem się za naukę. Dużo zrozumiałem, lecz pewnie nie wszystko. Kod napisałem od nowa. Po kompilacji wyskoczył mi błąd z którym nie za bardzo wiem co zrobić. Proszę o sprawdzenie czy ma on szansę działać. Założenia jak wyżej.


Kompilator wskazuje błąd w 42 linijce.

Gdzieś Ci zjadło sekcje argumentów w deklaracji funkcji.

Jaśniej po prostu wstaw nawiasy po słowie pomiar a w nim słowko void.
Każda funkcja posiada nawiasy okrągłe po jej nazwie, zarówno w deklaracji jak i przy jej wywoływaniu.
Tak samo w lini 30 i 34 musisz to zrobić(dodać nawiasy). Ja natomiast proponuję Ci byś wartość pomiaru wrzucił do jakiejś zmiennej i porównywał zmienną z Twoją wartością 717 bo niepotrzebnie dwa razy w jednym przebiegu pętli wykonujesz pomiar ADC.

Co Wy na to?

Sprawdzałeś to czy od razu na forum?
Bo ja nie widzę w while(1) wykonywania się funkcji pomiar zwracającej coś do zmiennej wynik.

Sprawdzić na razie nie mam na czym, bo programator jeszcze nie doszedł.
Rozumiem twoją uwagę. Coś wykombinuję.

Zaraz zaraz a funkcja pomiar skąd się wzięła? Sam ją napisałeś czy źródło?
Zobacz, zwracasz zmienną wynik, której nawet nie masz w tej funkcji, ani nic do tej zmiennej wynik nie zapisujesz. Wynikiem pomiaru jest rejestr ADCW (u mnie tak było)
Mogę Ci oczywiście pokazać kod na pomiar dla medzi8 ale pytanie czy chciałbyś gotowe rozwiązanie.

Gotowca nie chcę. Sam muszę do tego dojść. Ty mnie tylko lekko naprowadzaj.
Jeżeli wynik pomiaru jest zapisywany w 10 najmłodszych bitach 16 bitowego rejestru ADCW, to wystarczy przypisać ową wartość do zmiennej "wynik", czy trzeba ją przekonwerterować na wartość dziesiętną?

zwracasz rejestr ADCW, więc jeśli funkcja zwraca jego wartość to wrzucaj wartość zwracaną przez tą funkcję do zmiennej wynik. Potem dopiero ją porównuj ile wynosi czy zapalić diodę czy też nie. Bo aktualnie to porównuje czy wynik jest <717, który nie przechowuje żadnej wartości z pomiaru, tylko jest w nim zero (zmienne globalne automatycznie są zerowane) Nic nie trzeba konwertować ani maskować itp. (przynajmniej ja nie musiałem). Popraw po prostu funkcje odpowiedzialna za pomiar bo nie tak powinna ona wyglądać.

Poprawione wg Twoich uwag:

Wygląda nieco lepiej, ale nie ma takiego rejestru w atmega8 jak ADCSR, jest ADCSRA.
W dodatku wrzuć pomiar napięcia w funkcję jak miałeś na początku będziesz wiedział za miesiąc jak wrócisz do programu, które instrukcje odpowiadają za pomiar.
Kompilowałeś w ogóle ten poprawiony kod?
Drugie pytanie czy potrzebujesz tak często pomiar tego napięcia (robisz to cały czas w pętli, tak szybko pada ten akumulator?, może co sekundę)?
Bo jeśli napięcie będzie "skakało" między napięciem granicznym to co chwila będziesz włączał buzzer i go wyłączał. To samo z diodą, nie masz warunku, który gasi diodę, gdy napięcie jest większe, równe 10.5V, po naładowaniu aku dioda już się nie zgasi.

Małe pytanko.
W jaki sposób zainicjować funkcję (w moim przypadku "pomiar") na początku pętli?

Jeśli dobrze zrozumiałem frazę 'na początku' no to zaraz przy otwarciu klamer od while(1) o to chodziło?

Tak, przed porównaniem zmiennej "wynik".

Ale daj jakieś opóźnienie by nie było problemu przy napięciu granicznym tej wartości, i weź też pod uwagę, że dokładność pomiaru adc (u mnie tak jest) wynosi +/- 3 wartości.

Nie mogę dać sobie rady z wywołaniem mojej funkcji na początku pętli while(1).
Pomiar napięcia ADC jest wrzucony do funkcji:

Pętla while wygląda tak:


Chcę aby za klamerką while następowało wywołanie funkcji pomiaru, a póżniej przyrównanie. Możesz coś podpowiedzieć, bo Twój przykład jakoś nie chce bujać.

P.S.
Led_czas jest tylko diodą sygnalizacyjną powtórzenie pętli.

I już

proszę

EDIT: Twoim problemem pewnie było zrozumienie dlaczego do zmiennej wynik nie wpada wartość, którą potem porównujesz w ifie, zgadłem?
Chodzi o to, że Twoja funkcja coś zwraca (typ liczby całkowitej przed nazwą funkcji w deklaracji), a jeśli zwraca to musisz przypisać wartość jej wywołania do jakiejś zmiennej, którą potem porównujesz w if'ie, który potem dobrze stworzyłeś.
EDIT2: Możesz od razu zwracać return ADCW bez przypisywania zmiennej wynik wartości ADC.
Podpowiedziałem Ci za dużo, reszte kombinuj sam. To są niejako podstawy, które możesz zdobyć sam.

Można to zrobić tak

Czyli przypisując wynik pomiaru do zmiennej i potem tą zmienną porównywać w ifach.

Jeśli masz tylko jednego ifa do porównania to można zrezygnować ze zmiennej wynik i funkcję pomiar() wywołać bezpośrednio w ifie


Rozwiązań może być tyle ilu jest programistów.

Ja jeszcze tu z trzy minimum widzę na szybki rzut oka. Ale tak jak @jacek232 wspomniał wiele jest rozwiązań, wybierz ten zapis, który jest dla CIebie najłatwiejszy i przede wszystkim działa

Dzięki wielkie, wszystko działa jak należy.
Jeśli przyrównujemy funkcję pomiar bezpośrednio w if'ie, to za każdym przyrównaniem następuje pomiar w ADC, a jeśli najpierw przypiszemy wynik do zmiennej (np. na początku pętli), np tak:
to pomiar W ADC wykonuje się tylko raz na obieg pętli? Dobrze pojmuję?

Tak dokładnie tak jak mówisz
wrzucaj wartość pomiaru do zmiennej i potem porównuj zmienną.

Czy istnieje jakaś metoda, aby odczytać stan pinu wyjściowego?
Odczytywanie w taki sposób, jak się odczytuje pin wejściowy, nie przynosi zamierzonego efektu.


Zamierzenie jest takie, aby LED_CZAS podążała za LED.