TIMER1 Overflow interrupt wird nicht aufgerufen

[M. H. (dbzwerg)]

Hallo zusammen,

irgendwie steh ich auf dem Schlauch...

Bei angefügtem Code wird der TIMER1 OVerflow Interrpt nicht 
aufgerufen... einer ne Idee?

Die ISR soll eigentlich alle 100ms aufgerufen werden und mir einen 
Portpin toggeln ( nur zu testzwecken).

Hoffe jemand ist nicht so blind wie ich im moment.

MFG

 
//+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

// Demoprogramm für den ClockClock Slave

// µC :ATmega8

// CPU Takt : 8MHz

// Funktionsbeschreibung:

//------------------------------

// Version: V0.1
// Datum: 05.08.12



//------------------------------

//+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

#include <avr/io.h>
#include <avr/interrupt.h> 
#include "uart.h" //Enthällt uart funktionen (Achtung uart.c muss in makefile eingetragen werden, wenn dies nicht von der ide gemacht wird)
#include <stdlib.h> // enthällt die Umwandlung von int zu char (itoa)
#include <util/delay.h> // Enthällt delay funktionen
#include <avr/wdt.h>  // Enthält Watchdog Funktionen

// Definitionen

  #define TIMER1_preload     53036

  #ifndef F_CPU
  #define F_CPU         8000000UL  // CPU Takt in Hz
  #endif
  #define UART_BAUD_RATE    9600  




//Deklaration der Variablen

  int test_text                = 122;

  int time_counter              =    0; // Enhällt die Bits zur Intervall Erzeugung
                            // 1Bit = 100ms

//Deklaration der Funktionsprototypen

void send_int_uart(int value); // Deklaration für UART INt SEND Unterfunktion



//Interrupts


// Aktion bei Timer 1 Overflow (alle 100ms)
ISR (TIMER1_OVF_vect)
{

  PORTD ^= (1 << PD5 ); // Test Led 1 toggeln
  
  
  TCNT1 = 53036; // TImer1 vorladen damit er  nach 100ms überläuft
}





// Hauptschleife
//++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++  
  int main (void) // Start Hauptschleife
  {
  
  
  
  while(1)  // Start Dauerschleife
  {
    // Eingänge definieren
    


    // Ausgänge definieren
      DDRD |=(1<< PD1);       //UART Tx
      DDRD |=(1<< PD5);       //Test Led1
    
    // UART initialisieren

      uart_init( UART_BAUD_SELECT(UART_BAUD_RATE,F_CPU) );   

    
      // Startkonfig Timer 1 
  
        TCCR1B |= (1<<CS10);   // Prescaler /64 = Overflow alle 100ms und Timer starten
        TCCR1B |= (1<<CS11);   //
        
        TIMSK |= (1<<TOIE1);   // Overlfow ISR erlauben

        
        TCNT1 = 53036;      // TImer1 vorladen damit er  nach 100ms überläuft
    
    // Global Interrupts aktivieren
      sei();  

      

  
      

    
    /*
    uart_puts("Test");
    uart_puts("\n\r");
    send_int_uart(test_text);
    send_int_uart(test_text);
    uart_puts("\n\r");
    */
    
    
    

    
    
    
      

  }//Ende Dauerschleife
  }//Ende Hauptschleife

//+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

//Unterfunktion Integerwert über uart ausgeben
  void send_int_uart(int value)
  {
    char buffer [33];
    itoa (value,buffer,10);
    uart_puts(buffer);
  }

[MWS (Gast)]

Wie soll er auch weiterzählen, wenn er in der while immer wieder auf 
seinen Preload gesetzt wird ? Pack die while mal woanders hin.

[Karl H. (kbuchegg)]

Es macht keinen Sinn, wenn du den Timer dauern neu initialisierst

Mach deine Initialisierung EINMAL am Programmanfang und dann lass dein 
Programm die Hauptschleife betreten, in der es dann eben diesmal 
momentan (noch) nichts zu tun gibt. Ist ja kein Problem, wenn das so 
ist.

int main()
{

   die Initialisieungen
   wie zb Hardware Einstellungen
   Timer einstellungen etc.

   sei();

   while( 1 ) 
   {
      Logik in der Hauptschleife
   }
}

so sieht dein grundsätzlicher Programmaufbau aus.

[Karl H. (kbuchegg)]

Und schmeiss die exzessiven Leerzeilen raus!

Dein Programm wird dadurch nicht übersichtlicher. Ganz im Gegenteil, du 
ziehst es nur künstlich in die Länge, ohne dass du etwas dafür bekommst.

Achte lieber auf konsistente Einrückung. Da hast du mehr davon.

Dann erledigt sich die 'Bin ich blind' Frage oft ganz von alleine.

[M. H. (dbzwerg)]

Super Jungs!!!

Manchmal sieht man die einfachsten Dinge nicht :-)

@ Karl Heinz

Die Leerzeilen fliegen auf jedfefall noch raus...