LCD, timer, Problem

[al3ko (Gast)]

Hi,
Ich taste mich gerade an Timer heran. Dazu verwende ich eine Atmega8 
Beschaltung mit einer LED (PB1), einem Taster (PB0) und einem LCD.

Jede Sekunde soll die Variable bla aktualisiert und auf dem LCD 
angezeigt werden. Beim Druecken vom Taster soll die Variable auf 0 
zurueckgesetzt werden.

Leider zaehlt das Display einfach weiter. Die Tastererkennung 
funktioniert einwandfrei, da die LED ein- bzw. ausgeschaltet wird. Ich 
verstehe nur nicht, weshalb das bla=0 nicht in die ISR uebernommen wird.

Kann mir wer meinen Fehler aufzeigen?

Danke.

Hier der Code:

#include <avr/io.h>
#include <util/delay.h>
#include "lcd-routines.h"
#include <avr/interrupt.h>
#include <stdlib.h>

uint32_t bla;
char buffer[20];

/*
Makro for debouncing
*/
#define debounce( port, pin )                                         \
({                                                                    \
  static uint8_t flag = 0;     /* new variable on every macro usage */  \
  uint8_t i = 0;                                                      \
                                                                      \
  if( flag ){                  /* check for key release: */           \
    for(;;){                   /* loop ... */                         \
      if( !(port & 1<<pin) ){  /* ... until key pressed or ... */     \
        i = 0;                 /* 0 = bounce */                       \
        break;                                                        \
      }                                                               \
      _delay_us( 98 );         /* * 256 = 25ms */                     \
      if( --i == 0 ){          /* ... until key >25ms released */     \
        flag = 0;              /* clear press flag */                 \
        i = 0;                 /* 0 = key release debounced */        \
        break;                                                        \
      }                                                               \
    }                                                                 \
  }else{                       /* else check for key press: */        \
    for(;;){                   /* loop ... */                         \
      if( (port & 1<<pin) ){   /* ... until key released or ... */    \
        i = 0;                 /* 0 = bounce */                       \
        break;                                                        \
      }                                                               \
      _delay_us( 98 );         /* * 256 = 25ms */                     \
      if( --i == 0 ){          /* ... until key >25ms pressed */      \
        flag = 1;              /* set press flag */                   \
        i = 1;                 /* 1 = key press debounced */          \
        break;                                                        \
      }                                                               \
    }                                                                 \
  }                                                                   \
  i;                           /* return value of Macro */            \
})


int main(void)
{
  bla = 0;
  uint32_t FCPU = 1000000;
  uint32_t Prescaler = 64;
  uint8_t Timesec=1;
  OCR1A = ((FCPU/Prescaler)*Timesec)-1;        // 1 sekunde
  DDRB |= (1<<PB1);
  PORTB |= (1<<PB0);
  PORTD |= (1<<PD7);
  TCCR1B = 0x0B;        // 0b00001011
  TIMSK = 0x10;
  sei();
  lcd_init();
  lcd_string("Counter:");
  while(1)
  {
    if(debounce(PINB, PB0))
    {
      PORTB |= (1<<PB1);
      bla=0;
    }
    if(debounce(PIND, PD7))
    {
      PORTB &= ~(1<<PB1);
      bla=0;
    }
  }
}

ISR(TIMER1_COMPA_vect)
{
  itoa(bla,buffer,10);
  lcd_clear();
  lcd_string("Counter:");
  lcd_string(buffer);
  lcd_string("sek");
  bla++;
}

[Electronics'nStuff (Gast)]

Musst es wohl als volatile deklarieren. Also "volatile uint32_t bla;"

Gruss

[al3ko (Gast)]

Electronics'nStuff schrieb:
> Musst es wohl als volatile deklarieren. Also "volatile uint32_t bla;"
>
> Gruss

grml, funktioniert jetzt wunderbar. :D
danke.

Und hier stehts ja auch:
"Variablen, auf die sowohl innerhalb wie auch außerhalb einer 
Interruptserviceroutine zugegriffen wird (schreibend oder lesend), 
müssen (ähnlich wie Hardwareregister) mit dem Schlüsselwort volatile 
(flüchtig) versehen werden, damit der C-Compiler berücksichtigen kann, 
dass diese Variablen jederzeit (durch das Auftreten des Interrupts) 
gelesen oder geschrieben werden können. Ansonsten würde der C-Compiler 
das regelmäßige Abfragen oder Beschreiben dieser Variablen ggf. 
wegoptimieren, da er nicht damit rechnet, dass auf die Variable auch 
"ohne sein Zutun" zugegriffen wird."
http://www.mikrocontroller.net/articles/Interrupt#Volatile_Variablen

Wieder etwas dazu gelernt.

Gruss