Grundsatzproblem

[Arnhold (Gast)]

Hallo Leute,

ich habe hier ein ganz einfaches Testprogramm:

 #define F_CPU 8000000UL //Taktfrequenz auf 8 Mhz festlegen

#include <avr/interrupt.h>  //dient zur Behandlung der Interrupts
#include <stdint.h>       //definiert den Datentyp uint8_t
#include <avr/io.h>


uint8_t timer250ms = 0;


int main (void)
{
  //Ports initialisieren
  DDRD = 0b11111111; // PortD als Ausgang
  PORTD = 0b00000000; // Alle LEDs aus
    
  //Timer 1 konfigurieren alle 250ms
  TCCR1B |= (1<<CS11) | (1<<CS10);
  //Overflow Interrupt erlauben
  TIMSK |= (1<<TOIE1);

  sei();

  while (1) 
  {  
    if (timer250ms==3) PORTD = 0b11111111;
  }

  return 0;
}



ISR (TIMER1_OVF_vect)
{
  // Vorfüllen des Timers
  TCNT1 = 34286; //8 Mhz

  timer250ms++;
}

Eigentlich sollte der Timer vier Runden laufen und somit nach einer 
Sekunde die LEDs an Port D schalten. Es passiert jedoch NICHTS :(

Wenn ich den Code wie folgt ändere klappt es:

  #define F_CPU 8000000UL //Taktfrequenz auf 8 Mhz festlegen

#include <avr/interrupt.h>  //dient zur Behandlung der Interrupts
#include <stdint.h>       //definiert den Datentyp uint8_t
#include <avr/io.h>


uint8_t timer250ms = 0;


int main (void)
{
  //Ports initialisieren
  DDRD = 0b11111111; // PortD als Ausgang
  PORTD = 0b00000000; // Alle LEDs aus
    
  //Timer 1 konfigurieren alle 250ms
  TCCR1B |= (1<<CS11) | (1<<CS10);
  //Overflow Interrupt erlauben
  TIMSK |= (1<<TOIE1);

  sei();

  while (1) 
  {  
    
  }

  return 0;
}



ISR (TIMER1_OVF_vect)
{
  // Vorfüllen des Timers
  TCNT1 = 34286; //8 Mhz

  timer250ms++;
  if (timer250ms==3) PORTD = 0b11111111;
}

Es ist ein sehr viel umfangreicheres Projekt welches ich zur Fehlersuche 
soweit zurückgebaut habe das es so wie oben aussieht.

Braucht die while (1) Schleife mehr zu tun um mehrfach zu laufen? Wenn 
ich die if-Abfrage in der while-Schleife weglasse, dann gehen die LEDs 
auch an. Also einmal scheint die Schleife somit zu laufen.

Viele Grüße
Arnhold

[Tom K. (ez81)]

http://www.mikrocontroller.net/articles/AVR_Checkliste#Alle_Interrupt_Variablen_als_volatile_bezeichnet.3F

[Arnhold H. (arnhold)]

ZAUBEREI!!! DANKE

[Arnhold H. (arnhold)]

Falls jemand auf diesen Artikel stößt:

Die Lösung ist bei der Variablendefiniion

volatile uint8_t timer250ms = 0;

zu schreiben.

[test (Gast)]

Wenn du dich jetzt noch insofern mit volatile beschäftigst und dann 
weißt WARUM das dein Problem behebt, dann hast du wirklich die Lösung.