AVR Frequenzmessung

[Jakov K. (jackenkoffer)]

Hallo !

Ich habe folgende Methode für den AVR Atmega8 geschreiben um am Pin PB0 
eine anliegende Frequenz zu messen. Jedoch habe ich einen fiesen Bug 
drin, den ich nicht finden kann.
Wie man sieht habe ich nur ein teil meines Quelltextes gepostet, da der 
Rest unwichtig ist.

#define TICKS 16000000 // 16 MHZ Quarz

volatile u16 ic_zp_A,ic_zp_B; // Variablen, um das Register ICR1 auszulesen
volatile u8 i; // job-Variable
volatile u8 overflow; // zählt die Überlaufe

inline void ICP_init()
{
  TCCR1B |= (0<<ICNC1)|(0<<ICES1)|(0<<CS12)|(0<<CS11)|(1<<CS10); 
  // Prescaler = 1, 
  //Input capture enable, noise canceler disable, reagiere auf fallende flanke
  TIMSK |= (1<<TICIE1); // Input capture Interrupt enable
  DDRB &= ~(1<<PB0); // PB0 ( ICP1) als eingang
}

ISR(TIMER1_CAPT_vect)
{
  if (i == 0)    // 1.High Flanke
  {
    ic_zp_A = ICR1;
    overflow = 0;
    i = 1;
  }
  else           // 2.High Flanke
  {
    ic_zp_B = ICR1;  // Messung fertig
    i = 2;
  }
}

ISR(TIMER1_OVF_vect)
{
  overflow++;
}

float frequenzmessen(void)
{
  float fr;
  float icr1;
  u16 zw_Erg;
  
  lcd_string("Wird Gemessen"); // "Debug-funktion"
  i = 0;
  sei();
  TCNT1 = 0;
  while(i<2){}
  
  zw_Erg = ((overflow*65536) + ic_zp_B - ic_zp_A);  // Overflow berücksichtigen
  fr = TICKS / zw_Erg;      //16MHz Quarz
  cli();
  set_cursor(0,2);
  lcd_string("Wurde Gemessen");// "Debug funktion"
  _delay_ms(500);
  lcd_clear();
  return(fr);
}

Ich habe als "debug"-Funktion für die Frequenzmessung das LCD Display 
benutzt, um erkennen zu können, wo der Fehler ist.
Der Text "Wurde Gemessen" erscheint nicht auf meinem Display. daher 
denke ich , dass es die While-Schleife ist, die zu einer endlosschleife 
wird.
Fazit: ich suche eine möglichkeit den whileschleifen-Fehler zu 
eliminieren.
Bitte um eure Hilfe &&
Danke im Voraus !
Jakov

[Fuenf Tassen (Gast)]

Schmeiss mal alles raus, was nicht absolut wichtig ist und lass dir den 
capture wert geben. zB auf das UART.

[Jakov K. (jackenkoffer)]

Das funktioniert, das habe ich schon gemacht ;)
Einfach ein Programm, das bei tastendruck den Capture-wert an das LCD 
gibt.
Ich habe jedoch das Problem, dass ich die Zeit zwischen zwei Flanken 
brauche.
Gruß, Jakov

[Karl H. (kbuchegg)]

Jakov K. schrieb:

> Wie man sieht habe ich nur ein teil meines Quelltextes gepostet, da der
> Rest unwichtig ist.

Das sagen sie immer. Und dann stellt sich ein ums andere mal heraus, 
dass es nicht stimmt.

Hast du denn wenigstens den abgespeckten Teil getestet ob er den Fehler 
zeigt?

[Jakov K. (jackenkoffer)]

Hier ist der ganze quelltext.
der quelltext lässt sich feherfrei compilieren , und die hex datei lässt 
sich auch auf den controller laden, doch Der controller tut nicht das 
was man ihm sagt.

[Fuenf Tassen (Gast)]

Wenn man die zeit zwischen den flanken braucht, muss man im ersten 
interrupt die flanke umschalten.

[STK500-Besitzer (Gast)]

Jakov K. schrieb:

   u16 zw_Erg;
   zw_Erg = ((overflow*65536) + ic_zp_B - ic_zp_A);  // Overflow berücksichtigen

Wie willst du den 65536 in eine 16-Bit-Veriable bekommen?

[Karl H. (kbuchegg)]

Lass dir halt mal in der Schleife dein i (übrigens: ganz schlechter 
Name. i heißt jede 3. Hilfsvariable. Diese Variable ist WICHTIG. Die hat 
sich auch einen sprechenden Namen verdient) ausgeben. Bzw. in der ISR 2 
LED schalten, damit du siehst ob die Steuerung dort funktioniert.

[Elmar (Gast)]

Hi

Du ahst nach der While Schleife beide geschw. Klammern, also eine 
Endlosschleife!!

[Elmar (Gast)]

Ok i wird in der ISR Routine erhöht bzw. gesetzt.....

[900ss (900ss)]

Jakov K. schrieb:
> doch Der controller tut nicht das was man ihm sagt.

Doch, er tut genau(!) das, was du ihm sagst. ;-)

SCNR :-)

[Jakov K. (jackenkoffer)]

sag bloß..