ATMega8, Timer1, CTC, ISR wird nur einmal aufgerufen

[Sebastian (Gast)]

Hallo,

ich möchte gerne an einem ATMEGA8 einen Timer (Timer1, 16 Bit) im CTC 
Modus verwenden. Leider habe ich bei meinem Code das Problem, dass die 
ISR immer nur einmal aufgerufen wird. Ich finde leider den Fehler nicht.
IDE ist Atmel Studio 7 mit gcc Toolchain.
Sieht hier jemand den Fehler? Ich bin wahrscheinlich schon 
betriebsblind. :)

 #define F_CPU 8000000

 #include <avr/io.h>
 #include <avr/interrupt.h>
 #include <util/delay.h>
 #include "lcd-routines.h"
 #include "lcd-routines.c"

static int ISRs = 0;                // Zählvariable


void timer_init( void )
{ 
  TCCR1B = (1 << WGM12)|(1 << CS11)|(1 << CS10);  // CTC und Prescaler 64
  OCR1B = 125;                  // 8.000.000 / 64 / 1000 für 1ms
  TIMSK = 1<<OCIE1B;                // Timerinterrupts aktivieren
}


ISR( TIMER1_COMPB_vect )             
{
  ISRs++;          // Zählvariable hochzählen
  PORTC |= (1<<PORTC0);  // LED einschalten (Zeigt, dass die ISR einmal abgearbeitet wird)
  TIFR |= (1 << OCF1B);  // Muss ich hier OCF1B manuell zurücksetzen? Eigentlich nicht, oder?
}


int main( void )
{
  DDRC |= (1<<DDC0);    // Ausgang für LED
  PORTC &= ~(1<<PORTC0);  // LED zu Beginn auschalten
  timer_init();      // Timer initialisieren
  lcd_init();        // LCD initialisieren
  sei();          // Interrupts global aktivieren

  while(1)
  {

    if (ISRs == 500)
    {
      cli();              // Interrupts abschalten
      lcd_setcursor(0,2);        // LCD-Cursor positionieren
      lcd_string("ISR = 500");    // Text ausgeben
      _delay_ms(1000);        // Kurz warten
      lcd_clear();          // LCD wieder löschen
      sei();              // Interrupts wieder aktivieren
      ISRs = 0;            // Zählvariable resetten
    }
    
  }
  return 0;
}

[Falk B. (falk)]

Es gibt keinen CTC Mode mit OCR1B als Einstellmöglichkeit. Nimm mal 
OCR1A, ebenso für die ISR.

[Falk B. (falk)]

   if (ISRs == 500)
    {
      cli();              // Interrupts abschalten
      lcd_setcursor(0,2);        // LCD-Cursor positionieren
      lcd_string("ISR = 500");    // Text ausgeben
      _delay_ms(1000);        // Kurz warten
      lcd_clear();          // LCD wieder löschen
      sei();              // Interrupts wieder aktivieren
      ISRs = 0;            // Zählvariable resetten
    }

AUA! Interruptsperre für 1s? Hello? NOOOOOO!!!

Siehe Interrupt.

[Karl M. (Gast)]

Das ist doch sinnfrei die Interrupts zu sperren !

cli();              // Interrupts abschalten
      lcd_setcursor(0,2);        // LCD-Cursor positionieren
      lcd_string("ISR = 500");    // Text ausgeben
      _delay_ms(1000);        // Kurz warten
      lcd_clear();          // LCD wieder löschen
      sei();

[Carl D. (jcw2)]

static volatile int ISRs = 0;     // Zählvariable

Die ISR wird eventuell schon öfter gerufen, aber woran soll main() das 
erkennen? An der Variable die es selbst gerade auf 0 gesetzt hat und die 
nicht als "jederzeit veränderlich markiert" ist?

[Sebastian (Gast)]

Danke für die Antworten.
Ich habe alle vorgeschlagenen Änderungen eingepflegt, jetzt wird die ISR 
garnicht mehr ausgeführt.

 #define F_CPU 8000000

 #include <avr/io.h>
 #include <avr/interrupt.h>
 #include <util/delay.h>
 #include "lcd-routines.h"
 #include "lcd-routines.c"

static volatile int ISRs = 0;                // Zählvariable

void timer_init( void )
{ 
  TCCR1A = (1 << WGM12)|(1 << CS11)|(1 << CS10);  // CTC und Prescaler 64
  OCR1A = 125;                  // 8.000.000 / 64 / 1000 für 1ms
  TIMSK = 1<<OCIE1A;                // Timerinterrupts aktivieren
}


ISR(TIMER1_COMPA_vect)             
{
  ISRs++;          // Zählvariable hochzählen
  PORTC |= (1<<PORTC0);  // LED einschalten (Zeigt, dass die ISR einmal abgearbeitet wird)
  TIFR |= (1 << OCF1A);  // Muss ich hier OCF1B manuell zurücksetzen? Eigentlich nicht, oder?
}


int main( void )
{
  DDRC |= (1<<DDC0);    // Ausgang für LED
  PORTC &= ~(1<<PORTC0);  // LED zu Beginn auschalten
  timer_init();      // Timer initialisieren
  lcd_init();        // LCD initialisieren
  sei();          // Interrupts global aktivieren

  while(1)
  {

    if (ISRs == 500)
    {
      lcd_setcursor(0,2);        // LCD-Cursor positionieren
      lcd_string("ISR = 500");    // Text ausgeben
      _delay_ms(100);          // Kurz warten
      ISRs = 0;            // Zählvariable resetten
    }
    
  }
  return 0;
}

[Falk B. (falk)]

@Sebastian (Gast)

>Ich habe alle vorgeschlagenen Änderungen eingepflegt, jetzt wird die ISR
>garnicht mehr ausgeführt.

Ein BISSCHEN Mitdenken wäre vorteilhaft!

Eher so. RTFM!

TCCR1B = (1 << WGM12)|(1 << CS11)|(1 << CS10);  // CTC und Prescaler 64

[Sebastian (Gast)]

Okay, danke, es funktioniert nun. Aber ich verstehe nicht so ganz warum 
ich TCCR1B statt TCCR1A konfigurieren muss. Im Datenblatt finde ich dazu 
auch keinen Hinweis.

Und das mit den Interrupts, wenn ich für diese Zeit keine Interrupts 
haben möchte, kann ich das doch so machen, oder gibt es, technisch 
gesehen, etwas was dagegen spricht?

LG und danke für die Hilfe,
Sebastian

PS: Ich fand den Ton hier auch schonmal freundlicher, auch wenn man vllt 
"blöde" Fragen stellt.

[Falk B. (falk)]

@Sebastian (Gast)

>Okay, danke, es funktioniert nun. Aber ich verstehe nicht so ganz warum
>ich TCCR1B statt TCCR1A konfigurieren muss. Im Datenblatt finde ich dazu
>auch keinen Hinweis.

Brille putzen!

TCCR1A ist NICHT direkt mit OCR1A verbunden und
TCCR1B ist NICHT direkt mit OCR1B verbunden!

TCCR1A und TCCR1B konfigurieren ZUSAMMEN Timer1!

>Und das mit den Interrupts, wenn ich für diese Zeit keine Interrupts
>haben möchte, kann ich das doch so machen, oder gibt es, technisch
>gesehen, etwas was dagegen spricht?

Ja, siehe Interrupt, gibt es leider noch nicht als Hörbuch ;-)

>PS: Ich fand den Ton hier auch schonmal freundlicher, auch wenn man vllt
>"blöde" Fragen stellt.

https://de.wikipedia.org/wiki/Ironie

Hab dich nicht so mädchenhaft.

[Karl M. (Gast)]

Sebastian schrieb:
> Okay, danke, es funktioniert nun. Aber ich verstehe nicht so ganz
> warum
> ich TCCR1B statt TCCR1A konfigurieren muss. Im Datenblatt finde ich dazu
> auch keinen Hinweis.
>
> Und das mit den Interrupts, wenn ich für diese Zeit keine Interrupts
> haben möchte, kann ich das doch so machen, oder gibt es, technisch
> gesehen, etwas was dagegen spricht?
>
> LG und danke für die Hilfe,
> Sebastian
>
> PS: Ich fand den Ton hier auch schonmal freundlicher, auch wenn man vllt
> "blöde" Fragen stellt.

Als Tipp, der nett gemeint ist, findest Du das Datenblatt und alle seine 
Register und Text unter:

http://www.atmel.com/images/atmel-2486-8-bit-avr-microcontroller-atmega8_l_datasheet.pdf

Der Timer1 Register sind ab S.96ff beschrieben.

[Karl M. (Gast)]

Zu der Interrupt-Sperre, wäre es für uns sicherlich interessant das 
Warum zu erfahren.
Wir sehen die Dinge aus langjähriger Erfahrung evtl. anders und können 
Dir einen Ratschlag geben.