ATMega 2560 Timer Interrupt absetzen

[Jochen (Gast)]

Schönen guten Abend,
ich versuche auf dem AT2560 ein Time-Interrupt auszulösen.
Beim meinem Vorgehen habe ich mich an
http://www.engblaze.com/microcontroller-tutorial-avr-and-arduino-timer-interrupts/
orientiert.
Leider wird das Interrupt nicht abgesetzt (die LEDS in der ISR Routine 
bleiben allesammt aus). Was ist zu ändern?
Schönen dank,
jo

#include <avr/io.h>
#include <avr/interrupt.h>
#define TCCR0 _SFR_IO8(0x33)
#define CS00 0
#define CS01 1
#define CS02 2
#define WGM01 3
#define COM00 4
#define COM01 5
#define WGM00 6

volatile uint8_t count;
int a=1;
int main( void )
{
  
  DDRL=0xff;
  // initialize Timer1
    cli();          // disable global interrupts
    TCCR1A = 0;     // set entire TCCR1A register to 0
    TCCR1B = 0;     // same for TCCR1B
 
    // set compare match register to desired timer count:
    OCR1A = 15624;
    // turn on CTC mode:
    TCCR1B |= (1 << WGM12);
    // Set CS10 and CS12 bits for 1024 prescaler:
    TCCR1B |= (1 << CS10);
    TCCR1B |= (1 << CS12);
    // enable timer compare interrupt:
    TIMSK1 |= (1 << OCIE1A);
    sei();          // enable global interrupts
 
 
while( 1 );
}

ISR(TIMER0_OVF_vect)
{
   //This is the interrupt service routine for TIMER0 OVERFLOW Interrupt.
   //CPU automatically call this when TIMER0 overflows.
   if(a=1){
   PORTL=0b00000111;
   a=0;
   }   else {
     a=1;
     PORTL=0b00000000;
   }
}

[Helfender (Gast)]

Hallo,

ich habe das Programm nur kurz überflogen, aber eines ist mir direkt 
aufgefallen: Du gibst beim Timer1 den Compare-Interrupt frei, aber der 
ISR-Vektor ist der vom Timer0-Overflow-Interrupt. Das passt irgendwie 
nicht zusammen (und dürfte so nicht funktionieren) ...
Verwende den ISR-Vektor vom Compare-Interrupt des Timers 1.

MfG

[Jochen (Gast)]

Helfender, danke für den Tipp. Das ist dann klar, dass der Interrupt 
nicht ausgelöst wird.
Nun leuchten die 3 Leds, d.h. die if-Anweisung

 if(a=1){
   PORTL=0b00000111;
   a=0;
   }

wird offenbar ausgeführt. Jedoch leuchten die LEDs dauerhaft, was ich 
mir nicht erklären kann. Mit jeder 2ten Interrupt Routine sollten sie 
ausgeschaltet werden.
Was ist am Code noch nicht OK?

[Jochen (Gast)]

entweder der Interrupt wird nur einmal ausgelöst, oder an der "Schalt 
Logik" der LEDs ist etwas verkehrt. Nur was?
beste Grüße

[M. K. (avr-frickler)]

Jochen schrieb:
> int a=1;

Jochen schrieb:
> Jedoch leuchten die LEDs dauerhaft, was ich
> mir nicht erklären kann. Mit jeder 2ten Interrupt Routine sollten sie
> ausgeschaltet werden.
> Was ist am Code noch nicht OK?

Stiochwort: volatile
Versuch mal

volatile int a=1

Jochen schrieb:
> #define TCCR0 _SFR_IO8(0x33)
> #define CS00 0
> #define CS01 1
> #define CS02 2
> #define WGM01 3
> #define COM00 4
> #define COM01 5
> #define WGM00 6

Warum die ganzen defines? Die sollten eigentlich schon alle über 
#include <avr/io.h> definiert sein.

[Jochen (Gast)]

Danke! Die LEDs blinken leider immer noch nicht. Was kann das noch sein?

[Thomas E. (thomase)]

Jochen schrieb:
> Danke! Die LEDs blinken leider immer noch nicht. Was kann das noch sein?
Wenn du schon irgendwo abschreibst, dann schreib wenigstens richtig ab:

>And we’ll need to replace our overflow ISR with a compare match version:

>1234 ISR(TIMER1_COMPA_vect) {     digitalWrite(LEDPIN, !digitalRead
>(LEDPIN)); }

Helfender schrieb:
> ich habe das Programm nur kurz überflogen, aber eines ist mir direkt
> aufgefallen: Du gibst beim Timer1 den Compare-Interrupt frei, aber der
> ISR-Vektor ist der vom Timer0-Overflow-Interrupt. Das passt irgendwie
> nicht zusammen (und dürfte so nicht funktionieren) ...
> Verwende den ISR-Vektor vom Compare-Interrupt des Timers 1.
Steht doch alles schon da.

mfg.

[Jochen (Gast)]

Den Interruptvektor habe ich direkt nach dem Hinweis angepasst. Der Code 
gestaltet sich nach den Tipps nun
[

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


volatile uint8_t count;
volatile int a=1;
int main( void )
{
  
  DDRL=0xff;
  // initialize Timer1
    cli();          // disable global interrupts
    TCCR1A = 0;     // set entire TCCR1A register to 0
    TCCR1B = 0;     // same for TCCR1B
 
    // set compare match register to desired timer count:
    OCR1A = 15624;
    // turn on CTC mode:
    TCCR1B |= (1 << WGM12);
    // Set CS10 and CS12 bits for 1024 prescaler:
    TCCR1B |= (1 << CS10);
    TCCR1B |= (1 << CS12);
    // enable timer compare interrupt:
    TIMSK1 |= (1 << OCIE1A);
    sei();          // enable global interrupts
 
 
while( 1 );
}

ISR(TIMER1_COMPA_vect)
{
   //This is the interrupt service routine for TIMER0 OVERFLOW Interrupt.
   //CPU automatically call this when TIMER0 overflows.
    if(a=1){
   PORTL=0b01000111;
   a=0;
   }   else {
     a=1;
     PORTL=0b00000000;
   }
     
   }

Dennoch blinken die LEDs nicht.

[test (Gast)]

Jochen schrieb:
> if(a=1){

Schau dir das mal genau an.

[Thomas E. (thomase)]

Jochen schrieb:
> if(a=1){
Fällt dir da auch was auf?

mfg.

[Helfender (Gast)]

> if(a=1)

Stichwort: Unterschied der Schreibweise bei Vergleich und Zuweisung! 
(C-Grundlagen)

[Jochen (Gast)]

gleich 3 mal ;) Ja klar (bin noch relativ Anfänger ....)
danke an alle Helfer!!