Interrupt wird nur einmal ausgeführt

[Uwe (Gast)]

Hallo,

ich beschäftige mich aktuell mit Interrupts und habe ein Problem.

Ich verwende einen ATMEGA8 mit internen 8MHz.

Ich habe an Pin PD3 einen Taster gehängt (natürlich mit 10k gegen VCC) 
mit dem ich den Pin PD3 auf GND ziehe. Dieser Pin ist der 
Interrupteingang INT1 und dieser soll in der ISR eine Variable 
verändern. Es soll bei jedem Tastendruck die Variable s2 zwischen 1 und 
2 wechseln. Diese Variable wird in meinem Hauptprogramm ausgwertet und 
eine LED an- bzw. ausgeschaltet. Damit das Prellen des Tasters keinen 
Einfluß hat, habe ich in der ISR- Schleife eine Wartezeit definiert (ich 
weiß man sollte in ISRs keine "dummen" Wartezeiten programmieren aber 
für meine Anwendung reicht es aus).

Als Initialwert definiere ich in dem Hauptprogramm s2=2 und somit wird 
die LED eingeschaltet. Das funktioniert auch. Danach betätige ich den 
Taster und somit soll die Variable in der ISR von 2 auf 1 geändert 
werden und die LED soll ausgehen. Auch das funktioniert. ABER: Wenn ich 
jetzt den Taster wieder betätige passiert nichts mehr....

Anbei mein Code:

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

#define F_CPU 8000000L
#include <util/delay.h>
#include <avr/interrupt.h>


volatile int s2;

ISR(INT1_vect)
{
  if(s2==1)
  {
    s2=2;
  }
  
  if(s2==2)
  {
    s2=1;
  }
  _delay_ms(300);
}


int main(void)
{
  DDRB |= (1<<DDB3); 
  DDRD &= ~(1<<DDD3); 
  
  //Interrupts anschalten
  MCUCR |=(1<<ISC11);
  GICR |= (1<<INT1);
  
  s2=2;
  
  
    while(1)
  {
    //GICR |= (1<<INT1)|(1<<INT0);
    sei();
    
    if(s2==1)
    {
      PORTB &= ~(1<<PB3);
    }
  
    if(s2==2)
    {
      PORTB |= (1<<PB3);
    }
  }
}

Danke schon mal und Gruß Uwe

[icke (Gast)]

Naja, schau dir die Interrupt-Routine mal genauer an:

  if(s2==1)
  {
    s2=2;
  }
  
  if(s2==2)
  {
    s2=1;
  }

Im ersten Durchlauf greift das zeite if und die LED geht an.

Beim zweiten Durchlauf (s2==1) greift erst das erste if (s2 =2) und 
danach dann direkt das zweite if, dass s2 wieder auf 1 setzt.

Benutze

else if

antelle des zweiten if

[holgi (Gast)]

Da brauchst du dir nur gedanklich vorzustellen, was in der ISR passiert, 
wenn S2 gleich 1 ist. S2 wird auf 2 gesetzt und danach gleich nochmal 
abgefragt und was passiert dann...?

[steffen (Gast)]

Hallo Uwe,

zum ersten kommt sei(); vor den Mainloop.
Dann musst du dem MC noch sagen welche Richtung der Pin machen soll:

DDRB &= ~((1 << DDB3)); //Taster 
PORTB |= (1<<PB3); //Pullup ein

und zum dritten, du benutzt den INT1 als Ausgang wenn das ein Taster 
sein soll dann muss da ein Taster dran. Also am besten deinen Ausgang wo 
anders hin legen

Gruß Steffen

[Uwe (Gast)]

Oh Mann, wie dumm kann man sein???

Vielen Dank für diesen völlig blöden Fehler.

Danke und Gruß Uwe