ATMEGA8 SREG + I setzen

[Stefan (Gast)]

Guten Abend,

ich versuche verzweifelt an meinen ATMEGA 8 den Timerinterupt zum laufen 
zu bekommen.
Laut Datenblatt muss ich dazu auch im SREG das I-Bit auf 1 setzen.
Ich bekomm es leider nicht hin.

SREG |= (1<<I);

Das war leider etwas naiv. Kann mir jemand weiterhelfen bitte?

[holger (Gast)]

>Ich bekomm es leider nicht hin.
>
>SREG |= (1<<I);
>
>Das war leider etwas naiv. Kann mir jemand weiterhelfen bitte?

sei();

[J. T. (chaoskind)]

Das I-Flag setzt du mit "sei", übrigens werden einige Flags auch 
gesetzt, indem man eine null schreibt... Da muss man immer das 
Datenblatt lesen, aber da du Ausschnitte davon zitiert hast, scheinst du 
das ja fleißig zu nutzen ;-)

[Stefan (Gast)]

Oh ok, dann wird damit das I-Bit gesetzt? Dann hab ich das doch mit 
drin. Aber irgendwas stimmt nicht. Denn das Programm hängt sich 
zwischendurch irgendwie auf.

        wenn ich hier eine Variable verändere, dann erfolgt die Änderung
  TIMSK |= (1<<TOIE0);  
  TCCR0 = (1<<CS01);    
  sei();   
        wenn ich hier die Variable verändere, dann wird die Änderung nicht übernommen

[holger (Gast)]

>        wenn ich hier eine Variable verändere, dann erfolgt die Änderung

Was soll der Quatsch? Zeig dein Programm so wie es ist.
Prosa kannst du dir an die Wand nageln.

[Stefan (Gast)]

#define F_CPU 1000000UL

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

#define Port_Zeile    PORTD
#define Z1        PD0
#define Z2        PD1
#define Z3        PD2
#define Z4        PD3
#define Z5        PD4
#define Z6        PD5
#define Z7        PD6

#define Port_Spalt    PORTC
#define S1        PC0
#define S2        PC1
#define S3        PC2
#define S4        PC3
#define S5        PC4


//globale Variable zur Auwertung bei Interrupts
volatile uint8_t isrFlag;
//globale Variable für Zeilendurchlauf (veränderbar in der ISR (volatile))
volatile uint8_t isrZeile;


//Array für die Matrix
char leds[7] =  {0b00011111,\
         0b00010101,\
         0b00000000,\
         0b00000111,\
         0b00011000,\
         0b00000111,\
         0b00000000};


ISR( TIMER0_OVF_vect )       // Overflow Interrupt Vector
{

  //Zeile ausschalten
    Port_Zeile |= (1<<isrZeile);

  //Inkrementieren der Zeilen
    isrZeile++;

  //Nach der 7.Zeile wieder auf 0 zurücksetzen
    if (isrZeile > 6)
      isrZeile =0;

  //Spalte festlegen
    Port_Spalt = leds[isrZeile];

  //Zeile einschalten
    Port_Zeile &= ~(1<<isrZeile);
}




int main (void) {  
  //Datenrichtungen PORTD/PORTC festlegen
  DDRD  = 0b01111111;       
  DDRC  = 0b00011111;    
  
    leds[0] =  0b00000001;
    leds[1] =  0b00000001;
    leds[2] =  0b00000001;
    leds[3] =  0b00000001;
    leds[4] =  0b00000001;
    leds[5] =  0b00000001;
    leds[6] =  0b00000001;

  //Timer konfigurieren
  TIMSK |= (1<<TOIE0);  // den Overflow Interrupt des Timers freigeben
      leds[0] =  0b00000010;
    leds[1] =  0b00000010;
    leds[2] =  0b00000010;
    leds[3] =  0b00000010;
    leds[4] =  0b00000010;
    leds[5] =  0b00000010;
    leds[6] =  0b00000010;

  TCCR0 = (1<<CS01);    // Vorteiler 8, jetzt zählt der Timer bereits

    leds[0] =  0b00000100;
    leds[1] =  0b00000100;
    leds[2] =  0b00000100;
    leds[3] =  0b00000100;
    leds[4] =  0b00000100;
    leds[5] =  0b00000100;
    leds[6] =  0b00000100;
  //Interrupts aktivieren
  sei();                

    leds[0] =  0b00001000;
    leds[1] =  0b00001000;
    leds[2] =  0b00001000;
    leds[3] =  0b00001000;
    leds[4] =  0b00001000;
    leds[5] =  0b00001000;
    leds[6] =  0b00001000;

  while(1){

  
    
  }
 
}

Nach sei(); wird die Änderung nicht übernommen.

ATMEGA8A-PU

[holger (Gast)]

>Nach sei(); wird die Änderung nicht übernommen.

Wozu auch. Die Variablen werden nicht mehr verwendet und
vom Compiler wegoptimiert.

Versuch mal:

volatile char leds[7] =  {0b00011111,\

[Stefan (Gast)]

Ach mist! Den Fehler hatte ich vor kurzen schonmal. Danke dir. 
Funktioniert natürlich.