CTC- Compare IRQ Anwendung macht Probleme

[Gerhard H. (oderlachs)]

Hallo Freunde !!


Ich versuche  eine LED im Sekundentakt blinken zu lassen und die andere 
LED soll alle 5 sec. toggeln.
Ich bekomme das mit den 5 sec. einfach nicht in den Griff, bestimmt habe 
ich mich schon selber in meinen Codefehler blind gesehen ;)

#define F_CPU 3686400UL
#include <avr/io.h>
#include <avr/interrupt.h>


/*====================================================================*/
/*                                                                    */
/*            MCU = ATmega 8                                          */
/*                                                                    */
/*                                                                    */
/*====================================================================*/
// OCR1A = (F_CPU/256)-1

 
#define LED0 (1<<PB0)                // LED auf Portpin B0
#define LED1 (1<<PB1)                // LED auf Portpin B1
#define OCRWERT  (F_CPU/256)-1; 
unsigned int sec= 0;

void ctc_init()
{
  DDRB |= (1<<PB0);               // Portpin B0 = output
  DDRB |= (1<<PB1);        // Portpin B1 = output
  //PORTB = 0xff;             
      TCCR1B |= (1 << CS12);          // Vorteiler auf 256 und Timer start
      TCCR1B |= (1 << WGM12);         // Mode 4, CTC on OCR1A
      OCR1A = OCRWERT;                // Compare match auf OCRWERT setzen 
                  // = (F_CPU/256)-1  
                  
      TIMSK |= (1 << OCIE1A);      // Interrupt bei compare match aktivieren
     
      sei();                        // enable interrupts
    
}

int main(void)
{
    ctc_init();
     
    while(1)
    {
    if (sec==5)
    { 
      PORTB ^= LED1;  // toggle LED ..ist NICHT OK 
           sec= 0;         // sec auf 0 setzen
          }
    }
}
 
ISR (TIMER1_COMPA_vect)            // Timer1 Compare Match Interrupt
{
    PORTB ^= LED0;                 // LED im Sekundentakt toggeln lassen = OK !!
     sec = sec+1;                    // sec um 1 erhöhen
                   
}

Was mache ich da nur falsch ??

Gruss und Dank

Gerhard

[holger (Gast)]

volatile unsigned int sec= 0;

[Martin O. (ossi-2)]

Mach die Variable sec mal volatile.

[Gerhard H. (oderlachs)]

Vielen Dank !!!!

Das war es , daran hat ich in meinen "Jugendlichem Leichtsinn" gar nicht 
gedacht.... Ich habe schon an mich selber gezweifelt!! tzzz

Gruss & Dank

Gerhard

Nachtrag : Kommt zwar mit den 5 Sec. nicht zeitmässig hin, aber das ist 
erst mal ein anders "Bier"... :)

Auch diesen Fehler gefunden , Sass gleich daneben ;), habe die 
Leuchtzeiten gezählt..aber dazwischen war ja immer noch 1 Sec. 
dunkel-Phase der 1-Sec. -LED. Stoppuhr genommen Verdacht bestätigt : 
Denkfehler.

Vielen Dank für Eure Mithilfe, schönes Wo-Ende

Gerhard