Timer Interrupt wird nicht ausgelöst?

[dude (Gast)]

Hallo,

ich bin dabei mich mit dem timer des atmega 328p zu befassen. Ich will 
spätzer daraus eine Uhr bauen zu lernzwecken. Bin jetzt an einem Punkt, 
wo ich einfach mal Hilfe brauche.

Ich habe an meinem µController vier 7-Segmentanzeigen. Die ich wunderbar 
ansteuern kann. Jetzt möchste ich, dass die Ziffern der 7 Segmentanzeige 
wenn der Timer zum 50zigsten Mal seinen maximalwert erreicht hat 
umgeschaltet wird. Die Ziffern habe ich in einem Array. Jedoch bleiben 
meine 7-Segmentanzeigen dauerhaft auf 0 0 0 0, wenn ich das ganze mit 
dem Timer versuche. In der Hauptschleife wird die Bedingung 
if(extraTime>50) nie erfüllt bzw. der Interrupt wird gar nicht 
aufgerufen?

Jemand eine Idee?

/*
 * Uhr.c
 *
 * 
 *  Author: xxxx
 */ 
#ifndef F_CPU
#define F_CPU 8000000UL


  //Externer Quarztakt
#endif
#include <avr/io.h>
#include <util/delay.h>    //Warteschleifen
#include <avr/interrupt.h>  //Interruptheader
int extraTime=0;

//Konsanten
const char sieben_seg_ziffer[10] = {0b01000100,0b11001111,0b01010010,0b11000010,0b11001001,0b11100000,0b01100000,0b11001110,0b01000000,0b11000000};//Ziffern 0 bis 9 der 7-Seganzeige
  


ISR(TIMER0_COMPA_vect)
{
  extraTime++;
}

  

int main(void)
{

  
  TCCR0A |= (1<<WGM01);//CTC Mode aktivieren
  OCR0A = 156;//Max Wert bis Interrupt auslöst
  TIMSK0 |=  (1<<OCIE0A);
  sei();
  
  TCCR0A |= (1<<CS00)|(1<<CS02);//Setting Prescaler clk/1024
  
  
  
  DDRD   = 0b11111111;                      //Ausgang; Siebensegmentanzeige
  DDRC   = 0b11111111;            //Ausgang; Die vier Anoden der 7-Segmentanzeigen
  
  
  //vier 7-Segmentanzeigen aktivieren
  PORTC|=(1<<PC0);
  PORTC|=(1<<PC1);
  PORTC|=(1<<PC2);
  PORTC|=(1<<PC3);
  

  
  while(1)
    {
    
    int i=0;
  
    do 
    {
      PORTD=sieben_seg_ziffer[i];
    
      if(extraTime>50)
      {
        i++;
        extraTime=0;
      }
      } while (i<10);
    

  }
}

[Karl H. (kbuchegg)]

dude schrieb:


> int extraTime=0;

volatile int extraTime = 0;


FAQ: Was hat es mit volatile auf sich

[dude (Gast)]

Danke für den Hinweis. Mit volatie int extraTimer habe ich jetzt 
ausgeschlossen, dass irgendwas wegoptimiert wurde. Trotzdem wird wie es 
aussieht die Bedinung if(extraTime>50) nicht erfüllt, da die Anzeige bei 
0 0 0 0 bleibt. hhhmm irgendwo ist noch ein bock...

[Dude (Gast)]

Ich finde den Fehler nicht...... -.-

Ich habe das Gefühl, dass der timer interrupt gar nicht auflöst, da die 
Bedingung in der if Abfrage nicht erfüllt wird.

[OldMan (Gast)]

Als erstes würde ich Dir vorschlagen, dass Du mal eine "Debug"-Funktion 
mittels einer LED in die ISR einbaust.
Suche Dir einen freien PORT mit passendem PIN
und daran schliesst Du eine LED an. Vergiss nicht den Port als Ausgang 
zu definieren.

Die ISR erweiterst Du wie folgt

ISR(TIMER0_COMPA_vect)
{
  PORTx ^= 0x80;      // Bspw. an PORTx (der freie Port) mit dem Bit7 wackeln
  extraTime++;
}

Dann siehst Du, ob Deine ISR überhaupt funktioniert.
D.h., die LED sollte flackern. Notfalls mal eine Scope anschliessen,
dann siehst Du auch die Taktrate Deines ISR. D.h., Du kannst am Scope 
sehen, wieviel Zeit vergeht bis der ISR wieder aufgerufen wird.

Als weiteres würde ich die Variable extraTime als char bzw. uint8_t 
definieren. Denn int ist 16-Bit. Damit gibt es immer Probleme, denn 
ein AVR ist ein 8-bittiger Prozessor. Auch wenn sie als volatile 
definiert ist.
Gewöhne Dir bitte an die Typdeklarationen zu benutzen die hier üblich 
sind.
Also: nicht char sonder als int8_t bzw. unsigned char als uint8_t zu 
deklarieren, usw.. Damit wird es Dir letzten Endes leicher und 
verständlicher gemacht. Weil Du genau weisst, wieviel Bits deine 
Variable definitiv hat.

Und als letzte Info: Nimm den #include <util/delay.h>  raus und setze 
die Optimierung auf -O0, sprich schalte sie aus.

Wenn alles läuft kannst Du später die Optimierung wieder aktivieren.

[OldMan (Gast)]

dude schrieb:
> sei();
>
>   TCCR0A |= (1<<CS00)|(1<<CS02);//Setting Prescaler clk/1024

Nachtrag:

Die Interrupts sollte man immer erst dann freigeben, wenn
alle Register vollständig initialisiert sind.
D.h., Dein sei() sollte nach dem setzen des Prescalers erfolgen.

Viel Erfolg!