ATmega8 Interrupt wird zwei Mal ausgeführt

[Michi R. (snowsaw)]

Hallo

In der Schlaufe legt sich der uC gleich schlafen.
Mit INT0 wird er wieder geweckt.
Nun läuft der Code in der ISR jedoch zwei mal ab. Warum?

/*
  IC: ATmega8
 */ 

#define F_CPU 3686400UL
#include <avr/io.h>
#include <util/delay.h>
#include <avr/sleep.h>
#include <avr/interrupt.h>

// global variable to count the number of overflows
volatile uint8_t tot_overflow;

void goSleep()
{
    set_sleep_mode(SLEEP_MODE_PWR_DOWN);
    GICR |= (1 << INT0); // enable external interrupt
    sleep_enable();
    sleep_cpu();
    sleep_mode();
    sleep_disable();
    GICR &= ~(1 << INT0); // disable external interrupt here, in case the external low pulse is too long
}

int main(void)
{
  DDRB |= (1 << PB0);   // Setzen Output
  DDRB |= (1 << PB1);   // Setzen Output
  DDRB |= (1 << PB2);   // Setzen Output
  PORTC |= (1 << PC1);   // Setzen PulUp Eingang
  PORTD |= (1 << PD2);   // Setzen PulUp Eingang
  
  MCUCR &= ~((1 << ISC01) | (1 << ISC00)); // INT0 fallende Flake
  sei();
  
  while(1){
    PORTB |= (1 << 1);  // led ein
    goSleep();  // schlafen
  }  
}

ISR(INT0_vect)
{
  PORTB &= ~(1 << 1);  // led aus
  PORTB &= ~(1 << 2);  // led aus
  _delay_ms(200);
  PORTB |= (1 << 2);  // led ein
  _delay_ms(200);
  PORTB &= ~(1 << 2);  // led aus
  _delay_ms(200);
  PORTB |= (1 << 2);  // led ein
  _delay_ms(200);
  PORTB &= ~(1 << 2);  // led aus
}

Danke

[Max D. (max_d)]

Ich bin zu faul ins DB zu gucken welche der beiden Versionen zutrifft:
1. Dein Kommentar zum Interruptmodus stimmt, dann setzt Kontaktprellen 
dir den flag gleich nach dem Eintritt nochmal
2. Dein Kommentar in gosleep stimmt, dann springt er dir da vor dem 
abschalten des interrupts raus und führt die isr aus bis der low weg ist

[Michi R. (snowsaw)]

Was könnte man im DB nachschauen?

[Max D. (max_d)]

Michi R. schrieb:
> MCUCR &= ~((1 << ISC01) | (1 << ISC00)); // INT0 fallende Flake

impliziert Flankenauslösung (einmalig wenn Pin 1->0 geht)

Michi R. schrieb:
> GICR &= ~(1 << INT0); // disable external interrupt here, in case the
> external low pulse is too long

impliziert Pegelinterrupt (solange Pin = 0 wird getriggert)

Das Datenblatt sagt dann:
ISC00 = 0; ISC01 = 0; The low level of INT0 generates an interrupt 
request;

Ergo ist dein Kommentar neben der Zuweisung schlicht FALSCH.
Solange dein Pin low ist feuert der Interrupt ununterbrochen.

Falls du den Pegelinterrupt behalten willst, dann schalte die 
Interruptbehandlung schon in der ISR aus (aber vor dem Schlafengehen 
wieder an natürlich).

[Michi R. (snowsaw)]

So geht es...

Nach dem Aufwachen wird der Interrupt nur einmal ausgeführt.

Mir ist noch nicht so ganz klar warum. Auf jeden Fall wird cli() erst in 
den darauffolgenden Tackten ausgeführt und die Interrupts sind während 
des Schlafens aktiv.

void goSleep()
{
  set_sleep_mode(SLEEP_MODE_PWR_DOWN);
  cli();
  sleep_enable();
  sei();
  sleep_cpu();
  sleep_disable();
}

ISR(INT0_vect)
{
  GICR &= ~(1 << INT0); // disable external interrupt here, in case the external low pulse is too long
  GIFR |= (1 << INTF0);  // int0 reset
  GICR |= (1 << INT0); // enable external interrupt
}

[MWS (Gast)]

Michi R. schrieb:
> GICR &= ~(1 << INT0);
> ...
> GICR |= (1 << INT0);

Ist Quatsch, in der ISR sind Interrupts global gesperrt und zum Löschen 
eines Int-Flags muss man den entsprechenden Interrupt auch nicht 
ausschalten.

Nur das Löschen des Int-Flags macht Sinn um ein weiteres eingetretenes 
Triggerereignis zu löschen.