Timerinterrupt C++ wird nicht ausgelöst

[Franz (Gast)]

Hallo,
ich bekomme es leider immer noch nicht vernünftig hin, Interruptroutinen 
für einen Atmega in C++ zu implementieren.
Jetzt wollte ich die Objekte Global machen und in den ISRs dann die 
entsprechende Funktion aufrufen.

#include "system.h"
#include <util/delay.h>
#include <avr/io.h>
#include "Timer/timer.h"

timer timer0(TCCR0A,
             ~(1 << WGM00) | (1 << WGM01),
             TCCR0B,
             (1 << CS00) | (1 << CS01) & ~(1 << CS02),
             OCR0A,
             OCR0A = 250 - 1,
             TIMSK0,
             (1 << OCIE0A));

int main(void)
{
  while(1)
  {
    if(timer0.getNewTime() == true){
    ....
    }
  }
}

void TIMER0_COMPA_vect(void)
{    
    timer0.interrupt();
}

in timer.cpp

timer::timer(volatile uint8_t &tccra, uint8_t tccraNewState,
             volatile uint8_t &tccrb, uint8_t tccrbNewState,
             volatile uint8_t &ocra, uint8_t ocraNewState,
             volatile uint8_t &timsk, uint8_t timskNewState)
{
    tccra |= tccraNewState;
    tccrb |= tccrbNewState;
    ocra |= ocraNewState;
    timsk |= timskNewState;
}

timer::~timer()
{
}

bool timer::getNewTime()
{
    return _newTime;
}
void timer::interrupt()
{
    _milliseconds++;
    if(_milliseconds >= 500)
    {
        _milliseconds = 0;
        _seconds++;
        PORTB ^= ( 1 << PORTB5 );
        //_newTime = true;
    }
}

Zum Testen möchte ich zunächst wieder die LED and PB5 blinken lassen. 
Aber diese bleibt aus. Auch in der Simulation wird nicht in die ISR 
gesprungen. Kann mir da jemand evtl. sagen, wieso die ISR nicht 
aufgerufen wird?
In der Simulation wird der Brakepoint an der Stelle

if(timer0.getNewTime() == true){

auch nie erreicht. Vorher wird alles Resettet und von vorne gestartet.

Gruß
Franz

[hp-freund (Gast)]

Habe lange nichts mit Atmel gemacht, aber braucht das nicht ein:

sei();

[Stefan E. (sternst)]

avr/interrupt.h einbinden, und das ISR()-Makro verwenden.

[Matthias H. (hallamen)]

Bitte ganzen Code posten:

#include "system.h"
#include "Timer/timer.h"

[Franz (Gast)]

hp-freund schrieb:
> sei();

natürlich, das habe ich ausversehen mit rausgelöscht. habe ich aber vor 
der while-schleife stehen.

Stefan E. schrieb:
> avr/interrupt.h einbinden, und das ISR()-Makro verwenden.

habe ich geändert

ISR(TIMER0_COMPA_vect)
{    
    timer0.interrupt();
}

Matthias H. schrieb:
> #include "system.h"
> #include "Timer/timer.h

system.h: hier ist nur F_CPU definiert

#ifndef SYSTEM_H_
#define SYSTEM_H_

#define  F_CPU 16000000

#endif

timer.h

#ifndef TIMER_H_
#define TIMER_H_

#include <avr/io.h>
#include <stdint.h>


class timer {
    

    public:
    timer(volatile uint8_t &tccra, uint8_t tccraNewState,
          volatile uint8_t &tccrb, uint8_t tccrbNewState,
          volatile uint8_t &ocra, uint8_t ocraNewState,
          volatile uint8_t &timsk, uint8_t timskNewState);
    ~timer();

    uint16_t getMilliseconds();
    uint8_t getSeconds();
    uint8_t getMinutes();
    uint8_t getHours();
    bool    getNewTime();

    void setMilliseconds(uint16_t milliseconds);
    void setSeconds(uint8_t seconds);
    void setMinutes(uint8_t minutes);
    void setHours(uint8_t hours);
    void setNewTime(bool newTime);
    void interrupt();
    

    private:
    
    bool _newTime;
    
    uint16_t _milliseconds;
    uint8_t _seconds;
    uint8_t _minutes;
    uint8_t _hours;
};

#endif

[Werner (Gast)]

Schau mal bei Ihm hier, ruft sogar direkt die Methode auf:

https://github.com/Phunkafizer/avr_seesys/blob/master/timer.h

(unten)

Werner

[Vincent H. (vinci)]

Franz schrieb:
> habe ich geändert
>

> ISR(TIMER0_COMPA_vect)
> {
>     timer0.interrupt();
> }

Übersetzt du den obigen Code mit g++?
Wenn ja, dann musst du die Funktion als extern "C" deklarieren.

extern "C" ISR(TIMER0_COMPA_vect)
{
    timer0.interrupt();
}

Generell sollte man die Nutzung von C-Makros unter C++ vermeiden, vor 
allem wenn sie wie im Fall von Atmel offensichtlich nicht für C++ 
ausgelegt sind.


Die Methode direkt Aufrufen ist ausschließlich dann möglich, wenn sie 
statisch ist.

[Wilhelm M. (wimalopaan)]

Franz schrieb:
> Hallo,
> ich bekomme es leider immer noch nicht vernünftig hin, Interruptroutinen
> für einen Atmega in C++ zu implementieren.

Ich denke, mal, dass der Timer gar nicht läuft. Du stellst da TimerMode 
6 (reserved) ein. Schau Dir mal an, was Du in tccr0a schreibst.

[Stefan E. (sternst)]

Vincent H. schrieb:
> Übersetzt du den obigen Code mit g++?
> Wenn ja, dann musst du die Funktion als extern "C" deklarieren.

Nein, muss er nicht. Das erledigt das ISR-Makro bereits.

Vincent H. schrieb:
> wenn sie wie im Fall von Atmel offensichtlich nicht für C++
> ausgelegt sind

Wie kommst du darauf?