[ATMega32]Unbekannter Fehler beim Timer0 overflow

[Jahn-Hendrik K. (pitillo)]

Hallo,
ich habe einen code für eine Maschine ähnlich einer CNC-Fräse 
geschrieben darin gibt es jedoch probleme beim Timer0 und ich weiß 
einfach nich wo der fehler entsteht und wie ich ihn beheben soll.
Main.c

#include <avr/io.h>
#include <avr/interrupt.h>
#define F_CPU 1000000UL  // 1 MHz
#include <util/delay.h> 
#include "Main.h"



int main(void)
{
   //Timer 0 auf 1024 prescale
   TCCR0 = (1<<CS01 | 1<<CS02);
   // Overflow Interrupt erlauben
   TIMSK |= (1<<TOIE0);
   //interupts erlauben
   sei();
  //Port C als eingang definieren
   DDRC = 0x00;
   DDRD  = 0xFF;
   Motor3 = 0;
   //in Motor1 und 2 wird die anzahl der schritte pro tastendruck angegeben
   Motor2 = 0;
   Motor1 = 0;
   while (1)
   {
     //Taster abfragen
     if (PINC0 == 1) 
     {
       Motor3 =+ 1;
     }         
     if (PINC1 == 1)
     {
      Motor3 =- 1;
     }
     if (PINC2 == 1)           
     {
       
       for (uint8_t i = Motor1;i++;)
       {
         if (M1 == 1)
         {
           PORTD = (1<<PD0);
         }
         if (M1 == 0)
         {
           PORTD = (0<<PD0);
         }
         PORTD = (1<<PD1);
         _delay_ms(1);
         PORTD = 0x00;
         _delay_ms(1);       
       }         
     }    
     if (PINC3 == 1)           
     {
       for (uint8_t i = Motor1;i++;)
       {
         if (M1 == 0)
         {
           PORTD = (1<<PD0);
         }
         if (M1 == 1)
         {
           PORTD = (0<<PD0);
         }
         PORTD = (1<<PD1);
         _delay_ms(1);
         PORTD = 0x00;
         _delay_ms(1);     
       }        
     }     
     if (PINC4 == 1)           
     {
       for (uint8_t i = Motor2;i++;)
       {
         if (M2 == 1)
         {
           PORTD = (1<<PD2);
         }
         if (M2 == 0)
         {
           PORTD = (0<<PD2);
         }
         PORTD = (1<<PD3);
         _delay_ms(1);
         PORTD = 0x00;
         _delay_ms(1);
         PORTD = 0x00  ;       
     }    
     if (PINC5 == 1)           
     {
       for (uint8_t i = Motor2;i++;)
       {
         if (M2 == 0)
         {
           PORTD = (1<<PD2);
         }
         if (M2 == 1)
         {
           PORTD = (0<<PD2);
         }
         PORTD = (1<<PD3);
         _delay_ms(1);
         PORTD = 0x00;
         _delay_ms(1);
         PORTD = 0x00;       
      }    
     }   
   }
  
}
#ifndef TIMER0_OVF_vect
#define TIMER0_OVF_vect TIMER0_OVF0_vect
#endif
ISR (TIMER0_OVF_vect)
{
  for (uint8_t i = Motor3; i++;)
  {
    if (M3 == 1)
    {
      PORTD = (1<<PD4);
    }
    PORTD = (1<<PD5);
    _delay_ms(1);
    PORTD = 0x00;
    _delay_ms(1);
  }
  for (uint8_t i = Motor3; i++;)
  {
    if (M3 == 1)
    {
      PORTD = (0<<PD4);
    }
    PORTD = (1<<PD5);
    _delay_ms(1);
    PORTD = 0x00;
    _delay_ms(1);
  }
}

Und auch noch die Header Datei

//Drehrichtungs einstellung
#define M1 1
#define M2 1 
#define M3 1
unsigned int Motor1 = 0;
unsigned int Motor2 = 0;
unsigned int Motor3 = 0;

ich hoffe ihr könnt mir helfen

[Karl H. (kbuchegg)]

Jahn-Hendrik K. schrieb:

> ich hoffe ihr könnt mir helfen

Wie soll ich es dir sagen?

OK. Die freundliche Variante ist: in deinem Code gibt es eine Menge 
Probleme.

Die aufrichtige Variante: Lern bitte erst mal die Grundlagen, ehe du 
dich an die reale Harware wagst und noch etwas kaputt machst. In deinem 
Code sind in 3 Zeilen (überspitzt ausgedrückt) 5 Fehler. Und zwar von 
oben bis unten. Fast egal, welche 3 Zeilen man herausnimmt.
Und das fängt bereits beim Setzen von bestimmten Portpins auf 1 oder 0 
an, geht weiter über die korrekte Syntax von for-Schleifen und endet 
nnoch lange nicht beim Abfragen von Portpins. Über so Kleinigkeiten wie 
einem total verkorkstem Aufbau der ISR reden wir erst mal gar nicht. 
Dass du deine Eingabepins nicht flankengesteuert und entprellt abfragst 
und daher die Variable Motor3 (mehr oder weniger zufällig) ihren Wert 
gar nicht ändern kann, weil sich die Effekte der Portpins, so man sie 
korrekt abfragen würde, aufheben ist dann nur noch das Sahnehäubchen.

[Mutti (Gast)]

Mache niemals delays in Interrup methoden.

 TCCR0 = (1<<CS01 | 1<<CS02) ergibt m.E nicht 1024 prescale
Prüfe das bitte nochmal.

[Jahn-Hendrik K. (pitillo)]

könntet ihr mir vielleicht sagen was genau meine fehler waren ich hab 
mir die grundlagen bereits durchgelesen ich hab halt vorherr nur NXT´s 
programmiert(in NXC) deswegen dachte ich mir wenn ich mir das tutorial 
durchlese wird das schon nicht so schwer werden. vorher hab ich 
eigendlich auch bascom nutzen wollen jedoch reichen mir evt die 4kb 
nachherr nicht ausalso fing ich an avr gcc zu lernen und assambler....
könntet ihr mir wirklich ma bitte sagen was ich beim programmieren 
falsch mache  wäre wirklich super nett von euch

MfG Pitillo

[Oliver (Gast)]

Jahn-Hendrik K. schrieb:
> und ich weiß
> einfach nich wo der fehler entsteht

Nun ja, wenn du nicht mal beschreiben kannst, wie sich "der Fehler" 
(bzw. "unbekannter Fehler") eigentlich äussert, wird es sehr schwer.

Jahn-Hendrik K. schrieb:
> deswegen dachte ich mir wenn ich mir das tutorial
> durchlese wird das schon nicht so schwer werden.

Das ist schon nmal ein guter Ansatz, nur reicht lesen nicht. Du solltest 
das Kapitel für Kapitel durcharbeiten und ausprobieren, so lange, bis 
das dort besprochene auf deinem Prozessor tut, was es soll.

Erst, wenn du erfolgreich IO setzen und lesen, einen Taster abfragen, 
und eine LED per Timer blinken lassen kannst, erst dann kannst du 
vielleicht mal drüber nachdenken, das alles zu einem größeren Programm 
zusammenzusetzen.

Oliver

[Ulrich (Gast)]

Da sind viele Fehler drin, vielleicht nicht ganz so viele wie in der 
ersten Antwort angedroht, aber doch reichlich.

Einer der ersten ist:
 if (PINC0 == 1)

PINC0 ist als eine Konstante (=0) definiert, die die Bit Nummer angibt. 
Der sollte der Compiler ein if (0 == 1) sehen und entsprechend eine 
Warnung ausgeben, dass die Bedingung immer Falsch ist.

Auch sonst sind da noch grobe Patzer dirn, es fehlt z.B. eine Entprellen 
der Taster, Delay im Interrupt, kein Volatile.