CTC-Verständnis: OCR1A scheint (!) ein Eigenleben zu führen

[Malte M. (mmalte)]

Hallo allerseits,

ich verwende den Timer1 meines ATmega88PA mit CTC, um eine LED mit 180 
bpm (3 Hz) blinken zu lassen. Ohne die Taster zu bedienen dachte ich, 
dass folgendes passieren müsste: Die LED an PB0 ist am Anfang an und 
wechselt dann eben mit 3 Hz zwischen an und aus, weil der Timer immer 
bis 5207 (Inhalt von OCR1A) zählt. Nun geht sie aber nach ner 
Drittelsekunde aus (soweit also richtig), danach wechselt sie aber nur 
noch ungefähr alle vier Sekunden. Zurückgerechnet wäre eine Erklärung 
dafür, dass der Timer einmal bis 5207 zählt, danach aber bis 
65535=0xFFFF, also OCR1A ignoriert oder zurücksetzt.

Was hab ich da falsch verstanden bzw. müsste ich anders machen, damit 
der Timer immer zum gleichen Wert zählt?

#define F_CPU  1000000UL // 1 MHz

#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>

#define LED (1 << PB0)
#define D_LED DDRB
#define P_LED PORTB
#define TASTER1_AUS (PINC & (1 << PC5))
#define TASTER2_AUS (PINC & (1 << PC4))

#define PRESCALER_LOG 6
#define PRESCALER_BITS ((1 << CS11) | (1 << CS10))

volatile int8_t blinken = 1;

ISR (TIMER1_COMPA_vect) {

  if (blinken) {
    P_LED ^= LED;
  } else {
    P_LED &= ~LED;
  }
  
}

// muss das int sein?
int main(void) {

  // Datenrichtungsregister setzen, LED an
  D_LED |= LED;
  P_LED |= LED;

  // Timer 1:
  // CTC (TOP = OCR1A), Prescaler setzen
  TCCR1A |= (1 << WGM12);
  TCCR1B |= PRESCALER_BITS;

  uint16_t bpm = 180;

  // Rundungsfehler beim Shift, wenn F_CPU nicht durch den Prescaler teilbar ist:
  // PRESCALER_LOG =  8 => 6*10^-5
  // PRESCALER_LOG = 10 => 6*10^-4
  // -1 am Ende, weil ja von 0 gezählt wird (?, TODO: nochmal nachlesen)
  OCR1A = 60 * (F_CPU >> PRESCALER_LOG) / bpm - 1;

  // Compare-Interrupt erlauben
  TIMSK1 |= (1 << OCIE1A);
  // Global Interrupts aktivieren
  sei();

  int8_t entpr_taster1 = 0;
  int8_t entpr_taster2 = 0;

  while (1) {

    _delay_ms(10);

    // TASTER1
    if (TASTER1_AUS) {
      entpr_taster1 = 0;
    } else if (entpr_taster1 < 4) {
      entpr_taster1++;
    } else if (entpr_taster1 == 4) {
      entpr_taster1++;
      // AKTION:
      blinken = !blinken;
    }

    // TASTER2
    if (TASTER2_AUS) {
      entpr_taster2 = 0;
    } else if (entpr_taster2 < 4) {
      entpr_taster2++;
    } else if (entpr_taster2 == 4) {
      entpr_taster2++;
      // AKTION:
      bpm += 10;
      cli(); // TODO: nötig? Denke schon wg. 16bit.
      OCR1A = 60 * (F_CPU >> PRESCALER_LOG) / bpm - 1;
      sei();
    }

  }

  return 0;

}

[g457 (Gast)]

> TCCR1A |= (1 << WGM12);
       ^          ^^^^^
WGM12 ist in TCCR1B.

[Malte M. (mmalte)]

g457 schrieb:
>> TCCR1A |= (1 << WGM12);
>        ^          ^^^^^
> WGM12 ist in TCCR1B.

Vielen Dank! Da wär ich ja nie drauf gekommen … Im Datenblatt stehen 
WGM13:0 alle schön übersichtlich in einer Tabelle im Abschnitt über 
TCCR1A. Und da hab ich den Hinweis überlesen, dass WGM13:2 sich in 
TCCR1B befinden ;)