Timerinterrupt beim 644PA

[R. B. (britzi)]

Hallo zusammen,

ich möchte dass meine Steuerung, sofern sie den Kontakt zum Computer 
verliert in einen sicheren Zustand geht.

Um dies zu erreichen möchte ich den Timer2 des ATmega 644PA nutzen.
Der Computer soll dabei per UART spätestens alle 30 Sekunden eine 
Variable zurücksetzen müssen, damit das System nicht in den sicheren 
Zustand geht.
Die UART-Funktionen funktionieren schon alle.

Mit fehlt nur noch der Timer.
Die anderen Timer sind schon für PWM in Betrieb, daher muss ich leider 
mit dem 8-bit Timer vorlieb nehmen.

Dieser ist, sofern ich das Datenblatt richtig verstehe sowieso immer am 
laufen und zählt immer die 8 bit voll und beginnt dann von vorne.
Ich möchte einfach bei jedem Überlauf eine Zählvariable incrementieren, 
die dann, wenn sie einen bestimmten Wert erreicht, meine maximale 
akzeptable Dauer ohne Signal vom PC darstellt.

Das hier ist mein Codeversuch:

        uint8_t  sicherheitsflag=0;
 volatile   uint16_t  zeit=0;
       uint8_t  systemAus=0;

void init_saeftyTimer()
{
  TCCR2B |= (1 << CS22)|(1 << CS21)|(1 << CS20); // 18,4320MHz / 1024 = 18kHz      
  //Überlauf nach: 0,0142857143 Sekunden
  //30 Sekunden = 2100 Überläufe  
  // Interrupt TOV2
}

ISR(TIMER2_OVF_vect_num)  // Interrupt-Service-Routine
{
  zeit++;
}

void saeftyTimer()
{
  if (sicherheitsflag==1)
  {
    zeit=0;
    sicherheitsflag=0;
  }
  if (zeit>=2100)
  {
    systemAus=1;
    zeit=0;
  }
}

Allerdings meckert er bei der ISR:
"expected declaration specifiers or '...' before numeric constant"

Was ist mein Fehler?
Und ist der Ansatz sonst so ok?

Vielen Dank :-)

[Jörg W. (dl8dtl) (Moderator)]

R. B. schrieb:

> Die anderen Timer sind schon für PWM in Betrieb, daher muss ich leider
> mit dem 8-bit Timer vorlieb nehmen.

Nein.  Auch ein PWM-Timer kann durchaus normale Timerinterrupts
machen, insbesondere kann er noch einen Interrupt beim Erreichen
des maximalen Zählerstands triggern.  Der stört die PWM überhaupt
nicht, und für sowas Grobschlächtiges wie deine 30 s reicht das
wohl allemal.

> ISR(TIMER2_OVF_vect_num)

Du hast den Unterschied zwischen TIMER2_OVF_vect_num und TIMER2_OVF_vect
nicht verstanden.

[R. B. (britzi)]

Jörg Wunsch schrieb:

> Nein.  Auch ein PWM-Timer kann durchaus normale Timerinterrupts
> machen, insbesondere kann er noch einen Interrupt beim Erreichen
> des maximalen Zählerstands triggern.  Der stört die PWM überhaupt
> nicht, und für sowas Grobschlächtiges wie deine 30 s reicht das
> wohl allemal.

Das ergibt Sinn :-)


> Du hast den Unterschied zwischen TIMER2_OVF_vect_num und TIMER2_OVF_vect
> nicht verstanden.

Ganz ehrlich... ich kenne den Unterschied leider nicht und habe auch im 
Datenblatt nichts dazu gefunden.

Ich hatte da auch erst TIMER2_OVF_vect stehen, aber da gibts dann 
zumindest auch Warnungen:

return type defaults to 'int' [enabled by default]
type of '__vector_11' defaults to 'int' [enabled by default]
control reaches end of non-void function [-Wreturn-type]

Liegt das daran, dass ich ein uint16 genommen habe, der ja ein 
spezieller Integer ist?

[Thomas E. (thomase)]

R. B. schrieb:
> return type defaults to 'int' [enabled by default]
Du hast interrupt.h nicht eingebunden. Dann weiss der Compiler nicht, 
was das ist.

ISR(TIMER2_OVF_vect) ist der richtige Aufruf.

mfg.

[R. B. (britzi)]

Thomas Eckmann schrieb:
> R. B. schrieb:
>> return type defaults to 'int' [enabled by default]
> Du hast interrupt.h nicht eingebunden. Dann weiss der Compiler nicht,
> was das ist.
>
> ISR(TIMER2_OVF_vect) ist der richtige Aufruf.
>
> mfg.

Danke :-)

Was für ein blöder Fehler, nun funktioniert es :-)

[R. B. (britzi)]

Es funktioniert leider doch nicht :-(

Der Code erzeugt zwar keine Fehler, aber der Timer läuft anscheinend 
nicht.

Zumindest wird die ISR nie aufgerufen.

Ich probiere hier schon seit Stunden rum, aber es läuft nicht.

Woran liegt es?

Ich finde das Datenblatt in Bezug auf die Timer leider leicht verwirrend 
:-(

Es ist doch so, dass der Timer immer läuft, oder nicht?
So verstehe ich zumindest das Datenblatt...
Ich habe ja nur einen Prescaler konfiguriert.


Vielen Dank :-)

[Falk B. (falk)]

@ R. B. (britzi)

>Der Code erzeugt zwar keine Fehler, aber der Timer läuft anscheinend
>nicht.

>Zumindest wird die ISR nie aufgerufen.

>Ich probiere hier schon seit Stunden rum, aber es läuft nicht.

>Woran liegt es?

Posten deinen aktuellen Code als Anhang.

>Es ist doch so, dass der Timer immer läuft, oder nicht?

Nö, man kann ihn auch anhalten.
Oder die Interrupts deaktivieren.

[R. B. (britzi)]

Der Code ist noch länger.

Das hier ist ein Minimalbeispiel:

#include <avr/io.h>
#include "DAC_SPI.h"
#include <avr/interrupt.h>
#include "globale.h"


// Defines für Gruene Statuslampe
#define  ledgruen_aus    PORTB |=(1<<PB1)
#define  ledgruen_an    PORTB &= ~(1<<PB1)

uint8_t sicherheitsflag;
volatile uint16_t saeftyzeit;

void init_saeftyTimer()
{
  TCCR2B |= (1 << CS22)|(1 << CS21)|(1 << CS20); //18,4320MHz /1024 = 18 kHz
  // Überlauf nach 0,0142857143 Sekunden
  // 30 Sekunden = 2100 Überläufe
  DDRB |= (1<<DDB1); //Gruene Statuslampe
  sicherheitsflag=0;
  saeftyzeit=0;
  ledgruen_an;
}
  
ISR(TIMER2_OVF_vect)
{
  saeftyzeit++;
  put_string0("ISR"); //Um über UART zu sehen, ob die ISR aufgerufen wird
  put_string1("ISR"); //Zweiter UART
}
  
void saeftyTimer()
{
  if (sicherheitsflag>=1) //Rücksetzen durch PC
  {
    saeftyzeit=0;
    sicherheitsflag=0;
  }
  if (saeftyzeit>=210) // Wenn 30s rum ausmachen! //Zeit bewusst auf 3s zum Testen gesetzt
  {
    write_to_dac0(0,0);  //System ausschalten
    write_to_dac0(1,0);
    saeftyzeit=0;
    ledgruen_aus;
  }
}

int main ()
{
   init_saeftyTimer(); //LED wird Grün, aber geht nie aus :-(
   sei();
  
   while(1)
    {
     saeftyTimer();
    }
}

Vielen Dank :-)

[Falk B. (falk)]

@ R. B. (britzi)


#include <avr/io.h>
#include "DAC_SPI.h"
#include <avr/interrupt.h>
#include "globale.h"



>void init_saeftyTimer()
>{
>  TCCR2B |= (1 << CS22)|(1 << CS21)|(1 << CS20); //18,4320MHz /1024 = 18 >kHz
>  // Überlauf nach 0,0142857143 Sekunden
>  // 30 Sekunden = 2100 Überläufe
>  DDRB |= (1<<DDB1); //Gruene Statuslampe
>  sicherheitsflag=0;
>  saeftyzeit=0;
>  ledgruen_an;
>}

Wo wird der TImer overflow freigeschaltet?
Das fehlt

TIMSK2 |= (1<<TOIE2);

[R. B. (britzi)]

Falk Brunner schrieb:

> Wo wird der TImer overflow freigeschaltet?
> Das fehlt
>
> TIMSK2 |= (1<<TOIE2);

Super!

Vielen vielen Dank :-)
Das hab ich einfach im Datenblatt übersehen.

Ich habs auch gleich nachgeschlagen. ;-)


Schönes Wochenende :-)