Atmega328p wacht nicht auf, wo steckt der Fehler?

[David (Gast)]

Hallo,
ich möchte meinen MCU alle 2 Sekunden wecken, um ein Sensor kurz zu 
überprüfen und dann wieder schlafen legen lassen. Zusätzlich habe ich in 
der ISR eine Countdown Schaltung programmiert. Mit dem Flag zeige ich an 
wann der MCU alarmieren soll. Leider glaube ich das der MCU gar nicht 
aus dem Schlafmodus aufwacht, denn der Sensor reagiert nicht mehr und 
der Stromverbrauch ist konstant bei 42µA.

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

#define F_CPU 8000000L 

volatile uint8_t flag;

int main(void)
{

 

 
 DDRC |= (1<<PC5) | (1<<PC4); 
 DDRC &= ~(1<<PC3);
 
 
 GTCCR |= (1<<TSM) | (1<<PSRASY);
 ASSR |= (1<<AS2); 
 TCCR2A |= (1<<WGM21); 
 TCCR2B |= (1<<CS22) | (1<<CS21); 
 OCR2A = 256-1; 
 TIMSK2 |= (1<<OCIE2A); 
 GTCCR &= ~(1<<TSM); 
 

 
 sei(); 
 
 
 
 while(1)
    {

   
   set_sleep_mode(SLEEP_MODE_PWR_SAVE);
   sleep_mode();
 
 
   
   
   
   PORTC |= (1<<PC4); 
   _delay_ms(1); 
   
   while(!(PINC &= (1<<PINC3)))
   {
   PORTC |= (1<<PC5); 
   }
   
   PORTC &= ~(1<<PC5); 
   PORTC &= ~(1<<PC4); 
   
   
     
   while(flag==1)
   {
   PORTC|= (1<<PC5); 
   }
   
    }
 return (0);
}




ISR(TIMER2_COMPA_vect)
{
 static uint8_t zweitesekunde;
 static uint8_t minute=0;
 static uint8_t stunde=0;
 static uint8_t tag=0;
 
 zweitesekunde++;
 if(zweitesekunde==30)
 {
 zweitesekunde=0;
 minute++;
 }
    if(minute == 60)
    {
    stunde++;
    minute = 0;
    }
    if(stunde == 24)
    {
    stunde = 0;
 tag++;
 flag=1; 
    }
 
}

[Daniel V. (danvet)]

David schrieb:
>

> 
> ....
>    while(!(PINC &= (1<<PINC3)))
>    {
>    PORTC |= (1<<PC5);
>    }
> 
> .....
>
>    while(flag==1)
>    {
>    PORTC|= (1<<PC5);
>    }
> ....
>

Deine "while()s" würde ich nochmal überdenken.
Sollte vielleicht "if()" heißen?

[David (Gast)]

Das erste würde ich so lassen, weil ich solange der Input PINC3 ungleich 
high ist , die LED einschalten möchte und sie auch solange eingeschaltet 
bleiben soll bis der Input wieder zurück auf HIGH geht.

Beim zweiten hast du recht, da die Zeit überschritten wurde und dort die 
LED nur durch resetten zurückgesetzt werden kann.

[Stefan S. (sschultewolter)]

Solange eines der beiden Abfragen 1 ist, wird dort nichts in den 
Schlafmodus gehen.

Warum dauerhaft die Ports setzen?

  while(!(PINC &= (1<<PINC3)))
   {
   PORTC |= (1<<PC5); 
   }
   
   PORTC &= ~(1<<PC5); 
   PORTC &= ~(1<<PC4); 
   
   
     
   while(flag==1)
   {
   PORTC|= (1<<PC5); 
   }

[Daniel V. (danvet)]

David schrieb:
> Das erste würde ich so lassen, weil ich solange der Input PINC3 ungleich
> high ist , die LED einschalten möchte und sie auch solange eingeschaltet
> bleiben soll bis der Input wieder zurück auf HIGH geht.
>
> Beim zweiten hast du recht, da die Zeit überschritten wurde und dort die
> LED nur durch resetten zurückgesetzt werden kann.

Deine Timing-Vorstellungen verstehe ich nicht.
Erst schläfst du 2 Sekunden, dann willst sofort und gleich auf ein 
Eingangssignal reagieren.
Was wenn das Signal innerhalb des 2 Sekundenschlafs kommt?

Funktioniert denn dein Programm ohne Sleep?

Edit:
hier schon geschaut?:
http://www.mikrocontroller.net/articles/Sleep_Mode

[Thomas E. (thomase)]

David schrieb:
> Zusätzlich habe ich in der ISR eine Countdown Schaltung programmiert.

Sehe ich nicht.

> Mit dem Flag zeige ich an wann der MCU alarmieren soll.

Nach 24 Stunden um 0.00h?

mfg.

[Joachim B. (jar)]

David schrieb:
> set_sleep_mode(SLEEP_MODE_PWR_SAVE);

womit soll er denn geweckt werden?

möglich wäre:
Timer 2
ext. Interrupts
TWI
Watchdog

wenn er schläft läuft er nicht mehr....

[David (Gast)]

Ja das Programm funktioniert ohne Sleep funktion wunderbar es geht mir 
eigentlich auch nur um die Sleep Funktion.
Ich habe das Programm jetzt einfach mal vereinfacht, damit es um das 
wesentliche geht, das aufwachen:

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

#define F_CPU 8000000L

volatile uint8_t flag;


int main(void)
{

  
  //konfigurieren der PINS:

  
  DDRC |= (1<<PC5) | (1<<PC4); //Pin pc5 und pc4 als ausgang definieren
  //DDRC &= ~(1<<PC3); //Pin pc3 als eingang definieren
  
  
  //Start der Initalisierung des Timer 2:
  GTCCR |= (1<<TSM) | (1<<PSRASY); //Timer anhalten, Prescaler Resetten, um die Einschwingzeit des Oszillators zu überbrücken (Einschwingzeit 100ms-1000ms
  ASSR |= (1<<AS2); //Timer Asynchron-Mode einschalten 32768 RTC hängt an XTAL1+XTAL2
  TCCR2A |= (1<<WGM21); // Versetze in Compare Mode
  TCCR2B |= (1<<CS22) | (1<<CS21); //Prescaler auf 256
  OCR2A = 256-1; //(=OCR2A Compare Register) Berechnung der Aufwachzeit (32768/256)^-1*256= Aufwachzeit alle 2 Sekunden wird der MCU aufgeweckt
  TIMSK2 |= (1<<OCIE2A); //Timer 2 Interrupt Enable
  GTCCR &= ~(1<<TSM); //"Timer anhalten Bit" löschen
  

  
  sei(); // Global Interrupt Mode aktiviert
  
  
  
  while(1)
  {
    
    set_sleep_mode(SLEEP_MODE_PWR_SAVE);
    sleep_mode();
    
    
    // Hier wird aufgewacht
    
    if(flag==1)
    {
      PORTC |= (1<<PC5); //LED anschalten
      _delay_ms(100);
      PORTC &= ~(1<<PC5); //LED abschalten
      flag=0;
    }
    

    
  }
  return (0);
}




ISR(TIMER2_COMPA_vect)
{  
  flag=1;
}

[David (Gast)]

Joachim B. schrieb:
> David schrieb:
>> set_sleep_mode(SLEEP_MODE_PWR_SAVE);
>
> womit soll er denn geweckt werden?
>
> möglich wäre:
> Timer 2
> ext. Interrupts
> TWI
> Watchdog
>
> wenn er schläft läuft er nicht mehr....

Deswegen habe ich eigentlich den Timer2 in Compare-Mode programmiert, im 
Power Save Mode sollte dann nur Timer 2 mit einem 32,768khz Quarz laufen 
sobald die 2 Sekunden um sind sollte der Interrupt erfolgen und ihn 
aufwecken, der dann das Hauptorogramm weiter ausführt.

[Joachim B. (jar)]

David schrieb:
> es geht mir
> eigentlich auch nur um die Sleep Funktion.

verstehe ich, nur wenn er schläft schläft er, ich schrieb ja schon die 
Aufwachmöglichkeiten, externer Uhrenquarz am Timer2 32kHz, watchdog, 
oder externer Interrupt.

Ich weiss immer noch nicht wie du den aufwecken willst wenn kein 
watchdaog programmiert ist, kein externer Uhrenquarz dranhängt und kein 
externer Interrupt ausgelöst wird.

[Thomas E. (thomase)]

Joachim B. schrieb:
> kein externer Uhrenquarz dranhängt

???

mfg.

[David (Gast)]

Uhrenquarz hängt dran. Fehlt da jetzt was in der Programmierung? :D 
Sorry ich verstehe dich nicht ganz.

[Thomas E. (thomase)]

David schrieb:
> Uhrenquarz hängt dran. Fehlt da jetzt was in der Programmierung?

Mach mal so:

ISR(TIMER2_COMPA_vect)
{  
  PINC |= (1<<PC5); 
}

Wenn die LED jetzt nicht blinkt, läuft der Timer nicht.

mfg.

[Joachim B. (jar)]

Thomas Eckmann schrieb:
> ???

so ähnlich bei mir auch

also mit sleepmodi hatte ich schon mal das Vergnügen, nur noch nicht mit 
der allgemein beliebten Timer Variante mit dem 32kHz Quarz am extra 
Eingang, nicht dem Systemquarz. Ich las immer davon das der den AVR 
aufwecken kann um alle Skunde die SoftUhr einen weiterlaufen zu lassen. 
Ich bevorzugte die RTC DS3231 die mit ihrem Alarmregister und /INT 
Ausgang am AVR per PCINT weckt, da muss der arme AVR nicht alle Sekunde 
gewckt werden, kann fröhlich pennen bis er wirklich dran ist.

[David (Gast)]

Leider kein blinken :/
Heißt das, das der Uhrenquarz kaputt ist?

[Thomas E. (thomase)]

David schrieb:
> Leider kein blinken :/
> Heißt das, das der Uhrenquarz kaputt ist?

Die Konfiguration des Timers ist OK.

Kontrollier erstmal deine Lötstellen auf Verbindung und Kurzschluss, 
bevor du den Quarz abschreibst. Was für Kondensatoren hast du dran?

mfg.

[David (Gast)]

Das ganze ist auf Breadbord,
die Kondensatoren sind 2 x 27 pF, da ich damals keine 22 pF 
Kondensatoren zur Hand hatte - diese habe ich jetzt aber wieder besorgt, 
soll ich diese mal anbringen?

[Joachim B. (jar)]

David schrieb:
> sollte dann nur Timer 2 mit einem 32,768khz Quarz

hatte ich überlesen

Thomas Eckmann schrieb:
> ???
>
> mfg.

und wurde im Eröffnungbeitrag auch nicht geschrieben ;-)

[Chris (Gast)]

Hast du den Quarz zwischen TOSC 1 und TOSC 2?
Sonst klappt es nicht. Da benötigtst du auch keine extra Kondensatoren.

http://www.atmel.com/Images/Atmel-1259-Real-Time-Clock-RTC-Using-the-Asynchronous-Timer_AP-Note_AVR134.pdf

[Thomas E. (thomase)]

Chris schrieb:
> Hast du den Quarz zwischen TOSC 1 und TOSC 2?
> Sonst klappt es nicht. Da benötigtst du auch keine extra Kondensatoren.

Ja, lass die Kondensatoren ganz weg.
22pF ist sowieso zu gross und 27pF viel zu gross.

mfg.

[David (Gast)]

Chris schrieb:
> Hast du den Quarz zwischen TOSC 1 und TOSC 2?
> Sonst klappt es nicht. Da benötigtst du auch keine extra Kondensatoren.

Ja hab ich.

Der Timer scheint jetzt zu funktionieren. Aber anstatt alle 2 Sekunden 
zu blinken braucht er jetzt fast 10 sekunden?

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

#define F_CPU 8000000L

volatile uint8_t flag;


int main(void)
{

  
  //konfigurieren der PINS:

  
  DDRC |= (1<<PC5) | (1<<PC4); //Pin pc5 und pc4 als ausgang definieren
  //DDRC &= ~(1<<PC3); //Pin pc3 als eingang definieren
  
  
  //Start der Initalisierung des Timer 2:
  GTCCR |= (1<<TSM) | (1<<PSRASY); //Timer anhalten, Prescaler Resetten, um die Einschwingzeit des Oszillators zu überbrücken (Einschwingzeit 100ms-1000ms
  ASSR |= (1<<AS2); //Timer Asynchron-Mode einschalten 32768 RTC hängt an XTAL1+XTAL2
  TCCR2A |= (1<<WGM21); // Versetze in Compare Mode
  TCCR2B |= (1<<CS22) | (1<<CS21); //Prescaler auf 256
  OCR2A = 256-1; //(=OCR2A Compare Register) Berechnung der Aufwachzeit (32768/256)^-1*256= Aufwachzeit alle 2 Sekunden wird der MCU aufgeweckt
  TIMSK2 |= (1<<OCIE2A); //Timer 2 Interrupt Enable
  GTCCR &= ~(1<<TSM); //"Timer anhalten Bit" löschen
  

  
  sei(); // Global Interrupt Mode aktiviert
  
  
  
  while(1)
  {
    _delay_ms(500);
    PORTC &= ~(1<<PC5); //LED abschalten

    
    set_sleep_mode(SLEEP_MODE_PWR_SAVE);
    sleep_mode();
    
    
    // Hier wird aufgewacht
    

    
  }
  return (0);
}




ISR(TIMER2_COMPA_vect)
{  
          PORTC |= (1<<PC5); //LED anschalten
}

[David (Gast)]

Danke es lag tatsächlich an den Kondensatoren!

[Thomas E. (thomase)]

David schrieb:
> Danke es lag tatsächlich an den Kondensatoren!

Eine Stolperfalle hast du noch drin:

http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Die_Timer_und_Z%C3%A4hler_des_AVR#Wenn_er_die_Minute_in_20s_schafft

mfg.