Timer-Problem Atmega16 und Code::Blocks

[Laurenz K. (gimmebottles)]

Hallo,

ich habe ein Problem, ich möchte den Timer0 eines Atmega16 mit 
Overflow-Interrupt nutzen. Der Code steht unten, es wird aber nie in die 
ISR-gesprungen.

Ich verwende avr-gcc 4.5.3 und COde::Blocks 10.05. Erkennt jemand den 
Fehler?

Im Anhang ist der komplette Projektordner, vielleicht stimmen ja auch 
meine Einstellungen in Code::Blocks nicht.

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

#include "main.h"


uint8_t step, column, frame[100];

int main(void)
{
   uint8_t buf[100], i, ii;


   init();

//   PORTA = 0b11111111;
//   PORTC = 0b11111111;
//   PORTD = 0b11111111;

   for(i=0; i<100; i++) {
      frame[i] = 50;
   }

   while(0) {  //Hauptschleife

      while(!(SPSR & (1<<SPIF)))
         ;
      buf[i] = SPDR;
      if(buf[i] == 255) {
         switch(buf[0]) {
            case 0:  //Einzelne Lampe setzen
               frame[buf[1]] = buf[2];
            break;
            case 1:  //Alle Lampen gleich setzen
               for(ii = 0; ii < 97; ii++)
                  frame[ii] = buf[1];
            break;
            case 2:  //Ganzes Bild setzen
               for(ii = 0; ii < 97; ii++)
                  frame[ii] = buf[ii+1];
            break;
            case 3:  //Ganze Spalte setzen
               for(ii = 0; ii < 6; ii++)
                  frame[(buf[1]*7)+ii] = buf[2];
            break;
            case 4:  //Ganze Reihe setzen
               for(ii = 0; ii < 13; ii++)
                  frame[buf[1]+(ii*7)] = buf[2];
            break;
         }
         i = 0;
      }
      else
         i++;
   }

    return 0;
}

void init(void)
{
   DDRA = 0b11111111;   //Ports als Ausgänge schalten
   DDRC = 0b11111111;
   DDRD = 0b11111111;
   DDRB = 0b00001111;

   PORTA = 0b00000000;  //Alle Pins low, alle LEDs aus
   PORTC = 0b00000000;
   PORTD = 0b00000000;
   PORTB = 0b00000000;

   column = 0;
   step = 0;

   DDRB = (1<<PORTB6); //MISO als Ausgang
   SPCR = (1<<SPE);  //SPI aktivieren

   TCCR0 = (1<<CS01);   //timer0 konfigurieren,
   TIMSK |= (1<<TOIE0); //starten,
   TCNT0  = 141;        //und vorladen
   sei();
}

ISR(TIMER0_OVF_vect)
{
   uint8_t i;

   PORTB = 0b00001111;  //WIRD NIE AUSGEFÜHRT


   TCNT0  = 141;

   PORTA = 0b00000000;  //RESET ROWS A
   PORTC = 0b00000000;  //RESET ROWS B
   PORTC = 1 << column; //SET COLUMN column

   for(i = 0; i < 7; i++) {
      if(frame[(column*7)+i] <= step) {
         PORTA = 1 << i;   //SET ROW i IN A
      }

      if(frame[(column*7)+i+49] <= step) {
         PORTC = 1 << i;   //SET ROW i IN B
      }
   }

   column++;
   if(column > 6) {
      column = 0;
      step++;
      if(step > 50)
         step = 0;
   }
}

[Karl H. (kbuchegg)]

>    DDRB = (1<<PORTB6); //MISO als Ausgang

Du hast dir soeben die unteren 4 BIts am Port B, die in der ISR als 
Nachweis benutzt werden und die du vorher als Ausgang konfiguriert hast, 
wieder auf Eingang zurückkonfiguriert.

[Laurenz K. (gimmebottles)]

so, ich habe jetzt mal alles überflüssige entfernt, trotzdem springt er 
nicht in die ISR, PORTD0 bleibt high.

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

#include "main.h"

#define F_CPU 16000000UL


int main(void)
{



   init();

   PORTD |= (1<<PD0);



   while(0) {  //Hauptschleife


   }

    return 0;
}

void init(void)
{
   DDRA = 0b11111111;   //Ports als Ausgänge schalten
   DDRC = 0b11111111;
   DDRD = 0b11111111;
   DDRB = 0b00001111;

   PORTA = 0b00000000;  //Alle Pins low, alle LEDs aus
   PORTC = 0b00000000;
   PORTD = 0b00000000;
   PORTB = 0b00000000;

   TCCR0 = (1<<CS01);   //timer0 konfigurieren,
   TIMSK |= (1<<TOIE0); //starten,
   TCNT0  = 141;        //und vorladen
   sei();
}

ISR(TIMER0_OVF_vect)
{
   uint8_t i;


   PORTD = 0b00000000;

   TCNT0  = 141;



}

[Stefan E. (sternst)]

Laurenz K. schrieb:
> trotzdem springt er
> nicht in die ISR

Weil du main() verlässt. Nach main() kommt eine Endlosschleife mit 
ausgeschalteten Interrupts.

[Peter D. (peda)]

Laurenz K. schrieb:
> while(0) {  //Hauptschleife

Alles darin ist toter Code.


Schreib besser:

  for(;;){
    ...
  }

Das funktioniert immer.
Und jeder weiß sofort, was damit gemeint ist.


Peter