frequenz auslesen und auf lcd anzeigen

[Tobias (Gast)]

Hallo,
ich möchte bei meinem selbstgebauten funktionsgenerator die freuenz 
auslesen und auf ein display ausgeben. die frequenz lässt sich von 
5Hz-15kHz einstellen. ich möchte mit dem timer0 eine torzeit von 200ms 
abfragen welche wie viele positive flanken timer1 gezählt hat.

leider funktioniert es noch nicht so. feste variablen lassen sich im 
display anzeigen. jedoch funktionieren die interrupts noch nicht so. 
sobald ich die interrupts mit "sei();" antivieren möchte, schreibt er 
nichts mehr ins display.

das ist mein code bis jetzt:

cdatei:

#define F_CPU 4000000UL

#include <avr/io.h> 
#include <stdlib.h> //Standardlib
#include <stdio.h>  //für sprintf
#include <avr/interrupt.h> //für interrupt
#include <avr/delay.h>  //Zeitschleife
#include "lcd.h"

int main (void) {            
 
  DDRA  = 0xff; //Ausgänge             
  DDRB  = 0x00; //Eingänge
  DDRC  = 0xff;
  DDRD  = 0xff;

//init LCD
  
  RS_0;
  PORTA=0b00111000; //8bit, 2zeilig, 5x7
  E();
  wait();
  PORTA=0b00001100; //Display ein
     E();
  wait();

//Timer0 config
  TCCR0=0b00000100; //Prescaler256
  TIMSK |=(1<<TOIE0); //Overflow Interrupt erlauben

//Timer1 config
  TCCR1B=0b00000111; //increment bei rising edge
  sei(); //global Interrupts aktivieren

//Ausgabe

  while(1) //Endlosschleife
  {
  _delay_ms(500);
  hz=10000;
  sprintf(buffertakt, "%02dHz", hz);
  ausgabe();
  }
   
   return 0;
}

//Zählerinterrupt Timer0
ISR (TIMER0_OVF_vect)
{
zaehler++;
  if (zaehler == 12)//ale 200ms min 5Hz
  {
  hz=TCNT1L;
  zaehler=0;
  }
//}

hdatei:

char buffertakt [50];
int zaehler;
int zahl;
int hz;
int i;

#define RS_1    PORTC |= (1<<PC7)
#define RS_0    PORTC &= ~(1<<PC7)

int wait ()
{
_delay_ms(2);
return 0;
}

int E ()
{
PORTC |= (1<<PC6);
PORTC &= ~(1<<PC6);
return 0;
}

int ausgabe ()
{
RS_0;
PORTA=0b10000000; //Cursorpoisition
E();
wait();
zahl=0;
  for(i=0; i<=6; i++)
  {
  RS_1;
  PORTA=buffertakt [zahl++];
  E();
  wait();
  }
return 0;
}

[Karl H. (kbuchegg)]

Wieso ist hier

ISR (TIMER0_OVF_vect)
{
zaehler++;
  if (zaehler == 12)//ale 200ms min 5Hz
  {
  hz=TCNT1L;
  zaehler=0;
  }
//}

die letzte } auskommentiert?

Du musst unbedingt dein Einrückschema verbessern. Im Moment sieht man 
bei dir nur sehr schwer, wo eine { anfängt bzw. wo die korrespondierende 
} ist.

Meiner Zählung nach müsste die auskommentierte } eigentlich nicht 
auskommentiert sein.

[Karl H. (kbuchegg)]

Und tu dir selbst einen Gefallen und schaff dir erst mal eine 
unabhängige Software-Ebene, in der du die LCD Funktionen ansiedelst. Das 
hat keinen wirklichen Sinn, wenn du die Portzugriffe da mitten in deinen 
Applikationscode einbaust.
Da sieht man ja den Wald vor lauter Bäumen nicht mehr.

Ein Minialsatz an Funktionen wäre zb

lcd_init()
lcd_gotoxy( uint8_t line, uint8_t column )
lcd_char( char c )
lcd_string( const char* str )

[Karl H. (kbuchegg)]

> jedoch funktionieren die interrupts noch nicht so.
> sobald ich die interrupts mit "sei();" antivieren möchte, schreibt
> er nichts mehr ins display.

Sowas ist normalerweise ein starker Hinweis darauf, dass du irgendwo 
einen Interrupt freigegeben hast, für den du keine ISR hast.

Hast du hier

ISR (TIMER0_OVF_vect)
{
  ...

vom Compiler irgendwelche Warnungen bekommen?

[Tobias (Gast)]

ohh das auskommentiere habe ich versehendlich mitkopiert. das habe ich 
schon geändert. ja das stimmt der code ist noch nicht sehr schön. aber 
ich möchte ja erstmal das es funktioniert. das display zeit ja das 
richtige an, wenn ich was von hand in die variablen schreibe. die 
interrupts funktionieren nur nicht. es kommen auch 3 warnungen:

../funktionsgenerator.c:48: warning: return type defaults to `int'
../funktionsgenerator.c: In function `ISR':
../funktionsgenerator.c:48: warning: type of "TIMER0_OVF_vect" defaults 
to "int"
../funktionsgenerator.c:55: warning: control reaches end of non-void 
function

was bedeuten die?

[Karl H. (kbuchegg)]

Tobias schrieb:
> ohh das auskommentiere habe ich versehendlich mitkopiert. das habe ich
> schon geändert. ja das stimmt der code ist noch nicht sehr schön. aber
> ich möchte ja erstmal das es funktioniert.

Das ist ein Irrtum.
Du rennst Fehlern nach, die nicht sein müssen. UNd alles nur deswegen, 
weil du sie selber nicht siehst.

Ausserdem erlaubt dir ein vernünftiger Satz an LCD Funktionen problemlos 
und ohne großen Aufwand, dir Testausgaben ins Programm einzubauen, so 
dass du selber leicht nachvollziehen kannst, was da passiert.

> ../funktionsgenerator.c:48: warning: return type defaults to `int'
> ../funktionsgenerator.c: In function `ISR':
> ../funktionsgenerator.c:48: warning: type of "TIMER0_OVF_vect" defaults
> to "int"
> ../funktionsgenerator.c:55: warning: control reaches end of non-void
> function
>
> was bedeuten die?

Das die Interrupt Funktion auf deinem µC anders heißt.

[Karl H. (kbuchegg)]

Karl Heinz Buchegger schrieb:

>> ../funktionsgenerator.c:48: warning: return type defaults to `int'
>> ../funktionsgenerator.c: In function `ISR':
>> ../funktionsgenerator.c:48: warning: type of "TIMER0_OVF_vect" defaults
>> to "int"
>> ../funktionsgenerator.c:55: warning: control reaches end of non-void
>> function
>>
>> was bedeuten die?
>
> Das die Interrupt Funktion auf deinem µC anders heißt.

Nein. Warte mal. Dann wäre der Text der Warnung ein anderer.

Bist du sicher, dass du

#include <avr/interrupt.h> //für interrupt

in dem Code drinnen hast, den du compilierst.

Mit sowas:
> ohh das auskommentiere habe ich versehendlich mitkopiert.
haben wir hier gar keine Freude.
Hänge das C-File an, so wie du es compilierst und kopiere nichts extra 
fürs Forum. Denn dann suchen wir Fehler, die in deinem richtigen Code 
nicht vorhanden sind.

[Tobias (Gast)]

ja das habe ich nicht auskommentiert...

[Karl H. (kbuchegg)]

Und wenn du jetzt noch verraten würdest um welchen µC es sich handelt, 
dann könnte man den hier sogar im AVR-Studio mal einstellen und 
nachsehen
* welche Warnungen sonst noch ignoriert wurden
* wie der ISR Handler wirklich heißen muss

Dass man den Leuten immer alles aus der Nase ziehen muss :-)

[Tobias (Gast)]

tut mir leid :) ich verwende einen atmega 16 ;)
ich habe mal meinen frequenzgenerator als bild angehängt...

[Tobias (Gast)]

sobald ich die interrupts aktiviere geht es nicht mehr :/
ich hab jetzt auch mehrfach im avr tutorial nachgeschaut aber ich finde 
den fehler nicht...

[J.-u. G. (juwe)]

Mal davon abgesehen, dass Deine Funktionsdefinitionen nicht nach lcd.h 
gehören, müssen Deine Variablen, die Du sowohl in der ISR, als auch in 
anderen Funktionen verwendest, als "volatle" deklariert werden.

http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Datenaustausch_mit_Interrupt-Routinen

[Tobias (Gast)]

daran lags leider nicht...der interrupt geht immer noch nicht...

[Tobias (Gast)]

beim interrupt habe ich folgende fehlermeldungen:

../funktionsgenerator.c:53: warning: return type defaults to `int'
../funktionsgenerator.c: In function `ISR':
../funktionsgenerator.c:53: warning: type of "TIMER2_OVF_vect" defaults 
to "int"
../funktionsgenerator.c:59: warning: control reaches end of non-void 
function

kann mir jemand sagen, was ich da falsch gemacht habe?

[J.-u. G. (juwe)]

Tobias schrieb:
> kann mir jemand sagen, was ich da falsch gemacht habe?

> ISR (TIMER0_OVF_vect)

Zwischen "ISR" und "(" gehört kein Leerzeichen.

[Marco (Gast)]

ne das war es leider auch nicht. das komische ist eben, dass so alles 
prima funktioniert aber sobald ich mit sei(); die interrupts aktiviere 
geht gar nichts mehr :/

[J.-u. G. (juwe)]

Marco schrieb:
> ne das war es leider auch nicht.

Sind die Warnmeldungen, die Du oben gezeigt hast, jetzt weg oder nicht? 
Gibt es noch weitere Compilermelduneg?

[Tobias (Gast)]

ne die sind leider noch nicht weg...
das sind alle:

Build started 9.4.2013 at 21:01:17
avr-gcc  -mmcu=atmega16 -Wall -gdwarf-2 -Os -std=gnu99 -funsigned-char 
-funsigned-bitfields -fpack-struct -MD -MP -MT funktionsgenerator.o -MF 
dep/funktionsgenerator.o.d  -c  ../funktionsgenerator.c
../funktionsgenerator.c:52: warning: return type defaults to `int'
../funktionsgenerator.c: In function `ISR':
../funktionsgenerator.c:52: warning: type of "TIMER2_OVF_vect" defaults 
to "int"
../funktionsgenerator.c:60: warning: control reaches end of non-void 
function
avr-gcc -mmcu=atmega16 -Wl,-Map=funktionsgenerator.map 
funktionsgenerator.o     -o funktionsgenerator.elf
avr-objcopy -O ihex -R .eeprom -R .fuse -R .lock -R .signature 
funktionsgenerator.elf funktionsgenerator.hex
avr-objcopy -j .eeprom --set-section-flags=.eeprom="alloc,load" 
--change-section-lma .eeprom=0 --no-change-warnings -O ihex 
funktionsgenerator.elf funktionsgenerator.eep || exit 0
/usr/bin/sh: /c/WinAVR/bin/avr-objcopy: Invalid argument
avr-objdump -h -S funktionsgenerator.elf > funktionsgenerator.lss
/usr/bin/sh: /c/WinAVR/bin/avr-objdump: Invalid argument
make: *** [funktionsgenerator.lss] Error 126
Build failed with 1 errors and 3 warnings...

[J.-u. G. (juwe)]

Tobias schrieb:
> ../funktionsgenerator.c:52: warning: type of "TIMER2_OVF_vect" defaults

"TIMER2_OVF_vect"? Wo kommt denn das nun wieder her. Entschuldige bitte, 
aber langsam wird es mir zu blöd. Zeige hier bitte exakt den Code, der 
bei Dir Probleme macht.

[Tobias (Gast)]

ich hatte mir ein beispiel mit dem timer2 kopiert und wollte es damit 
probieren. damit geht es aber auch leider nicht...
ich habe die aktuellen files mit der eben genannten fehlermeldung 
beigefügt...

[J.-u. G. (juwe)]

Im Funktionskopf einer C-Funktion darf zwischen Funktionsname und 
Parameterliste kein Leerzeichen stehen. Das hast Du bei allen Deinen 
Funktionen ,inklusive "main", nicht beachtet.

Außerdem hast Du Deine main-Funktion mit Rückgabewert "int" deklariert, 
sie enthält aber kein "return" Statement.

[Tobias (Gast)]

okay ich habe es korrigiert. aber das war der fehler leider auch nicht. 
die warnungen werden immernoch angezeigt...

[J.-u. G. (juwe)]

Tobias schrieb:
> aber das war der fehler leider auch nicht.
> die warnungen werden immernoch angezeigt...

Ich habe jetzt mal Deine beiden Datein in ein AVR4-Studio-Projekt 
eingefügt. Es kompilierte mit nur einer Warnmeldung, die besagt, dass 
die Headerdatei delay.h nun unter utils/delay.h zu finden ist. Nach 
Änderung dieser Zeile kompilierte das Projekt ohen Fehler- und 
Warnmeldungen.

[Tobias (Gast)]

ich glaube es liegt an meiner compilerversion...das war anfangs schon so 
komisch weil der den nicht von alleine gefunden hat. das delay musste 
ich extra ändern, da ich noch die alte version habe. ich lade gerade den 
neuen herunter...

[Tobias (Gast)]

das ist der hammer ich hing ein ganzen tag rum jetzt wegen dem mist und 
nur weil meine comlierversion veraltet war :/ jetzt geht alles...aber 
danke für deine bemühungen :)