Atemga8 (Interrupt), Programm springt nicht zurück

[Abi (Gast)]

Hallo zusammen,

ich hoffe, ihr könnt mir helfen.

Es geht um ein Programm für den Asuro.
es soll die status-LED kurz ausschalten, wenn man den Taster drückt. Das 
Problem ist, dass das Programm wahrscheinlich nicht zur While-schleife 
zurückspringt, um die LED dann wieder auf grün zu schalten.

Ich lasse die LED mit Absicht in der While-schleife auf grün setzen, um 
herauszufinden, ob das Programm nach dem Interrupt zur Schleife 
zurückspringt oder nicht.

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

void init (void)
{
  // Datenrichtung der I/O-Ports festlegen.

  DDRB |= (1 << PB0);

       //enable  all interrupts

  GICR |= (1 << INT1);
  SREG |= (1 << 7);

  // Interrupt wird bei fallender Flanke ausgelöst

  MCUCR |=  (1 << ISC11);
  MCUCR &= ~(1 << ISC10);
  sei();
}

SIGNAL (INT1_vect)
{
    PORTB &= ~(1 << PB0); // Grüne LED aus
    reti();
}

int main(void)
{
  init();
  while(1)
  {
    PORTB |= (1 << PB0); // Grüne LED an
  }
  return 0;
}


freundliche Grüße

Abi

[Karl H. (kbuchegg)]

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

 
void init (void)
{
  // Datenrichtung der I/O-Ports festlegen. 
  DDRB |= (1 << PB0);
 
  //enable  all interrupts
  GICR |= (1 << INT1);

  // Interrupt wird bei fallender Flanke ausgelöst

  MCUCR |=  (1 << ISC11);
  MCUCR &= ~(1 << ISC10);
}

ISR(INT1_vect)
{
  PORTB &= ~(1 << PB0); // Grüne LED aus
}
 
int main(void)
{
  init();

  sei();

  while(1)
  {
    PORTB |= (1 << PB0); // Grüne LED an
  }

  return 0;
}

du willst keinen reti() selber aufrufen (das macht sowieso der Compiler 
für dich). Du willst auf das SREG nicht selber zugreifen (das macht der 
sei()). Du willst den sei() dezidiert vor der while Schleife haben, 
damit du ihn siehst (und nicht in irgendwelchen obskuren Funktionen 
verstecken), du willst SIGNAL nicht mehr benutzen (sondern ISR, denn 
dann passt auch der Name der ISR zur Verwendung)

[Abi (Gast)]

Das Problem war wegen reti(). Wenn sie da ist, dann reagiert das 
Programm beim zweiten Tastendrücken nicht.
ob ich auf SREG zugreife oder nicht und ob ISR oder SIGNAL, das is egal.
hier ist das richtige Programm, was funktioniert, es soll die LED-Farbe 
zwischen Grün und Rot schalten, wenn man den Taster drückt.

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

void init (void)
{
  // Datenrichtung der I/O-Ports festlegen. 
  DDRB |= (1 << PB0);
  DDRD |= (1 << PD2);

  PORTB |= (1 << PB0); // Grüne LED an
  PORTD &= ~(1 << PD2); // Rote LED aus
 
  //enable  all interrupts
  GICR |= (1 << INT1);
  SREG |= (1 << 7);

  // Interrupt wird bei fallender Flanke ausgelöst

  MCUCR |=  (1 << ISC11);
  MCUCR &= ~(1 << ISC10);
}

SIGNAL(INT1_vect)
{
 if (PINB & (1 << PB0)){  // Grün  an ?
  PORTB &= ~(1 << PB0); // Grün aus.
  PORTD |= (1 << PD2);  // Rot an.
  
   }else if (PIND & (1 << PD2)){ // Rot an ?
      PORTD &= ~(1 << PD2); // Rot aus.
      PORTB |= (1 << PB0); // Grün an.
   }
}

int main(void)
{
  init();

  sei();

  while(1)
  {    
  }
  return 0;
}

[Timmo H. (masterfx)]

Dennoch solltest du nicht mehr SIGNAL benutzen. Momentan geht noch 
beides aber irgendwann ist SIGNAL weg. Und wenn du unbedingt SIGNAL 
benutzen musst, dann würde der Vector richtigerweise "SIG_INTERRUPT1" 
heissen

Also entweder

SIGNAL(SIG_INTERRUPT1)
...

oder

ISR(INT1_vect)
...

Aber wie gesagt, letzteres ist vorzuziehen weil SIGNAL "deprecated" also 
veraltet ist.