Timerprobleme

[Razer]

Hallo an alle

Ich hab ein Programm geschrieben, das alle ~5 Sekunden einen Wert über A2D einließt (Timergesteuert). Wenn dieser Wert unter 614 ist, wird dieser im Eeprom abgelegt und ein Zähler um 1 erhöht. Doeser wird auch im Eeprim abgelegt. Erst wenn der Wieder einmal über 614 war, kann der Wert und der Zähler um eins erhöht werden. Das ganze hab ich für 2 A2D Kanäle gemacht. Weiters hab ich eine Unterfunktion zum Auslesen der Wert aus dem Eeprom. (FUnktionsaufruf wenn PD7 gedrückt ist).

Nur leider funktioniert es nicht wie sollte. Bei jeder MEssungen kommen zusätzlich Zeichen aufs Display die eigentlich nicht dort hingehören. Ich weiß aber nicht warum :(

Sobald ein falsches Zeichen aus der main (messfunktion) am Display ist, werden bei der lesefunktion auch falsche Zeichen Dargestellt.

Das Abspeichern im Eeprom funktioniert auch nicht.

Hie nun mal der Code:



#include <avr/io.h>
#include <stdlib.h>
#include <avr/eeprom.h>
#include <avr/signal.h>
#include <avr/interrupt.h>
#include "avrlibdefs.h"
#include "avrlibtypes.h"

#include "a2d.h"
#include "lcd.h"

volatile uint16_t zelle[1];
volatile uint16_t counter =0;

int lcd_maske(void);
int lcd_put_f(uint16_t adc);
int timer_init (void);

int timer_init(void)
{
TIMSK = (1<<TOIE0);
TCNT0 = 0;
TCCR0 = (1<<CS02);

return(0);
}

int lcd_maske (void)
{
lcd_init(LCD_DISP_ON);
lcd_home();
lcd_clrscr();
lcd_puts(" Unterspannungsanzeige");
lcd_gotoxy(0,1);
lcd_puts("Zelle 1:");
lcd_gotoxy(0,2);
lcd_puts("Zelle 2:");
return(0);
}


int lcd_put_f (uint16_t adc) //Zum anzeigen der Spannungen am Display
{
float ganzzahl,komma;
uint8_t int_ganzzahl, int_komma;

ganzzahl = (adc/1024.0) * 5;
komma = (ganzzahl - (int) ganzzahl)*100;

int_ganzzahl = (int)ganzzahl;
int_komma = komma;

lcd_put_d(int_ganzzahl);
lcd_putc('.');
lcd_put_d(int_komma);
return(0);
}

SIGNAL (SIG_OVERFLOW0)
{
counter++;

if(counter == 600) //5s
{
counter=0;
zelle[0] = a2dConvert10bit(0);
zelle[1] = a2dConvert10bit(1);
PORTD ^= (1<<0);
}
}

int lesen (void)
{
cli();
lcd_clrscr();
lcd_home();
lcd_puts("Auslesen der Werte");
lcd_gotoxy(0,1);
lcd_puts("Niedrigster Wert Anzahl\n");
lcd_puts("Zelle 1:\n");
lcd_puts("Zelle 2:");

lcd_gotoxy(9,2);
lcd_put_f(eeprom_read_word((uint16_t*) 2));

lcd_gotoxy(9,3);
lcd_put_f(eeprom_read_word((uint16_t*) 4));

lcd_gotoxy(20,2);
lcd_put_d(eeprom_read_byte((uint8_t*)0));

lcd_gotoxy(20,3);
lcd_put_d(eeprom_read_byte((uint8_t*)1));

loop_until_bit_is_set(PIND, PD7);

lcd_clrscr();
lcd_home();
lcd_maske();
sei();
return(0);
}

int main (void)
{
uint8_t i, istpositiv[1], zaehler[1];

istpositiv[0] = 1;
istpositiv[1] = 1;

DDRC &=~ ((1<<PC0) | (1<<PC1));
PORTC &=~ ((1<<PC0) | (1<<PC1));
DDRD |= (1<<0);

a2dInit();
a2dSetPrescaler(ADC_PRESCALE_DIV32);
a2dSetReference(ADC_REFERENCE_AVCC);
lcd_maske();
timer_init();
sei();

while(1)
{
if(bit_is_clear(PIND, PD7) == 1)
lesen();

for(i=0; i <2; i++)
{

if(zelle[i] > 614)
istpositiv[i] = 1;

if(istpositiv[i] == 1)
{
if(zelle[i] < 614)
{
zaehler[i]++;
cli();
eeprom_write_byte((uint8_t*)i, zaehler[i]);
eeprom_write_word((uint16_t*)(i*2)+2, zelle[i]);
sei();
istpositiv[i] =0;
}
}
}

lcd_gotoxy(9,1);
lcd_put_f(a2dConvert10bit(0));
lcd_putc('V');
lcd_gotoxy(9,2);
lcd_put_f(a2dConvert10bit(1));
lcd_putc('V');
}
return(0);
}


Als Prozessor verwende ich einen Mega8

Danke im Voraus

Gruß Robert

[SprinterSB]

Das Abtesten (auch Lesen und Schreiben) mehrbytiger Werte musst du atomar machen, wenn das Ding in einer ISER geändert wird. Schau mal ins Wiki bei GCC oder den Fallstricken bei C-Programmierung, da hab ich das erklärt.

[Razer]

Hallo

Ganz gena versteh ich das noch nicht. Währen dem Lesen/Schreiben von Daten in den EEprom darf kein Interrup auftreten, oder??

Also wenn ich in der ISR vor dem Einlesen der beiden A2D Werte die Interrupts außschalte und danach wieder einschalte, geht das trotzdem nicht. Sobald ich die Funktion Lesen aufrufe, schalte ich ich die Interrupts aus. Beim Verlassen schalte ich sie wieder ein.

Vor EEprom Aktionen, schalte ich sie ja immer aus.

Danke im Voraus

Gruß Robert

[SprinterSB]

if(zelle[i] > 614)

Das ist ein Befehl, der auf AVR nicht in eine einzige Instruktion umgesetzt werden kann. Wird der Wert gegen 614 verglichen, was durch die Sequenz cp und cpc geschieht und taucht dazwischen ne IRQ auf, die bedient wird und ändert die den Wert, dann hat ein Byte den alten Wert und ein Byte cshon den den neuen Wert. Insgesamt ist der Wert korupt, so daß die Bedingung x>614 auch erfüllt sein kann, wenn x<= 614 ist bzw falsch sein kann, wenn x>614 ist.

https://www.roboternetz.de/wissen/index.php/Fallstricke_bei_der_C-Programmierung#Nicht-atomarer_Code

[Razer]

Hallo Georg - Johan

Das verstehe ich einmal. Das die Avrs nur mit 8 Bit Werten in "einem" rechnen können. Ich hab jetzt vor hedem Wert der größer als 8 Bit ist die Interrupts ausgeschaltet. Jedoch funktioniert es immer noch nicht :(

Bei jeder Messung (if(counter = 610)) werden am Display falsche Zeichen angezeigt. Wenn ich nan in die lEsen Funktion umschalte hab ich wieder flasche Zeichen. Da hilft nur ein Reset. Die Werte werden auch nicht Richtig in den EEprom geschrieben bzw nicht Richtig ausgelesen.

Kann es sein das hier irgendwo der Fehler steckt. Das die Adressierung der EEprom Zellen nicht stimmt?

Hier nun mal der Code:



#include <avr/io.h>
#include <stdlib.h>
#include <avr/eeprom.h>
#include <avr/signal.h>
#include <avr/interrupt.h>
#include "avrlibdefs.h"
#include "avrlibtypes.h"

#include "a2d.h"
#include "lcd.h"

volatile uint16_t zelle[1];
volatile uint16_t counter =0;

int lcd_maske(void);
int lcd_put_f(uint16_t adc);
int timer_init (void);

int timer_init(void)
{
TIMSK = (1<<TOIE0);
TCNT0 = 0;
TCCR0 = (1<<CS02);

return(0);
}

int lcd_maske (void)
{
lcd_init(LCD_DISP_ON);
lcd_home();
lcd_clrscr();
lcd_puts(" Unterspannungsanzeige");
lcd_gotoxy(0,1);
lcd_puts("Zelle 1:");
lcd_gotoxy(0,2);
lcd_puts("Zelle 2:");
return(0);
}


int lcd_put_f (uint16_t adc) //Zum anzeigen der Spannungen am Display
{
float ganzzahl,komma;
uint8_t int_ganzzahl, int_komma;

cli();
ganzzahl = (adc/1024.0) * 5;
komma = (ganzzahl - (int) ganzzahl)*100;
sei();

int_ganzzahl = (int)ganzzahl;
int_komma = komma;

lcd_put_d(int_ganzzahl);
lcd_putc('.');
lcd_put_d(int_komma);
return(0);
}

SIGNAL (SIG_OVERFLOW0)
{
counter++;

if(counter == 600) //5s
{
counter=0;
cli();
zelle[0] = a2dConvert10bit(0);
zelle[1] = a2dConvert10bit(1);
PORTD ^= (1<<0);
sei();
}
}

int lesen (void)
{
cli();
lcd_clrscr();
lcd_home();
lcd_puts("Auslesen der Werte");
lcd_gotoxy(0,1);
lcd_puts("Niedrigster Wert Anzahl\n");
lcd_puts("Zelle 1:\n");
lcd_puts("Zelle 2:");

lcd_gotoxy(9,2);
lcd_put_f(eeprom_read_word((uint16_t*) 2));

lcd_gotoxy(9,3);
lcd_put_f(eeprom_read_word((uint16_t*) 4));

lcd_gotoxy(20,2);
lcd_put_d(eeprom_read_byte((uint8_t*)0));

lcd_gotoxy(20,3);
lcd_put_d(eeprom_read_byte((uint8_t*)1));

loop_until_bit_is_set(PIND, PD7);

lcd_clrscr();
lcd_home();
lcd_maske();
sei();
return(0);
}

int main (void)
{
uint8_t i, istpositiv[1], zaehler[1];

istpositiv[0] = 1;
istpositiv[1] = 1;

DDRC &=~ ((1<<PC0) | (1<<PC1));
PORTC &=~ ((1<<PC0) | (1<<PC1));
DDRD |= (1<<0);

a2dInit();
a2dSetPrescaler(ADC_PRESCALE_DIV32);
a2dSetReference(ADC_REFERENCE_AVCC);
lcd_maske();
timer_init();
sei();

while(1)
{
if(bit_is_clear(PIND, PD7) == 1)
lesen();

for(i=0; i <2; i++)
{
cli();
if(zelle[i] > 614)
sei();
istpositiv[i] = 1;

if(istpositiv[i] == 1)
{
cli();
if(zelle[i] < 614)
{
sei();
zaehler[i]++;
cli();
eeprom_write_byte((uint8_t*)i, zaehler[i]);
eeprom_write_word((uint16_t*)(i*2)+2, zelle[i]);
sei();
istpositiv[i] =0;
}
}
}

lcd_gotoxy(9,1);
lcd_put_f(a2dConvert10bit(0));
lcd_putc('V');
lcd_gotoxy(9,2);
lcd_put_f(a2dConvert10bit(1));
lcd_putc('V');
}
return(0);
}


Ich hoffe mir kann jemand helfen

Gruß Robert

[NumberFive]

wenn ich SprinterSB richtig verstanden habe geht es bei dem Interupt problem nur um variablen die so woll in einer Interrupt routine ab gefragt / geändert werden als auch im Haupt Programm. Nich immer wenn eine grössere Variable auf tausch muß man die Interupts abschalten.

Aber das Problem gibt es nicht nur beim AVR sonder so was Ähnliches hatte ich auch schon mal bei einer Funktion unter windows die eindeutige Zaehler in eine Multithread Programm erzeugen sollte.

mir ist überhaupt nicht klar wie das Funktionieren soll auf den wert in zelle wir ja nie zu gegriffen. oder ? auf die Funktion lcd_put_f auch nicht.
Ich geben zu so satel fest bin ich in AVR C noch nicht aber hier fehlt doch die Hälfte. Was mir auch fehlt ist das warten bis der AD wandler fertig ist aber das kann ja sein das das deine Includes machen. das habe ich bei mir alles selbst geschrieben.

Gruß

[SprinterSB]

Hier nun mal der Code:



#include <avr/io.h>
#include <stdlib.h>
#include <avr/eeprom.h>
#include <avr/signal.h>
#include <avr/interrupt.h>
#include "avrlibdefs.h"
#include "avrlibtypes.h"

#include "a2d.h"
#include "lcd.h"

!!! Du greifst auf zelle[0] und zelle[1] zu, definierst zelle[] aber nur zu Länge1, das crasht!!! Es muss mindestens Länge 2 haben!
volatile uint16_t zelle[1];

volatile uint16_t counter =0;

int lcd_maske(void);
int lcd_put_f(uint16_t adc);
int timer_init (void);

int timer_init(void)
{
TIMSK = (1<<TOIE0);
TCNT0 = 0;
TCCR0 = (1<<CS02);

return(0);
}

int lcd_maske (void)
{
lcd_init(LCD_DISP_ON);
lcd_home();
lcd_clrscr();
lcd_puts(" Unterspannungsanzeige");
lcd_gotoxy(0,1);
lcd_puts("Zelle 1:");
lcd_gotoxy(0,2);
lcd_puts("Zelle 2:");
return(0);
}


int lcd_put_f (uint16_t adc) //Zum anzeigen der Spannungen am Display
{
float ganzzahl,komma;
uint8_t int_ganzzahl, int_komma;

cli();
ganzzahl = (adc/1024.0) * 5;
komma = (ganzzahl - (int) ganzzahl)*100;
sei();

int_ganzzahl = (int)ganzzahl;
int_komma = komma;

lcd_put_d(int_ganzzahl);
lcd_putc('.');
lcd_put_d(int_komma);
return(0);
}

SIGNAL (SIG_OVERFLOW0)
{
counter++;

if(counter == 600) //5s
{
counter=0;
cli();
zelle[0] = a2dConvert10bit(0);
zelle[1] = a2dConvert10bit(1);
PORTD ^= (1<<0);
sei();
}
}

int lesen (void)
{
cli();
lcd_clrscr();
. lcd_home();
lcd_puts("Auslesen der Werte");
lcd_gotoxy(0,1);
lcd_puts("Niedrigster Wert Anzahl\n");
lcd_puts("Zelle 1:\n");
lcd_puts("Zelle 2:");

lcd_gotoxy(9,2);
lcd_put_f(eeprom_read_word((uint16_t*) 2));

lcd_gotoxy(9,3);
lcd_put_f(eeprom_read_word((uint16_t*) 4));

lcd_gotoxy(20,2);
lcd_put_d(eeprom_read_byte((uint8_t*)0));

lcd_gotoxy(20,3);
lcd_put_d(eeprom_read_byte((uint8_t*)1));

loop_until_bit_is_set(PIND, PD7);

lcd_clrscr();
lcd_home();
lcd_maske();
sei();
return(0);
}

int main (void)
{
uint8_t i, istpositiv[1], zaehler[1];

istpositiv[0] = 1;
istpositiv[1] = 1;

DDRC &=~ ((1<<PC0) | (1<<PC1));
PORTC &=~ ((1<<PC0) | (1<<PC1));
DDRD |= (1<<0);

a2dInit();
a2dSetPrescaler(ADC_PRESCALE_DIV32);
a2dSetReference(ADC_REFERENCE_AVCC);
lcd_maske();
timer_init();
sei();

while(1)
{
if(bit_is_clear(PIND, PD7) == 1)
lesen();

for(i=0; i <2; i++)
{
cli();
if(zelle[i] > 614)
sei();
istpositiv[i] = 1;

if(istpositiv[i] == 1)
{
cli();
if(zelle[i] < 614)
{
sei();
zaehler[i]++;
cli();
eeprom_write_byte((uint8_t*)i, zaehler[i]);
eeprom_write_word((uint16_t*)(i*2)+2, zelle[i]);
sei();
istpositiv[i] =0;
}
}
sei(); !!! IRQs müssen auch wieder eingeschaltet werden, wenn if(...) falsch ist.
}

lcd_gotoxy(9,1);
lcd_put_f(a2dConvert10bit(0));
lcd_putc('V');
lcd_gotoxy(9,2);
lcd_put_f(a2dConvert10bit(1));
lcd_putc('V');
}
return(0);
}


Ich hoffe mir kann jemand helfen

Gruß Robert

Die Änderungen hab ich mit !!! gekennzeichnet.

[Razer]

Hallo an alle

@Number Five:

Der Wert in zelle wird in den Eeprom geschrieben oder auch ausgelesen (Lesefunktion). Die Funktion lcd_put_f wird sehr wohl angewendet. Genau schauen... ;)

Danke SprinterB :) Da mit der Arraygröße hab ich mich vertan :(. Jedoch funktioniert der Zähler für den Eeprom noch nicht Richtig. Der EEprom hat doch nach einem Flash den Wert 255 (0xFF). Jedoch stimmt der Wert beim nächsten Zählerzugriff nicht und es steht ein falscher Wert. Ab hier zählt nun der Zähler im um 1 aufwärts.

Was ist da los?? Wann wird der EEprom gelöscht?? Nur wenn ich das explizit im AVRDUDE mach???

Weiters versteh ich noch nicht das Auslesen. Das Auslesen der WORDs funktioniert nur mit den Adressen 4,6. Ich dacht die Adressen sind 2,4.

Warum??

Hier nun der Code




#include <avr/io.h>
#include <stdlib.h>
#include <avr/eeprom.h>
#include <avr/signal.h>
#include <avr/interrupt.h>
#include "avrlibdefs.h"
#include "avrlibtypes.h"

#include "a2d.h"
#include "lcd.h"

volatile uint16_t zelle[2];
volatile uint16_t counter =0;

int lcd_maske(void);
int lcd_put_f(uint16_t adc);
int timer_init (void);

int timer_init(void)
{
TIMSK = (1<<TOIE0);
TCNT0 = 0;
TCCR0 = (1<<CS02);

return(0);
}

int lcd_maske (void)
{
lcd_init(LCD_DISP_ON);
lcd_home();
lcd_clrscr();
lcd_puts(" Unterspannungsanzeige");
lcd_gotoxy(0,1);
lcd_puts("Zelle 1:");
lcd_gotoxy(0,2);
lcd_puts("Zelle 2:");
return(0);
}


int lcd_put_f (uint16_t adc) //Zum anzeigen der Spannungen am Display
{
float ganzzahl,komma;
uint8_t int_ganzzahl, int_komma;

cli();
ganzzahl = (adc/1024.0) * 5;
komma = (ganzzahl - (int) ganzzahl)*100;
sei();

int_ganzzahl = (int)ganzzahl;
int_komma = komma;

lcd_put_d(int_ganzzahl);
lcd_putc('.');
lcd_put_d(int_komma);
return(0);
}

SIGNAL (SIG_OVERFLOW0)
{
counter++;

if(counter == 600) //5s
{
counter=0;
cli();
zelle[0] = a2dConvert10bit(0);
zelle[1] = a2dConvert10bit(1);
PORTD ^= (1<<0);
sei();
}
}

int lesen (void)
{
cli();
lcd_clrscr();
lcd_home();
lcd_puts("Auslesen der Werte");
lcd_gotoxy(0,1);
lcd_puts("Niedrigster Wert Anzahl\n");
lcd_puts("Zelle 1:\n");
lcd_puts("Zelle 2:");

lcd_gotoxy(9,2);
lcd_put_f(eeprom_read_word((uint16_t*) 4));

lcd_gotoxy(9,3);
lcd_put_f(eeprom_read_word((uint16_t*) 6));

lcd_gotoxy(20,2);
lcd_put_d(eeprom_read_byte((uint8_t*)0));

lcd_gotoxy(20,3);
lcd_put_d(eeprom_read_byte((uint8_t*)1));

loop_until_bit_is_set(PIND, PD7);

lcd_clrscr();
lcd_home();
lcd_maske();
sei();
return(0);
}

int main (void)
{
uint8_t i, istpositiv[2], zaehler[2];

istpositiv[0] = 1;
istpositiv[1] = 1;

DDRC &=~ ((1<<PC0) | (1<<PC1));
PORTC &=~ ((1<<PC0) | (1<<PC1));
DDRD |= (1<<0);

a2dInit();
a2dSetPrescaler(ADC_PRESCALE_DIV32);
a2dSetReference(ADC_REFERENCE_AVCC);
lcd_maske();
timer_init();
sei();

while(1)
{
if(bit_is_clear(PIND, PD7) == 1)
lesen();

for(i=0; i <2; i++)
{
cli();
if(zelle[i] > 614)
{
sei();
istpositiv[i] = 1;
}
sei();
if(istpositiv[i] == 1)
{
cli();
if(zelle[i] < 614)
{
sei();
zaehler[i]++;
cli();
eeprom_write_byte((uint8_t*)i, zaehler[i]);
eeprom_write_word((uint16_t*)(i*2)+2, zelle[i]);
sei();
istpositiv[i] =0;
}
}
sei();
}

lcd_gotoxy(9,1);
lcd_put_f(a2dConvert10bit(0));
lcd_putc('V');
lcd_gotoxy(9,2);
lcd_put_f(a2dConvert10bit(1));
lcd_putc('V');
}
return(0);
}


Danke im Voraus

Gruß Robert

[SprinterSB]

Da ist noch ein Schnatzer:

(uint16_t*)(i*2)+2

Du castest den int i*2 nach uint16_t*. Auf diesen Pointer addierst du 2, was nach der Semantik von C ist +2*sizeof(uint16_t), also den Pointer um 2 Einheiten worauf er zeigt, weiterzählen.

Was du wahrscheinlich meinst ist

(uint16_t*)((i*2)+2)

Erst die Addition als Skalar, und danach erst den Cast?

[Razer]

Hallo Georg-Johann

Danke :) Ich dachte der Term wird als ganzes gesehen.

Es funktioniert nun eigentlich schon fast. Jedoch hab ich noch ein Problem. Das Auslesen oder das Abspeichern der 16Bit Werte funktioniert noch nicht ganz Richtig. Nach einem Reset ist das Flag für das Abspeichern auf 1 gesetzt. Es soll ein Abspeichern erfolgen wenn der ADC Wert unter 614 ist. Jedoch funktioniert das nicht :(



/*================================================= ================
Programmtitel: Unterspannungsaanzeige
Autor: Robert Schilling
Datum: 2.3.2006
Hardware: LCD an PORTB
Led an PortD0
Poti an PC0, PC1
Schalter an PD7

Programmbeschreibung:
Die beiden Potis simulieren 2 Lipo Zellen.
Sobald der Spannungswert unter 3V Wird ein Zähler um
1 erhöht und der Wert in den EEprom geschrieben.
Es wenn der Wert wieder über 3V steigt wird die
nächste Unterspannung "gewertet" und in den EEprom geschrieben.
Die Messungen sind Timergesteuert (8bit Timer0). Sie
lösen einen Overflow Interrupt aus. Sobald 600 Interrupts (~5sek)
werden die Zellen gemessen.

================================================== ==================*/
#include <avr/io.h>
#include <stdlib.h>
#include <avr/eeprom.h>
#include <avr/signal.h>
#include <avr/interrupt.h>
#include "avrlibdefs.h"
#include "avrlibtypes.h"

#include "a2d.h"
#include "lcd.h"

volatile uint16_t zelle[2]; //Varibale die von der Interruptroutine
volatile uint16_t counter =0; //und auch von anderen Funktionen gelesen werden können

int lcd_maske(void); //LCD Maske
int lcd_put_f(uint16_t adc); //Funktion zum Anzeigen des Spannungswertes
int timer_init (void); //Initialisieren des Timers

int timer_init(void) //Initialisieren des Timers
{
TCCR0 = (1<<CS02); //Freq = F_CPU/256
TCNT0 = 0; //Counter = 0
TIMSK = (1<<TOIE0); //Enable Overflow Interrupts
return(0);
}

int lcd_maske (void) //Aufbau der LCD Maske
{
lcd_init(LCD_DISP_ON); //LCD initialisiern
lcd_home(); //lcd home
lcd_clrscr(); //Clear Screen
lcd_puts(" Unterspannungsanzeige");
lcd_gotoxy(0,1);
lcd_puts("Zelle 1:");
lcd_gotoxy(0,2);
lcd_puts("Zelle 2:");
return(0);
}


int lcd_put_f (uint16_t adc) //Zum anzeigen der Spannungen am Display
{ //ADC Wert wird in eine Spannung umgerechnet
float ganzzahl,komma;
uint8_t int_ganzzahl, int_komma;

cli(); //Disable Interrupts
ganzzahl = (adc/1024.0) * 5;
komma = (ganzzahl - (int) ganzzahl)*100;
sei(); //Enable Interrupts

int_ganzzahl = (int)ganzzahl;
int_komma = komma;

lcd_put_d(int_ganzzahl);
lcd_putc('.');
lcd_put_d(int_komma);
lcd_putc('V');
return(0);
}

SIGNAL (SIG_OVERFLOW0) //Overflow Routine
{
counter++; // bei jedem Overflow counter++

if(counter == 600) //Wenn counter = 600 (~5sek)
{
counter=0;
cli(); //Disable Interrupts
zelle[0] = a2dConvert10bit(0); //Messen der Werte
zelle[1] = a2dConvert10bit(1);
sei(); //Enable Interrupts
PORTD ^= (1<<0); //Led umschalten

}
}

int lesen (void) //Lesefunktion zum Auslesen der Werte
{
cli(); //Disable Interrupts
lcd_clrscr();
lcd_home();
lcd_puts("Auslesen der Werte");
lcd_gotoxy(0,1);
lcd_puts("Niedrigster Wert Anzahl\n");
lcd_puts("Zelle 1:\n");
lcd_puts("Zelle 2:");

lcd_gotoxy(9,2);
lcd_put_f(eeprom_read_word((uint16_t*) 2)); //Wert von Zelle 1 auslesen

lcd_gotoxy(9,3);
lcd_put_f(eeprom_read_word((uint16_t*) 4)); //Wert von Zelle 2 auslesen

lcd_gotoxy(20,2);
lcd_put_d(eeprom_read_byte((uint8_t*)0)); //Anzahl der Unterspannungen vo Zelle1

lcd_gotoxy(20,3);
lcd_put_d(eeprom_read_byte((uint8_t*)1)); //Anzahl der Unterspannungen von Zelle2

loop_until_bit_is_set(PIND, PD7); //warten bis Schalter an PD7 wieder ausgeschalten ist.

lcd_clrscr();
lcd_home();
lcd_maske();
sei(); //Enable Interrupts
return(0);
}

int main (void) //Mainfunktion
{
uint8_t i, istpositiv[2], zaehler[2];

istpositiv[0] = 1; //Flag ob positiv
istpositiv[1] = 1;
zaehler[0] = eeprom_read_byte((uint8_t*) 0); //Zähler aus dem EEprom auslesen
zaehler[1] = eeprom_read_byte((uint8_t*) 1); //Zähler aus dem EEprom auslesen

if(zaehler[0] == 255) //Wenn Zähler =255 = 0xFF (nach dem Flashen)
zaehler[0] = 0; //Zähler = 0;

if(zaehler[1] == 255)
zaehler[1] = 0;

DDRC &=~ ((1<<PC0) | (1<<PC1)); //PC0, PC1 Eingänge
PORTC &=~ ((1<<PC0) | (1<<PC1)); //disabke pullups
DDRD &=~ (1<<7); //PD7 Eingang (schalter)
PORTD &=~ (1<<7); //Disable pullups
DDRD |= (1<<0); //PD0 Ausgang
PORTD &=~ (1<<0);

a2dInit(); //A2D initialisiern
a2dSetPrescaler(ADC_PRESCALE_DIV32); //Prescaler = 32
a2dSetReference(ADC_REFERENCE_AVCC); //Refernzspannung = Versorgungsspanung
lcd_maske();
timer_init(); //Timer initialisieren
sei();

while(1)
{
cli();
if(bit_is_clear(PIND, PD7) == 1) //PD7 auf high geschalten ist
lesen(); //Funktion lesen aufrufen (Auslesen der Werte)
sei();

for(i=0; i <2; i++) //Auswerten der Messungen
{
if(zelle[i] > 614) //wenn Zelle > 3V
{
istpositiv[i] = 1; //Flag setzen
}

if(istpositiv[i] == 1) //Wenn Flag gesetzt
{
if(zelle[i] < 614) //Wenn zelle i < 3V
{
zaehler[i]++; //Zähler um eins erhöhen
cli(); //Disable Interrups
eeprom_write_byte((uint8_t*)i, zaehler[i]); //wert des Zählers in den EEprim schreiben
eeprom_write_word((uint16_t*)((i*2)+2), zelle[i]); //Wert der Spannung in EEProm schreiben
sei(); //Enable Interrupts
istpositiv[i] =0; //Flag löschen
}
}
}

lcd_gotoxy(9,1);
lcd_put_f(a2dConvert10bit(0)); //Spannungswert anzeigen
lcd_gotoxy(9,2);
lcd_put_f(a2dConvert10bit(1)); //Spannungswert anzeigen
}
return(0);
}



Beim Auslesen wird immer 0.0V dargestellt. Bis ich den Wert ändere. Warum?

Danke im Voraus

Gruß Robert[/code]

[SprinterSB]

Das ist ja recht viel Code... so ohne weiteres seh ich da nicht, ob er das machst, was du von ihm willst das er er tun soll...

Nur noch eins: bit_is_clear ist warscheinlich ein Prädikat, liefert also 0, falls die Bedingung nicht erfüllt ist (falls PINx == HIGH) und irgendein Wert ungleich 0, falls die Bedingung erfüllt ist (Pinx == LOW).

In C sollte man bool'sche Werte immer mit if (wert) bzw. if(!wert) abtesten.,
was fleichbedeutend ist mit if (0 != wert) bzw if (0 == wert)
Dazu steht auch was im Wiki, bei "Fallstricke der C-Programmierung" ;-)