Sleep Mode + Pin Change Interrupt

[Tommi (Gast)]

Hallo,

ich stehe hier gerade total auf dem Schlauch und hoffe ihr könnt mir 
weiterhelfen.

Und zwar verwende ich einen atmega48 mit angeschlossenem Taster an PB5. 
Der Taster wird mit der "Comfortroutine" von Peter Danegger hier von der 
Seite entprellt/ausgewertet, was soweit auch wunderbar funktioniert.

Nun möchte ich den Controller bei langem Tastendruck (get_key_long) in 
den Sleep-Modus versetzen und beim nächsten Mal Drücken per 
Pin-Change-Interrupt wieder ins Leben zurück holen.

Jetzt besteht das Problem, dass der Controller zwar wie gewünscht in den 
Sleep-Modus versetzt wird, allerdings sofort beim loslassen der Taste 
wieder geweckt wird, da der Pin-Change-Interrupt auf beiden Flanken 
ausgelöst wird. Hoffe ihr versteht was ich meine.


Wie kann ich das in meinem Programm verhindern? Probiere jetzt schon 
ewig daran rum aber bekomme es nicht wie gewünscht zum laufen :(


Gruß

[Peter II (Gast)]

Tommi schrieb:
> Wie kann ich das in meinem Programm verhindern? Probiere jetzt schon
> ewig daran rum aber bekomme es nicht wie gewünscht zum laufen :(

erst schlagen gehen, wenn Taste losgelassen wird.

[Thomas E. (thomase)]

Peter II schrieb:
> g

f

[Tommi (Gast)]

Ja das hatte ich mir auch schon überlegt, scheint allerdings mit der 
Routine von Peter Dannegger nicht so einfach zu funktionieren, oder wie 
kann ich das realisieren?

Wollte den MC zuerst beim Loslass-Interrupt schlafen legen, allerdings 
lässt er sich dann nicht mehr aufwecken.

[Tommi (Gast)]

#define F_CPU 8e6/8

#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/sleep.h>
#include "debounce_2.h" // Peter Dannegger Routine
  
typedef struct {      // Flag Register
  unsigned sleep:1;
} flagbytes;
volatile flagbytes flag;

int main(void)
{            
  //Sleep Mode und Pin Change zum Aufwecken
  PCICR |= (1<<PCIE0);
  PCMSK0 |=(1<<PCINT5);
    set_sleep_mode(SLEEP_MODE_PWR_DOWN);
  
  sei();  
            
    while(1)
    {  
    if(get_key_long(1<<KEY1))
    {
      flag.sleep=1;
    }
  }
}

ISR(PCINT0_vect)
{  
  if(flag.sleep)
  {
    flag.sleep=0;
    sleep_mode();
  }
}

Habe es jetzt mal so probiert, er geht jetzt beim Taster Loslassen zwar 
in den Schlafmodus, allerdings lässt er sich so nicht mehr aufwecken. 
Warum nicht? Bin Ratlos...

[uwe (Gast)]

Dann leg ihn verzögert schlafen (nachdem die Taste losgelassen wurde und 
etliche Millisekunden verstrichen sind) mit Zählvariablen oder Timer...

[Thomas E. (thomase)]

Tommi schrieb:
> Warum nicht? Bin Ratlos...

Der Sleep-Aufruf darf nicht in einer ISR erfolgen.

[c-hater (Gast)]

Thomas E. schrieb:

> Der Sleep-Aufruf darf nicht in einer ISR erfolgen.

Das ist nicht ganz korrekt. Er darf schon in einer ISR erfolgen, aber 
nicht in exklusiv abgearbeiteten Code. Beim AVR8 ist das nicht 
synonym, da man jederzeit in einer ISR "sei" ausführen kann, womit man 
zwar nach wie vor innerhalb einer ISR ist, aber nicht mehr in nicht 
unterbrechbarem Code.

[Tommi (Gast)]

Thomas E. schrieb:
> Tommi schrieb:
>> Warum nicht? Bin Ratlos...
>
> Der Sleep-Aufruf darf nicht in einer ISR erfolgen.

Danke! Jetzt geht es.

Hier nochmal der Code:

    if(get_key_long(1<<KEY1))
    {
      sleep_state=1; //Schlafen legen
      PCICR |= (1<<PCIE0);
      PCIFR |= (1<<PCIF0);            
    }
    if(sleep_state==2)
    {
      sleep_state++;
      PCICR |= (1<<PCIE0);
      PCIFR |= (1<<PCIF0);      
      PORTD=0xFF;
      sleep_mode();
    }


ISR(PCINT0_vect)
{
  if(sleep_state==1)
  {
    // Im Hauptprogramm schlafen legen
    sleep_state++; 
  }
  else if(sleep_state==3)
  {
    // Grade aufgewacht
    set_display_number(bpm);
    wait_cnt=1;
  }  
  PCICR &=~ (1<<PCIE0);    
  PCIFR |= (1<<PCIF0);
}

[Peter D. (peda)]

Beitrag "Re: AVR Sleep Mode / Knight Rider"