Atmega48 wacht nicht durch INT1 auf (sleep_mode(), interrupt)

[Info (Gast)]

Folgender Code geht erwartungsgemäß in den Power-Down Modus, lässt sich 
dann aber durch einen low-Pegel an PD3 nicht mehr "wecken".

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

int main(void)
{
  wdt_disable();
  ACSR = (1<<ACD);
  PORTD = 0xFF; 
  DDRD  = 0xFF; 
  TCCR0A =   0;
  TCCR0B = (1<<CS01);
  OCR0A  =  124;
  TIMSK0 = (1<<OCIE0A);
  TCCR1A =   0;
  TCCR1B =  (1<<CS12) | (1<<CS10);
  OCR0A  =   65535;
  TIMSK1 = (1<<OCIE1A);
  LEDpattern = 0;
  sei();
  while(1)
  {
    LEDpattern++;
    set_sleep_mode(SLEEP_MODE_IDLE);
    sleep_mode();
  }
}

ISR( TIMER0_COMPA_vect ) 
{
  OCR1A += 124;
  PORTD = LEDpattern;
}

ISR( TIMER1_COMPA_vect ) 
{
  PORTD  = 0xFF;
  DDRD  &= ~(1<<PD3);
  EICRA = 0;
  EIFR = 0;
  EIMSK = (1<<INT1);
  set_sleep_mode(SLEEP_MODE_PWR_DOWN);
  sleep_mode();
}

ISR( INT1_vect ) 
{
  EIMSK = 0;
  DDRD  |= (1<<PD3);
  PORTD  = 0x00;
}

Es ist vermutlich der Aufruf von sleep_mode() im interrupt - wieso?

http://www.nongnu.org/avr-libc/user-manual/group__avr__sleep.html
http://www.atmel.com/images/Atmel-8271-8-bit-AVR-Microcontroller-ATmega48A-48PA-88A-88PA-168A-168PA-328-328P_datasheet_Complete.pdf

[Denis (Gast)]

Überleg mal was als erstes, beim Anspringen der Interrupt routine 
passiert,
dann weist du auch warum bei dir keine weiteren Interrupte auftreten ...

[Pandur S. (jetztnicht)]

Aaaaahhhhhhh....

ein sleep() in einem timer interrupt ....

[Info (Gast)]

Denis schrieb:
> was als erstes, beim Anspringen der Interrupt routine
> passiert,

Meinst du einen spezifischen? In beiden schalte ich den Interrupt aus, 
den Pin um, und den anderen Interrupt ein. Wo genau ist das Problem?

Jetzt N. schrieb:
> Aaaaahhhhhhh....
>
> ein sleep() in einem timer interrupt ....

Genau das ist die Frage: ist das das Problem, und wieso? Du musst es mir 
ja nicht erklären, aber hilfreicher wäre ein Verweis!

[Peter D. (peda)]

Der AVR nimmt zum Aufwachen die Interruptlogik.
D.h. bei globaler Interruptsperre kann er nicht aufwachen.

[Info (Gast)]

Aha, danke! Schon lange nichts mehr mit AVR gemacht (Gruß in den 
Wedding).

7.7 Reset and Interrupt Handling
...
When an interrupt occurs, the Global Interrupt Enable I-bit is cleared and all interrupts are disabled. The user 
software can write logic one to the I-bit to enable nested interrupts. All enabled interrupts can then interrupt the 
current interrupt routine. The I-bit is automatically set when a Return from Interrupt instruction – RETI – is 
executed. 

...

if one or more interrupt conditions occur while the Global 
Interrupt Enable bit is cleared, the corresponding Interrupt Flag(s) will be set and remembered until the Global 
Interrupt Enable bit is set, and will then be executed by order of priority. 

..

When the AVR exits from an interrupt, it will always return to the main program and execute one more 
instruction before any pending interrupt is served.

[Pandur S. (jetztnicht)]

Ein interrupt muss immer sofrt zu Ende ausgefuehrt werden. Ein sleep 
kommt immer, ausschliesslich, und genau einmal in den Endlosloop des 
main()

[Peter D. (peda)]

Du kannst ja nur das "set_sleep_mode(SLEEP_MODE_PWR_DOWN);" im Interrupt 
lassen, darfst es dann aber in der Mainloop nicht sofort wieder 
rücksetzen.

Das "sleep_mode();" sollte man generell nicht verwenden, es bewirkt eine 
Race Condition (siehe Erläuterung in <sleep.h>).

[Info (Gast)]

OK, danke.

Habe zwischenzeitlich die set_sleep_mode() in die jeweiligen ISRs 
gesetzt und in der Endlosschleife (noch) das sleep_mode().

    As the \c sleep_mode() macro might cause race conditions in some
    situations, the individual steps of manipulating the sleep enable
    (SE) bit, and actually issuing the \c SLEEP instruction, are provided
    in the macros \c sleep_enable(), \c sleep_disable(), and
    \c sleep_cpu().  This also allows for test-and-sleep scenarios that
    take care of not missing the interrupt that will awake the device
    from sleep.

    Example:
    \code
    #include <avr/interrupt.h>
    #include <avr/sleep.h>

    ...
      set_sleep_mode(<mode>);
      cli();
      if (some_condition)
      {
        sleep_enable();
        sei();
        sleep_cpu();
        sleep_disable();
      }
      sei();
    \endcode

    This sequence ensures an atomic test of \c some_condition with
    interrupts being disabled.  If the condition is met, sleep mode
    will be prepared, and the \c SLEEP instruction will be scheduled
    immediately after an \c SEI instruction.  As the intruction right
    after the \c SEI is guaranteed to be executed before an interrupt
    could trigger, it is sure the device will really be put to sleep.