Atmega8 Frequenzein-ausgang

[denver2013 (Gast)]

Hallo,
ich brauche ein wenig Hilfe.
Mein Programm soll folgendes machen:
Die Eingangsfrequenz (INT1) bewegt sich im Bereich von ca. 30-300 Hz.
Die Ausgangsfrequenz (wackeln an beliebigem Pin) bis ca. 20-30% +/- .
Das Problem ist, es kommen immer wieder Aussetzer in unregelmäßigen 
Abständen (mit Osci kontr.).
Vermutlich Timereinstellungen.
Wer hat einen Tip?

// Atmega8
// Timer1 Timer2 16Mhz Clock


#include <avr/io.h>
#include <avr/interrupt.h>
#include <math.h>

volatile int Takt;
int T2;
volatile double Eingang;
volatile double x = (-0.9916);
int main(void)
{
   TCCR2 |= (1 << WGM21); //CTC Timer2

   GICR  |= (1<<INT1);
   MCUCR |= (1<<ISC10) | (0<<ISC11); //Any logical change on INT1 generates
                                      // an interrupt request.

   TIMSK |= (1 << OCIE2) | (1<<OCIE1A); //Interrupt compare match Timer1+2
 


   TCCR2 |= (1 << CS22) | (1 << CS21) | (1 << CS20); //Priscaler 1024 Timer2
 

   TCCR1B |= (1 << WGM12);                //CTC Timer1
   TCCR1B |= (1 << CS11) | (1 << CS10);   // Prescaler 64
   sei(); // enable interrupts
    

DDRB |= (1 << PB1);

    while (1);
    {
    ; // Die Berechnung hier bringt nur Hausnummern ?????
    }
}

ISR(INT1_vect)
   { 
    Takt++;
  
   }

ISR (TIMER2_COMP_vect)
{
    T2++;
     if (T2 == 30) //~61 = 1 Sekunde
    {
      Eingang = Takt;
    Takt = 0;
    T2 = 0;    
    OCR1A = (pow( Eingang, x )*120221); //y = 120221x-0,9916
    }
  
}


ISR(TIMER1_COMPA_vect)
{
  PORTB ^= (1 << PB1);
}

[Uwe (de0508)]

Den gewohnten Doubledatentype hat du nicht als Zahlenbereich beim avr 
gcc.

Es gilt in ISR macht man nur das nötigste, das was schnell abgearbeitet 
wird.

Aber in deinem code steht solch ein Monster:

OCR1A = (pow( Eingang, x )*120221);

Gibt bitte mal eine Schätzung über das Zeitverhalten ab und dann 
schreibe ein neues Programm - Test - um die Laufzeit mit einem Timer zu 
messen.
Du wirst Dich wundern !

Also wenn solch eine Berechnung, dann im Main.

Ich will jetzt gar nicht auf die Umgehung des Datentype double ein 
gehen, das geht !

Das werden andere noch mache - Stichwort Fixpoint Interger Mathematik.

[denver2013 (Gast)]

@Uwe S.

Den Komentar in meinem Programm hast Du aber schon gelesen?
 // Die Berechnung hier bringt nur Hausnummern ?????

Vielleicht soll ich noch anfügen ich mach das Ganze nur aus Spaß an der 
Freud.
Leider hilft mir Dein Beitrag nicht weiter.
Trotzdem vielen Dank.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

denver2013 schrieb:
> T2++;
>      if (T2 == 30) //~61 = 1 Sekunde
>     {
>       Eingang = Takt;
>     Takt = 0;
>     T2 = 0;

Da fallen mir (abgesehen vom double Datentyp) mehrere Sachen auf:
* T2 ist nicht als volatile deklariert und dann auch noch als int. Hier 
reicht ein uint8_t (es wird ja höchstens bis 30 gezählt), damit 
entfallen alle Überlegungen hinsichtlich atomarer Zugriffe.
> ISR(INT1_vect)
>    {
>     Takt++;
>
>    }
* Takt ist immer eine Ganzzahl, deswegen ist es auch hier überflüssig, 
double zu benutzen. Abgesehen davon ist ein double ein so langer 
Datentyp, das hier unbedingt auf 'atomare' Zugriffe zu achten wäre.

Grundsätzlich gilt, das kleine MCs immer sehr viel Zeit brauchen, um 
Gleitkommazahlen zu behandeln. Mach am besten die gesamte Berechnung mit 
ganzen Zahlen und rechne sie nur am Schluss in die gewünschten Anzeige 
um.

[amateur (Gast)]

@denver2013

Bist Du sicher, dass Du an der richtigen Stelle nach Fehlern und 
Problemen suchst?

Wenn Du festgestellt hast:
>    ; // Die Berechnung hier bringt nur Hausnummern ?????
solltest Du nicht den Rechenort verlegen, sondern nach dem Warum suchen.

[denver2013 (Gast)]

@amateur

Ich hab schon alle "Rechenorte" ausprobiert leider ohne Erfolg :-(

[Wolfgang (Gast)]

denver2013 schrieb:
> Ich hab schon alle "Rechenorte" ausprobiert leider ohne Erfolg :-(

Eben, darin liegt es dann wohl nicht.

Uwe S. schrieb:
> Aber in deinem code steht solch ein Monster:
> OCR1A = (pow( Eingang, x )*120221);
> Gibt bitte mal eine Schätzung über das Zeitverhalten ab und dann
> schreibe ein neues Programm - Test - um die Laufzeit mit einem Timer zu
> messen.

denver2013 schrieb:
> Leider hilft mir Dein Beitrag nicht weiter.

Auf Deutsch: Guck dir mal die Rechenzeit für die Operation zur Belegung 
des OCR1A an und setze das in Relation zum Abstand der Interupts

[denver2013 (Gast)]

Vielen Dank an Alle.
Es funktioniert.