AVR-Zählervariable wird zurückgesetzt

[Alex (Gast)]

Hey Leute...

Ich bin dabei ein kleines Programm zu schreiben und folgendes Problem 
liegt vor:

Ich habe einen Timer für allgemeine Zählfunktionen eingerichtet.
Leider musste ich feststellen, dass das abfragen des Wertes nicht 
funktioniert.
Nach einigen hin und her fand ich heraus, dass die Variable "zaehler" 
wie von selbst auf 0 gesetzt wird, sobald der wert > 7 ist.
Das selbe passiert auch wenn ich den wert auf 50 setze und hochzählen 
lasse, dann zählt er bis 60 danach kommt die 0.

Der Code ist mittlerweile soweit reduziert, dass der Timer0 die Variable 
"zaehler" hochzählt und das ganze weiter unten nur mal abgefragt wird. 
Trotzdem das selbe Problem...

Kann mir jemand verraten worin der Fehler liegt?

Hier mein Code:

/*
 * LED_Badezimmer.cpp
 *
 * Created: 05.09.2015 15:49:07
 *  Author: Alex
 */ 

/*
  PD2 = Taster Interrupt0
  PB0 = Relais
  PB1 = Transistor PWM Timer/Counter
  1sek. Timer/Counter autom. abschaltung nach 2std ->120min ->7200sek.
*/


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

void entprellung (void);
 
bool interrupt;
bool Taster_use;

volatile uint8_t zaehler;



 
 ISR (INT0_vect)
 {
  interrupt = 1;

 }
 
 



 void write16bitregister (uint16_t m_OCR1) //OCR1A max. 10bit ->1023!
 {
 
   unsigned char sreg;
   
   sreg = SREG;
   
   cli();
   
   OCR1AH = (m_OCR1>>8);
 
   OCR1AL = (m_OCR1 & 0x0FF);
   
   
   SREG = sreg;
  sei();
 
 }



ISR(TIMER0_OVF_vect)
{
  zaehler++;

}


int main(void)
{

  MCUCR |= (1<<ISC01)|(1<<ISC00);
  GICR |= (1<<INT0);
  GIFR |= (1<<INTF0);
  
  TCCR1A |= (1<<COM1A1)|(1<<WGM12)|(1<<WGM11)|(1<<WGM10);
  TCCR1B |= (1<<CS10);
  TIMSK |= (1<<OCIE1A)|(1<<TOIE1);
   
   DDRB |= (1<<PB0)|(1<<PB1);
   
   

   
  TCCR0 |= (1<<CS00);
  TIMSK |= (1<<TOIE0);

   
   
   
  uint16_t soll;
  uint16_t ist;
   
   
   sei();
    while(1)
    {

    entprellung();
    if (Taster_use == 1)
    {
/*      write16bitregister(20);*/
      soll = 1023;
      PORTB |= (1<<PB0);
    }
    else
    {
/*      write16bitregister(0);*/
      soll = 0;
      PORTB &= ~(1<<PB0);
    }
        
    if(zaehler == 100)
    {
      ist++;
      write16bitregister(20);  
      zaehler = 0;
    }
    }
}


void entprellung (void)
{
  
  bool entprell;
  uint16_t time1;
  
  
  if (interrupt == 1 && entprell == 0)
  {
    Taster_use ^= 1;
    entprell = 1;
     interrupt = 0;
   }
  if(interrupt == 1 && entprell == 1)
  {
    interrupt = 0;
  }
  if (entprell == 1)
  {
    time1++;
  } 
  if(time1 > 20000)
  {
    entprell = 0;
    time1 = 0;
  }
}

Programmieren tue ich einen ATmega8 mit AVR Studio 6.2.9200.0 - Service 
Pack 2.

Vielen Dank für eure Mühe!

MfG Alex

[Tom (Gast)]

https://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Datenaustausch_mit_Interrupt-Routinen

[Tom (Gast)]

Und soll
  bool entprell;
vielleicht static sein? So wie jetzt macht das gar keinen Sinn.

[Alex (Gast)]

Tom schrieb:
> https://www.mikrocontroller.net/articles/AVR-GCC-T...

Danke für den Link, ich werde es mir nochmal genau durchlesen.

Tom schrieb:
> Und soll
>   bool entprell;
> vielleicht static sein? So wie jetzt macht das gar keinen Sinn.

ja stimmt, mein Fehler!



Habe ebend mal die ganzen Einstellungen für Timer1 also TCCR1A, TCCR1B 
und TIMSK auskommentiert, daraufhin hat Timer0 wie gewollt hochgezählt.

Sind die Timer in mancher Hinsicht nicht unabhängig?
Im Datasheet steht, dass die beiden sich den selben Prescaler teilen.
Andererseits steht dort aber auch, das dort trotzdem unterschiedliche 
Werte verwendet werden können.

[Peter D. (peda)]

Alex schrieb:
> Sind die Timer in mancher Hinsicht nicht unabhängig?

Aber nicht, wenn Du für den anderen Timer Interrupts ohne Handler 
freigibst.

[N. G. (newgeneration)]

interrupt sollte volatile sein, sonst wird der Vergleich mit ihm 
wegoptimiert.

[Mitlesa (Gast)]

Tom schrieb:
> Und soll  bool entprell;  vielleicht static sein?

Und sollte man nicht Variablen initialisieren bevor man sie benutzt?
(gilt für mehrere Variablen  hier)

... und könnte man sich vielleicht ein paar Leerzeilen sparen
um mehr Code auf dem Bildschirm auf einmal zu sehen?

[Alex (Gast)]

Peter D. schrieb:
> Alex schrieb:
> Sind die Timer in mancher Hinsicht nicht unabhängig?
>
> Aber nicht, wenn Du für den anderen Timer Interrupts ohne Handler
> freigibst.

Hatte es gestern noch schnell im Simulator getestet, scheint zu 
funktionieren!

N. G. schrieb:
> interrupt sollte volatile sein, sonst wird der Vergleich mit ihm
> wegoptimiert.

Mitlesa schrieb:
> Tom schrieb:
> Und soll  bool entprell;  vielleicht static sein?
>
> Und sollte man nicht Variablen initialisieren bevor man sie benutzt?
> (gilt für mehrere Variablen  hier)
>
> ... und könnte man sich vielleicht ein paar Leerzeilen sparen
> um mehr Code auf dem Bildschirm auf einmal zu sehen?

Danke, ich bin noch sehr unerfahren und werde versuchen das genannte 
einzuhalten!

Vielen Dank für eure Hilfe!
MfG Alex