Interrupt kommt zu schnell

[Icey (Gast)]

Hallo,

Ich habe gerade erst angefangen mit Mikrocontroller-Programmierung. Ich 
möchte über den Timer0 eine LED zum blinken bringen, dazu habe ich 
folgendes Programm geschrieben.

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

volatile char i=0;


ISR(TIMER0_COMP_vect)
{
  i++;
  PORTB ^= (1<<PB7);
}

int main(void)
{
  DDRB = 0xff;  //PortB als Ausgang
  DDRD = 0x00;  //PortD als Eingang
  TCCR0 = 0x0D;  //CTC-Modus Prescaler 1024
  OCR0 = 0x7F;  //Zählerwert eingestellt
  TIMSK = 0x01;  //Interrupt ermöglichen
  sei();
  
  
  PORTB = 0xff;  //LED´s aus
  
  while(1)
  {
    if(i>4)
    {
      i=0;
      PORTB ^= (1<<PB3);
    }
  }
}

Das ganze läuft mit einem ATmega8515 und einem 4MHz Quarz auf einem 
STK500. Leider läuft blinken die Lampen aber zu schnell, die 4.LED 
ungefähr 12mal in 10 Sekunden.

Irgendwie komm ich nicht so wirklich auf die Lösung warum der sowas mit 
mir macht :(

Ich benutzte WinAVR-20070525.

Ich hoffe Ihr könnt mir helfen.

Gruß,
Icey

[Andreas K. (a-k)]

Mal rechnen: Alle 128*1024 Takte ein Interrupt, danach Software-Teiler 
10: 128*1024*10 = ca. 1,3Mio. Passt doch prima, wenn man mal annimmt, 
dass du vergessen hast, den Mega vom internen 1MHz Takt auf den Quarz 
umzustellen.

Bloss: Wenn er dann mal mit 4MHz läuft: Langsamer wird dadurch nicht 
blinken. Eher im Gegenteil. Wie ist denn deine Rechnung?

[Icey (Gast)]

Moin,

Mist ich bin doof kopfaufdentischknall

Ich hab mich mit den Einheiten verwurstetelt, ich hab statt ns mit ms 
gerechnet, da kann ja nur das falsche rauskommen.

Danke für die schnelle Hilfe :)

Gruß,
Icey

[Simon K. (simon)]

Kleiner Tip: Es ist lesbarer, wenn du statt

TCCR0 = 0x0D;

eher sowas schreibst:

TCCR0 = (1<<WGM01) | (1<<CS02) | (1<<CS00);

[Bonzo (Gast)]

Kleiner Tip. Es ist noch lesbarer :

TCCR0 = 0x0D;  // WGM01 CS02 CS00

[Simon K. (simon)]

Bonzo wrote:
> Kleiner Tip. Es ist noch lesbarer :
>
> TCCR0 = 0x0D;  // WGM01 CS02 CS00

Ja, aber man muss immer das Comment anpassen. Bei meiner Variante 
kümmert sich dann der Compiler um das Ausrechen des korrekten 
Registerwerts... Das minimiert auch Fehlerquellen.

[Rolf Magnus (Gast)]

Nö. Da muß man ja doch wieder ins Datenblatt schauen und die Bits 
auseinanderklamüsern, um zu rauszubekommen, ob der Wert denn nun stimmt 
oder nicht.  Abgesehen davon ist Code, der sich selbst kommentiert, 
immer besser als unverständlicher Code, den man erst mit einem Kommentar 
erklären muß.

Am besten:

TCCR0 = _BV(WGM01) | _BV(CS02) | _BV(CS00);

[Icey (Gast)]

Hallo,

Danke für die Tipps. Kommentieren kann man nie genug ;)
Werd wohl ne Mischung aus beiden Varianten machen :)

Gruß,
Icey

der sich jetzt mal am UART versucht :D

[Andreas K. (a-k)]

> Werd wohl ne Mischung aus beiden Varianten machen :)

Sieht man öfter. Falscher Code und richtiger Kommentar daneben, oder 
umgekehrt. ;-)