Timer1 Problem mit ATMEGA1280

[Bene J. (terfagter)]

Hallo zusammen,

versuche seit Stunden Timer1 im Compare-Betrieb laufen zu lassen, sodass 
ein Interrupt alle 1ms ausgelöst wird.
Mein Programm unterbricht beim Initialisieren des Timers schon alles und 
führt nichts weiter aus!
In die Interrupt-Routine zählt ich bis zu einer Sekunde hoch und gebe 
einen STring aus. Das ist später nicht die Aufgabe der Routine, sondern 
gerade nur zum testen.
Quarz: 8000000

Was mache ich falsch?

Mein Code:

#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <stdio.h>
#include <avr/pgmspace.h>
#include <string.h>
#include <stdlib.h>

#include "main.h"

uint8_t mssec = 0;
  
int main (void) 
{
  
  init_uart0( (u16)(XTAL / BAUD0 / 16 - 0.5) );      

  TIMSK1 |= (1<<OCIE1A);
  
  OCR1A = 124;
  TCNT1 = 0;                        
  
  TCCR1A |= (1 << WGM12);
  TCCR1B |= (1<<CS11) | (1<<CS10);
  sei();  
  while(1){}
}


ISR(TIMER1_COMPA_vect)
{
  mssec++;
  if(mssec == 1000)
  {
    mssec = 0;
    uputs0("1 sec vergangen");
  }
}

Vielen Dank für eure Hilfe!

[Helfer (Gast)]

> Mein Programm unterbricht beim Initialisieren des Timers schon alles und
> führt nichts weiter aus!

Bug in (Endlosschleife?) oder beim Rücksprung (Stack falsch 
initialisiert? falscher AVR im Projekt eingestellt?) aus der nicht 
gezeigten Funktion init_uart0()?

Hardwareprobleme beim RS232 Transfer AVR <-> PC? Funktioniert UART im 
Hello World Programm?

> ISR(TIMER1_COMPA_vect)
>     uputs0("1 sec vergangen");

Angenommen die Interrupts kommen alle ms. Wie viele Interrupts verlierst 
du durch uputs0() bei Baudrate BAUD0? Funktioniert das Toggeln einer LED 
alle Sekunde in der ISR?

>  TCCR1A |= (1 << WGM12);
>  TCCR1B |= (1<<CS11) | (1<<CS10);

CTC1 Modus OK. Prescaler 64. 8000000/64/(124+1) = 1ms OK.
http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Die_Timer_und_Z%C3%A4hler_des_AVR

[Bene J. (terfagter)]

Also die uart Initialisierung funktioniert mit 9600 baud. Ohne die 
Initialisierung des timers läuft das Programm auch durch, wenn ich z.b. 
über die uart einen String ausgeben...

[Stefan E. (sternst)]

Welchen UART-Code verwendest du?

[Bene J. (terfagter)]

Von Peter danegger. interrupt gesteuert.

[Fabian G. (kjion)]

Hi,

uint8_t mssec = 0;
...  
if(mssec == 1000)

das kann so nicht funktionieren und dein Compiler sollte dir auch eine 
entsprechende Warnung ausgeben. uint8_t kann maximal bis 255 zählen.

Besser:

ISR(TIMER1_COMPA_vect)
{
  static uint16_t mssec = 0;
  
  mssec++;
  if(mssec == 1000)
  {
    mssec = 0;
    uputs0("1 sec vergangen");
  }
}

[Stefan E. (sternst)]

Bene Jan schrieb:
> Von Peter danegger. interrupt gesteuert.

Und du meinst, es wäre dann eine gute Idee, die Funktion uputs0 an einer 
Stelle aufzurufen, wo Interrupts gerade abgeschaltet sind? Wenn der 
Puffer nicht mehr den kompletten String aufnehmen kann, hast du einen 
schönen Dead-Lock.

[Bene J. (terfagter)]

Vielen Dank.
Es lag an zwei Sachen:
1. habe ich mich im Datenblatt verguckt und WGM12 in TCCR1A benutzt 
anstall in TCCR1B
2. In der Interrupt-Schleife uputs0 verwendet, was bei mir Interrupt 
gesteuert ist. Da gab es dann einen Dead-Lock.
Jetzt funktionierts.