ATNEL tech-forum

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-17T18:10:44+01:00

MaxEnergy napisał(a):

[syntax=c]if (PINB |= (LED))[/syntax]

Tak nie odczytasz stanu. Tylko tak:
[syntax=c]if (PINB & (LED))[/syntax]

Co ciekawe to wpisanie (1<Statystyki: Napisane przez Krauser — 17 gru 2014, o 18:10

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-17T17:59:11+01:00

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.
[syntax=c]PORTB ^= (LED);

if (PINB |= (LED))
{
PORTB &=~ (LED_CZAS);
}
else
{
PORTB |= (LED_CZAS);
}[/syntax]

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

Statystyki: Napisane przez MaxEnergy — 17 gru 2014, o 17:59

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-17T15:44:03+01:00

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

Statystyki: Napisane przez Sheep — 17 gru 2014, o 15:44

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-17T15:42:51+01:00

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:[syntax=c]wynik = pomiar();[/syntax]
to pomiar W ADC wykonuje się tylko raz na obieg pętli? Dobrze pojmuję?

Statystyki: Napisane przez MaxEnergy — 17 gru 2014, o 15:42

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-16T14:48:59+01:00

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

Statystyki: Napisane przez Sheep — 16 gru 2014, o 14:48

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-16T14:47:19+01:00

Można to zrobić tak
[syntax=c]#include
#include

//zmienna globalna do której przypisany zostanie wynik pomiaru
uint16_t wynik;

uint16_t pomiar(void);

int main(void)
{
//konfuguracja pinów I/O
//konfiguracja ADC

while(1)
{
//przypisanie wyniku pomiaru do zmiennej pomiar
wynik = pomiar();

PORTB ^= (LED_CZAS); // zmień stan na diodzie czas

if (wynik < 717) //jeśli napięcie poniżej 10,5V
{
PORTB &=~ (LED); //zapal diode
}
else //jeśli napięcie wieksze noż 10,5V
{
PORTB |= (LED); //zagaś diode
}

_delay_ms(1000);
}
}

uint16_t ADC_pomiar (void) // funkcja pomiaru
{
ADCSRA |= (1< while ( ADCSRA & ADSC ); //czeka na zakończenie pomiaru
return ADCW; //zwrócenie wyniku
}[/syntax]
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
[syntax=c]while(1)
{

PORTB ^= (LED_CZAS); // zmień stan na diodzie czas

if (pomiar() < 717) //jeśli napięcie poniżej 10,5V
{
PORTB &=~ (LED); //zapal diode
}
else //jeśli napięcie wieksze noż 10,5V
{
PORTB |= (LED); //zagaś diode
}

_delay_ms(1000);
}[/syntax]

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

Statystyki: Napisane przez jacekk232 — 16 gru 2014, o 14:47

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-16T14:27:08+01:00

I już
[syntax=c]uint16_t wynik_ADC=0;
while (1) //pętla nieskończona
{
_delay_ms ( 1000 ); //poczekaj 1s
PORTB ^= (LED_CZAS); // zmień stan na diodzie czas.
wynik_ADC = ADC_pomiar;
if (wynik_ADC < 717) //jeśli napięcie poniżej 10,5V
{
PORTB &=~ (LED); //zapal diode
}
else //jeśli napięcie wieksze noż 10,5V
{
PORTB |= (LED); //zagaś diode
}
}[/syntax]
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.

Statystyki: Napisane przez Sheep — 16 gru 2014, o 14:27

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-16T14:04:24+01:00

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:[syntax=c]uint16_t ADC_pomiar (void) // funkcja pomiaru
{
ADCSRA |= (1<while ( ADCSRA & ADSC ); //czeka na zakończenie pomiaru
wynik = ADCW; //przypisanie zmiennej wynik wartości pomiaru
return wynik; //zwrócenie wyniku
}[/syntax]

Pętla while wygląda tak:[syntax=c]while (1) //pętla nieskończona
{
_delay_ms ( 1000 ); //poczekaj 1s
PORTB ^= (LED_CZAS); // zmień stan na diodzie czas.

if (wynik < 717) //jeśli napięcie poniżej 10,5V
{
PORTB &=~ (LED); //zapal diode
}
else //jeśli napięcie wieksze noż 10,5V
{
PORTB |= (LED); //zagaś diode
}
}[/syntax]

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.

Statystyki: Napisane przez MaxEnergy — 16 gru 2014, o 14:04

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-14T23:34:17+01:00

[syntax=c]while(1)
{
pomiar = ADC_pomiar();
if(pomiar>717)
/// cos tam
else
/// cos innego
}[/syntax]
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.

Statystyki: Napisane przez Sheep — 14 gru 2014, o 23:34

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-14T23:31:05+01:00

Tak, przed porównaniem zmiennej "wynik".

Statystyki: Napisane przez MaxEnergy — 14 gru 2014, o 23:31

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-14T23:26:48+01:00

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

Statystyki: Napisane przez Sheep — 14 gru 2014, o 23:26

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-14T22:54:13+01:00

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

Statystyki: Napisane przez MaxEnergy — 14 gru 2014, o 22:54

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-14T13:23:07+01:00

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.

Statystyki: Napisane przez Sheep — 14 gru 2014, o 13:23

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-14T02:03:09+01:00

Poprawione wg Twoich uwag:
[syntax=c]#include //plik nagłówkowy

#define LED (1<#define BUZZER (1<#define AKU (1<

//GŁÓWNA FUNKCJA PROGRAMU//

int main( void )
{
ADMUX = (1<| (1<
ADCSR = (1<| (1<
uint16_t wynik; //definicja zmiennej wynik

//konfiguracja wejśc/wyjśc//

DDRB |= (LED) | (BUZZER); //LED i brzęczek jako wyjście
DDRC &=~ (AKU); //wejście ADC
PORTB |= (LED); //ustawienie HIGH na diodzie
PORTB &=~ (BUZZER); //ustawienie LOW na brzęczku

while (1)
{
ADCSR |= (1<while (ADCSR & ADSC); //czeka na zakończenie pomiaru
wynik = ADCW; //pzrypisanie wartości pomiaru do wyniku

if (wynik <= 717) //jeśli napięcie mniejsze, równe 10,5V
{
PORTB &=~ (LED); //zaświec diode
}
else if ((wynik <= 751) && (wynik > 717)) //jeśli napięcie pomiędzy 11V i 10,5V
{
PORTB |= (BUZZER); //załącz brzęczek
}
}
}[/syntax]

Statystyki: Napisane przez MaxEnergy — 14 gru 2014, o 02:03

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-14T01:24:08+01:00

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ć.

Statystyki: Napisane przez Sheep — 14 gru 2014, o 01:24

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-14T00:52:06+01:00

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ą?

Statystyki: Napisane przez MaxEnergy — 14 gru 2014, o 00:52

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-14T00:34:18+01:00

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.

Statystyki: Napisane przez Sheep — 14 gru 2014, o 00:34

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-14T00:29:59+01:00

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

Statystyki: Napisane przez MaxEnergy — 14 gru 2014, o 00:29

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-14T00:26:25+01:00

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.

Statystyki: Napisane przez Sheep — 14 gru 2014, o 00:26

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-14T00:08:25+01:00

Co Wy na to?
[syntax=c]#include //plik nagłówkowy

#define LED (1<#define BUZZER (1<#define AKU (1<
uint16_t pomiar (void); //deklaracja funkcji pomiaru
uint16_t wynik; //definicja zmiennej wynik

//GŁÓWNA FUNKCJA PROGRAMU//

int main( void )
{
ADMUX = (1<| (1<
ADCSR = (1<| (1<

//konfiguracja wejśc/wyjśc//

DDRB |= (LED) | (BUZZER); //LED i brzęczek jako wyjście
DDRC &=~ (AKU); //wejście ADC
PORTB |= (LED); //ustawienie HIGH na diodzie
PORTB &=~ (BUZZER); //ustawienie LOW na brzęczku

while (1)
{
if (wynik <= 717) //jeśli napięcie mniejsze, równe 10,5V
{
PORTB &=~ (LED); //zaświec diode
}
else if ((wynik <= 751) && (wynik > 717)) //jeśli napięcie pomiędzy 11V i 10,5V
{
PORTB |= (BUZZER); //załącz brzęczek
}
}
}

uint16_t pomiar (void) //definicja funkcji pomiaru
{
ADCSR |= (1<while (ADCSR & ADSC); //czeka na zakończenie pomiaru
return wynik; //zwrócenie wyniku
}[/syntax]

Statystyki: Napisane przez MaxEnergy — 14 gru 2014, o 00:08

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-13T00:56:40+01:00

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.

Statystyki: Napisane przez Sheep — 13 gru 2014, o 00:56

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-12T08:27:19+01:00

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

Statystyki: Napisane przez gardziej — 12 gru 2014, o 08:27

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-12-12T02:08:25+01:00

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.
[syntax=c]#include //plik nagłówkowy

#define LED (1<#define BUZZER (1<#define AKU (1<
uint16_t pomiar; //deklaracja funkcji pomiaru

//GŁÓWNA FUNKCJA PROGRAMU//

int main( void )
{
ADMUX = (1<| (1<
ADCSR = (1<| (1<

//konfiguracja wejśc/wyjśc//

DDRB |= (LED) | (BUZZER); //LED i brzęczek jako wyjście
DDRC &=~ (AKU); //wejście ADC
PORTB |= (LED); //ustawienie HIGH na diodzie
PORTB &=~ (BUZZER); //ustawienie LOW na brzęczku

while (1)
{
if (pomiar <=717) //jeśli napięcie mniejsze, równe 10,5V
{
PORTB &=~ (LED); //zaświec diode
}
else if ((pomiar <=751) && (pomiar >717)) //jeśli napięcie pomiędzy 11V i 10,5V
{
PORTB |= (BUZZER); //załącz brzęczek
}
}
}

uint16_t pomiar //definicja funkcji pomiaru
{
ADCSR |= (1<while (ADSCR & ADSC); //czeka na zakończenie pomiaru
return ADCW; //zwrócenie wyniku
}[/syntax]

Kompilator wskazuje błąd w 42 linijce.

Code:

error: expected '=', ',', ';', 'asm' or '__attribute__' before '{' token

Statystyki: Napisane przez MaxEnergy — 12 gru 2014, o 02:08

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-11-25T14:45:22+01:00

MaxEnergy napisał(a):

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

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

Statystyki: Napisane przez mirekk36 — 25 lis 2014, o 14:45

Re: Prosty komparator na Atmega8. Prośba o sprawdzenie.

2014-11-25T14:40:34+01:00

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?

Statystyki: Napisane przez MaxEnergy — 25 lis 2014, o 14:40