ATNEL tech-forum

Re: ADC-odczyt wartosci Atmega8A

2014-03-18T13:27:26+01:00

P.Mirku jak nie poprawie jak poprawie od rana do wieczora siedzę i grzebie w kodzie jednak teraz się przekonałem że wystarczy brak jednego nawiasu klamrowego i cały kod leży .

Statystyki: Napisane przez Bedi — 18 mar 2014, o 13:27

Re: ADC-odczyt wartosci Atmega8A

2014-03-16T20:19:23+01:00

Bedi napisał(a):

Tak wiem tutaj styl jest naganny po prostu to taki testowy plik gdzie wklejałem inne itp i dlatego tak wyszło staram się tylko uruchomić ADC ale nie startuje i nie wiem gdzie błąd

No to sam widzisz że pośpiech nie popłaca - więc popraw ten kod ... czy mam rozumieć że nie poprawisz ? pomimo że masz podpowiedź gdzie są błędy ? bo tak trochę nie rozumiem teraz tej wypowiedzi

Statystyki: Napisane przez mirekk36 — 16 mar 2014, o 20:19

Re: ADC-odczyt wartosci Atmega8A

2014-03-16T20:17:30+01:00

Tak wiem tutaj styl jest naganny po prostu to taki testowy plik gdzie wklejałem inne itp i dlatego tak wyszło staram się tylko uruchomić ADC ale nie startuje i nie wiem gdzie błąd

Statystyki: Napisane przez Bedi — 16 mar 2014, o 20:17

Re: ADC-odczyt wartosci Atmega8A

2014-03-16T19:59:35+01:00

Quote:

To prawdopodobnie niema znaczenia

ma ma znaczenie - bo jak widać cały kod jest takimi kwiatkami usiany

dlatego proponuję autorowi naprawdę przysiąść nad doprowadzeniem kodu do porządku i poprawnych wcięć a być może się okaże że kod ruszy z kopyta .... dlatego wciąż w książce piszę o DOBRYM stylu programowania

Statystyki: Napisane przez mirekk36 — 16 mar 2014, o 19:59

Re: ADC-odczyt wartosci Atmega8A

2014-03-16T19:57:01+01:00

To prawdopodobnie niema znaczenia ale w tych miejscach co zaznaczyłem masz niepotrzebne klamry.
Prawdopodobnie zostały po jakimś if-ie lub pętli.

[syntax=c]ISR(TIMER2_COMP_vect){

{ <-----Tutaj
if(++cnt>99) { // gdy licznik ms > 99 (minęła 1 sekunda)
sec_flag= 1; // ustaw flagę tyknięcia sekundy
{
sec++; // zwiększ licznika sekund o 1
if(sec>59) // jeśli ilość sekund > 59 to wyzeruj licznik sekund
{ sec = 0;
min++; // zwiększ licznik minut o 1
if(min>59) {
min = 0; // jeśli ilość minut > 59 to wyzeruj licznik minut
}
}
cnt = 0; // wyzeruj licznik setnych ms
}
} <-----Tutaj
}[/syntax]

Statystyki: Napisane przez kurnachatka — 16 mar 2014, o 19:57

Re: ADC-odczyt wartosci Atmega8A

2014-03-16T19:56:18+01:00

masz komunikat:

Quote:

main.c:(.text+0xe6): undefined reference to `fan_ster'

nie wiem jak ty robisz wcięcia - ale po tym co widać na forum to można oczopląsu jednak dostać a wtedy trudno znaleźć takie np kwiatki

[syntax=c]while(ADCSRA & (1< pomiar=ADC;}[/syntax]

prawda ?

Statystyki: Napisane przez mirekk36 — 16 mar 2014, o 19:56

ADC-odczyt wartosci Atmega8A

2014-03-16T19:44:13+01:00

Witam

Mam problem z odczytam wartości ADC na pinie PC5 z racji ze jest to moje pierwsze ADC opieram się na podobnym programie >>
topic4366.html

Jeszcze jeden mankament to że przy pliku mam znaczek x na czerwono a nic nie podkreśla w programie (ECLIPSE INDYGO) tak jak by nie było błędu
dodaje konsole

[syntax=c]/*

/*
* main.c
*
* Created on: 12-03-2014
* Author: MATEUSZ
*/

#include
#include

/* ### DEFINICJA PINÓW DLA DIODY LED I WENTYLATORA ### */
#define LED (1< #define FAN (1<
/* ### MAKRODEFINICJE DLA DIODY LED I WENTYLATORA ### */
#define LED_OFF PORTB &= ~LED // makrodefinicja wyłączenia diody LED
#define LED_TOG PORTB ^= LED // makrodefinicja zmiana stanu diody LED

#define FAN_ON PORTD |= FAN // makrodefinicja włączenia wentylatora
#define FAN_OFF PORTD &= ~FAN // makrodefinicja wyłączenia wentylatora

#define KEY_PIN2 (1<#define KEY_NC (PIND & KEY_PIN2)// sprawdzanie czy klawisz NC rozwarty

uint8_t key_lock;

#define KEY_PIN (1<#define KEY_NO !(PIND & KEY_PIN)// sprawdzanie czy klawisz NO zwarty

/* ### DEKLARACJE ZMIENNYCH ### */
volatile uint8_t cnt=0; // zmienna licznika 10ms
volatile uint8_t sec_flag; // flaga tyknięcia Timera0 co 1 sekundę
volatile uint8_t sec; // licznik sekund 0-59
volatile uint8_t min; // licznik minut 0-59

volatile uint16_t pomiar=0; // zmienna na potrzeby pomiaru z ADC

volatile uint8_t fan_state; // typ wyliczeniowy dla badania stanu wentylatora
enum {fan_off, fan_goes_to_off, fan_on};

/* ### DEKLARACJE FUNKCJI ### */
// deklaracja funkcji zmiany stanu wentylatora odnośnie pomiaru z ADC

void fan_ster (void); // deklaracja funkcji zmiany stanu wentylatora odnośnie pomiaru z ADC

/* ### INICJALIZACJA ### */

int main (void) {

// ustawienie timera

TCCR2 |= (1<

TCCR2 |= (1<

OCR2 = 78;// podział przez 78 rejestru porownaia 7812,5/78 = około 100Hz a to odpowiada 10ms

TIMSK |= (1<

/* ### INICJALIZACJA ADC ### */

ADCSRA |= (1<

ADCSRA |= (1<

ADMUX |= (1<

ADMUX |= (1<

ADCSRA |= (1<

DDRD &= ~KEY_PIN; // ustawienie portu jako wejscie dla klawisza NO
PORTD |= KEY_PIN;// programowe podciągnięcie do VCC

DDRD &= ~KEY_PIN2;// ustawienie portu jako wejscie dla klawisza NC
PORTD |= KEY_PIN2;//programowe podciągnięcie do VCC

// diody LED
DDRB |= LED;
DDRD |= FAN; // kierunki PIN i FAN jako wyjściowe
LED_OFF; // wyłączenie diody LED
FAN_OFF; // wyłączenie wentylatora

//włączenie przerwań
sei();

while(1) {
if (sec_flag) { // sprawdza flagę tyknięcia sekundy
if(0 == sec%3) LED_TOG; // zmień stan diody

if(1 == sec%3) {

//pomiar z ADC

ADCSRA |= (1<

while(ADCSRA & (1< pomiar=ADC;}

if(2 == sec%3) fan_ster(); // włącza lub wyłacza przekaźnika na podstawie pomiaru z ADC

}
sec_flag = 0;

}
}

/* ### USTAWIENIE LICZNIKA SEKUND I MINUT NA TIMER 0 ### */

// procedura posługi timera przerwanie

ISR(TIMER2_COMP_vect){

{
if(++cnt>99) { // gdy licznik ms > 99 (minęła 1 sekunda)
sec_flag= 1; // ustaw flagę tyknięcia sekundy
{
sec++; // zwiększ licznika sekund o 1
if(sec>59) // jeśli ilość sekund > 59 to wyzeruj licznik sekund
{ sec = 0;
min++; // zwiększ licznik minut o 1
if(min>59) {
min = 0; // jeśli ilość minut > 59 to wyzeruj licznik minut
}
}
cnt = 0; // wyzeruj licznik setnych ms
}
}
}

/* ### DEFINICJE FUNKCJI ### */

// definicja funkcji sterowania wentylatorem
void fan_ster (void) {
if (pomiar>10) // sprawdza warunek pomiaru, jeśli >400 to włącz wentylator
{
FAN_ON; // włączenie wentylatora
fan_state = fan_on; // stan wentylatora - wentylator włączony
}
if (pomiar<10 && fan_state == fan_on) { // sprawdza warunek pomiaru, jeśli <400 i wentylator włączony to przejdź do wyłączania
fan_state = fan_goes_to_off;
min = 0; // zerowanie licznika minut
sec = 0; // zerowanie licznika sekund
}
if(fan_state == fan_goes_to_off){
if(sec==5){ // po upłynięciu ok. 4-5 minut wyłącz wentylator
fan_state = fan_off; // stan wentylatora - wentylator wyłączony
FAN_OFF; // wyłącz wentylator

}
}
}
}[/syntax]

[syntax=c]/*

**** Build of configuration Release for project adc ****

make all
Building file: ../main.c
Invoking: AVR Compiler
avr-gcc -Wall -Os -fpack-struct -fshort-enums -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega8 -DF_CPU=8000000UL -MMD -MP -MF"main.d" -MT"main.d" -c -o "main.o" "../main.c"
Finished building: ../main.c

Building target: adc.elf
Invoking: AVR C Linker
avr-gcc -Wl,-Map,adc.map -mmcu=atmega8 -o "adc.elf" ./main.o
./main.o: In function `main':
main.c:(.text+0xe6): undefined reference to `fan_ster'
make: *** [adc.elf] Error 1

**** Build Finished ****[/syntax]

Statystyki: Napisane przez Bedi — 16 mar 2014, o 19:44