Finde meinen Fehler nicht :(

[Hans P. (bmwi)]

Hallo Leute.
Hab einen Frequenzzähler programmiert aber folgendes Problem....

Beim Einschalten (schalter ist oben) zeigt es die richtige 
Herzzahl(MODUS0) an (vorher rechnerisch ermitelt).. bei betätiges des 
schalters wird eine gerindere Herzzahl erwartet (rechnerrisch ermittelt) 
es zeigt diese Zahl an, ABER es wächselt sich ständig mit einer andere 
zahl (die gleich bleibt) ab (eigentlich MODUS1, da freq zwischen 10 und 
100)... das gleiche, wenn eine freq kleiner 10 kommt.... beim wieder 
hochschalten der schalters kommt wieder die 250 Hz (richtige Herzzahl in 
MODUS0)..

Ich hoffe ich habe mich deutlich ausgedrückt...


mfg

[Wolfgang Horn (Gast)]

Hi Hans,
Du Leigeplagter vom Dienst vom heute (jeder von uns ist mal dran, sich 
die Zähne auszubeißen...)

Aus Deinem Programm:

void main(void)
{
     i=0;
     ...
    while(1);
}

Dies "while(1);" ist eine Endlosschleife, vermutlich, bis ein Watchdog 
anschlägt und das Programm erneut startet.

Auch dies kommt mir verdächtig vor:
data unsigned int ausgabeFreq=0;

void rechnung()
{
...  {
    ausgabeFreq = 1000000/(256*TH0+TL0+i*65536);
  }
  else
  {
    ausgabeFreq = 1000000/(256*TH0+TL0+i*65536);
  }
}

Bei meinen AVR's ist int 16 Bit breit, schmäler als
"1000000".

Du findest hier im Forum Beiträge von Kollegen, die sich mit long long 
(also 64 Bit) herum geschlagen haben.
Wenn ich 1 GHz auf 1 Hz auflösen will, (ganz egal, dass mein Messsender 
das gar nicht kann), käme ich auf 30 Bit.

Ciao
Wolfgang Horn

[Peter D. (peda)]

Ein Fehler ist schonmal, daß Du im Forum Codesammlung postest.

Lesen müßte man können:

"Forum: Codesammlung
Wenn ihr eigene Programme oder Anleitungen geschrieben habt könnt ihr 
sie hier posten. Fragen werden gelöscht!"


Peter

[Karl H. (kbuchegg)]

verschoben

[Hans P. (bmwi)]

Hallo,

ja das ist mir bereits selber aufgefallen. Wenn ich jedoch es zu einem 
double mache steht die Fehlermeldung: to many types. Ich habe den code 
etwas abgeändert:

#include <at89c5131.h>
#include <stdio.h>
extern void initlcd (void);    //  Initialisierung LCD-Display an P0      
extern void loeschenlcd (void);
extern void textlcd (unsigned char *text,unsigned char zeile);  
#define MODUS0 0
#define MODUS1 1
#define MODUS2 2

void initTimer0undExtInt0();

void isrTimer0(void);
void isr_extInt0(void);      
void ausgabeLCD(void);  
void zwischenRoutine(void);

void stoppen(void);
void rechnung(void);
void modusWahl(void);
void einstellungLaden(void);
void start(void);


data int i;
data unsigned int freq=0;
data unsigned int ausgabeFreq=0;

unsigned char modus;

//Variablendeclaration global:
code unsigned char ueberschrift [] = "Frequenzmesser";
code unsigned char zeile2 [] = "Freq:           Hz ";
                      //mit dem Vorsatz code wird das array nicht im internen RAM (ziemlich klein)+
data unsigned char buf[20];

  
void main(void)
{
      i=0;
      modus = MODUS0; //geändert
    initlcd ();            //  Initialisierung LCD-Display an P0                
    loeschenlcd ();          //  Anzeige löschen  
    textlcd(ueberschrift,1);
    textlcd(zeile2,2);    
    initTimer0undExtInt0();
    while(1);                  


}

void zwischenRoutine(void)
{
    stoppen();
        modusWahl(); //eig nach rechnug()
    rechnung();
    ausgabeLCD();
    einstellungLaden();
    i=0;
       freq=0;
    start();
}

 void stoppen()
 {
   EA=0;
   TR0 = 0;
 }

void modusWahl()
{

    if(modus==MODUS0)
    {
        if(freq>=100)
        {
        modus=MODUS0;
        }
  
        else if(freq<100 && freq>=10)
        {
            modus=MODUS1;
        }
  
        else 
        {
        modus=MODUS2;
        }
    }
        else
            {
                modus = MODUS0;
            }
}

void rechnung()
{
  if(modus== MODUS0)
  {
    ausgabeFreq = freq;
  }
  else if(modus == MODUS1)
  {
    ausgabeFreq = 10/((256*TH0+TL0+i*65536)*1000);
  }
  else
  {
    ausgabeFreq = 1/((256*TH0+TL0+i*65536)*1000);
  }
}


void ausgabeLCD()
{
    sprintf(buf,"Freq:%5d Hz",ausgabeFreq);
    textlcd (buf,2);
}




void einstellungLaden()
{
    if (modus==MODUS0)
    {
     TH0 = 0x3C;                // Timer0 vorladen
        TL0 = 0xB0;                // 65536 - 50000 = 15536 = D8F0h (50ms)
    }
    else if(modus==MODUS1)
    {
        TH0 = 0x00;                // Timer0 fungiert als Zeitmesser
        TL0 = 0x00;                
    }
    else 
    {
        TH0 = 0x00;                // Timer0 fungiert als Zeitmesser
        TL0 = 0x00;                
    }
}

void start()
{
     TR0 = 1;
     EA=1;
     
}


void initTimer0undExtInt0()
{
   TMOD = 0b00000001;         // Timer0 konfigurieren, Modus1: 16 bit Zaehler
   TH0 = 0x3C;                // Timer0 vorladen
   TL0 = 0xB0;                // 65536 - 50000 = 15536 = D8F0h (50ms)
   TR0 = 1;                    // Start Timer0
   ET0 = 1;                    // Freigabe Timer0. Bit im Reg IE, Reg.-Adresse A8h
   IT0 = 1;                    // Start extit0
   EX0 = 1;          
   EA  = 1;                    // Globale INT Freigabe. Bit im Reg IE
}


void isrTimer0(void) interrupt 1
{   

  i++;
    if (modus==MODUS0)
    {
       TH0 = 0x3C;                // Timer0 vorladen
         TL0 = 0xB0;                // 65536 - 50000 = 15536 = D8F0h (50ms)
       
       if(i==20)
       {     
          zwischenRoutine();
        }
      }   
}
   
  
void isr_extInt0(void) interrupt 0
{
       
      freq++;
  
     if(modus==MODUS1)
     {
                // zwischenRoutine();
       if(freq==10)
       {
        zwischenRoutine();
       }
     }
     else if(modus==MODUS2)
     {
       if(freq==1)
       {
        zwischenRoutine();
       }
     }        
}

Ich war ausch so schlau und habe meine Taktgeberdatei geändert und eine 
größere Schleife eingebaut, sodass ich eine beringere frequenz erhalte. 
Was war? ... Ich habe trz. nur die 250 Hz angezeigt bekommen. Nun weiß 
ich nicht wieso das so ist.

Nun bin ich hingegangen und habe einen neues Code geschrieben, der etwas 
einfacher gestaltet ist:

#include <at89c5131.h>
#include <stdio.h>
extern void initlcd (void);    //  Initialisierung LCD-Display an P0      
extern void loeschenlcd (void);
extern void textlcd (unsigned char *text,unsigned char zeile);  

void initTimer0undExtInt0();

void isrTimer0(void);
void isr_extInt0(void);    
void messung();  
//double frequenzErmitteln();    
void ausgabeLCD();  
//void takt();

sbit exmesser = P3^2;

data int i;
data unsigned int freq=0;
data unsigned int mfreq=0;
//data unsigned int zeit=0;

//Variablendeclaration global:
code unsigned char ueberschrift [] = "Frequenzmesser";
code unsigned char zeile2 [] = "Freq:          Hz    ";
                      //mit dem Vorsatz code wird das array nicht im internen RAM (ziemlich klein)+
data unsigned char buf[20];


  
void main(void)
{
     i=0;
    initlcd ();            //  Initialisierung LCD-Display an P0                
    loeschenlcd ();          //  Anzeige löschen  
    textlcd(ueberschrift,1);
    textlcd(zeile2,2);    
//    ausgabeLCD();
    initTimer0undExtInt0();
//     init_ExtInt0();
    while(1);                  


}

//double frequenzErmitteln()
//{
      
//      mfreq=freq/zeit;
 //        return mfreq;
//}

void initTimer0undExtInt0()
{
   TMOD = 0b00000001;         // Timer0 konfigurieren, Modus1: 16 bit Zaehler
   TH0 = 0x3C;                // Timer0 vorladen
   TL0 = 0xB0;                // 65536 - 50000 = 15536 = D8F0h (50ms)
   TR0 = 1;                    // Start Timer0
   ET0 = 1;                    // Freigabe Timer0. Bit im Reg IE, Reg.-Adresse A8h
   IT0 = 1;                    // Start extit0
   EX0 = 1;          
   EA  = 1;                    // Globale INT Freigabe. Bit im Reg IE
}


void isrTimer0(void) interrupt 1
{
   TH0 = 0x3C;                // Timer0 vorladen
   TL0 = 0xB0;                // 65536 - 50000 = 15536 = D8F0h (50ms)
   i++;
   if(i==20)
   {
     i=0;   
      TR0 = 0;                  // Stopp Timer0
       EA = 0;                  // alle Interrupts ausschalten  Freigabe Timer0. Bit im Reg IE, Reg.-Adresse A8h >>> DEAKTIVIEREN
      //--------------------------------------------------------------------------
      //frequenzErmitteln();      // frequenz ermitteln
      ausgabeLCD();          // DisplayAusgabe 
      freq=0;              // Variablen zurück setzen
      //-------------------------------------------------------------------------
     TH0 = 0x3C;              // Timer0 vorladen
     TL0 = 0xB0;              // 65536 - 50000 = 15536 = D8F0h (50ms)
     TR0 = 1;                  // Start Timer0
     EA = 1;                  // Freigabe Timer0. Bit im Reg IE, Reg.-Adresse A8h

   }
}


void isr_extInt0(void) interrupt 0
{
    freq++;
}

void ausgabeLCD()
{
    sprintf(buf,"Freq:%5d Hz",freq);
    textlcd (buf,2);
}

... Nun erhalte ich die verschiedene Frequenzen: .. Bei und wenn ich die 
die schleiße doppel so groß mache, so erhalte ich auch eine halb so 
große frequenz.... ich werde es wahrscheinlich so lassen. aber wenn jmd. 
den fehler von mir erkennt, so bitte melde sich der dieser.