Interrupt Routine stoppt die Ausführung

[Roman G. (roman76aut)]

Hallo Leute,

ich habe folgendes einfaches Programm für meinen Atmega328p:

#define F_CPU 16000000UL

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



void main (void) {
    DDRD = (1<<PD3);
    DDRB = (1<<PB0);
    cli();
    TCCR2A = (1<<WGM21) | (1<<WGM20) | (1<<COM2B1);
    OCR2B = 0;
    TIMSK2 = 0x01;
    sei();
  TCCR2B = (1 << CS20);
  
  while(1){
    //PORTD ^= (1<<PD3);
    PORTB ^= (1<<PB0);
    _delay_ms(1000);
    OCR2B +=10;
    //PORTD ^= (1<<PD3);
    PORTB ^= (1<<PB0);
    _delay_ms(1000);
  
  }
}

/*
ISR (TIMER2_OVF_vect)
{
  _delay_ms(1000);
  reti();
}

*/

Ein shell Skript erstellt und sendet das Programm:

echo "Compiler Object"
avr-gcc -c -Os -mmcu=atmega328p -o SimpleIO.o SimpleIO.c
echo "Compiler ELF"
avr-gcc SimpleIO.o -o SimpleIO.elf
echo "Compiler HEX"
avr-objcopy -O ihex -j .data -j .text SimpleIO.elf SimpleIO.hex
echo "AVRDUDE"
sudo avrdude -patmega328p -P usb -cusbasp-clone -Uflash:w:SimpleIO.hex:a

Wenn ich die ISR auskommentiert lasse wie oben, dann funktioniert es das 
PB0 blinkt und PD3 mittels Fast-PWM in 10er Schritten alle zwei Sekunden 
hochgezählt wird.

Sobald ich ISR verwende (egal ob mit delay, reti() oder leer) bleiben 
alle LED dunkel. Bisher konnte ich es also auf die Existenz der ISR 
zurück führen.

Bitte um eure Hilfe da ich dieses Problem noch nicht im Netz gefunden 
habe.

Danke im voraus,
Roman

[Kurt B. (kurt-b)]

Roman G. schrieb:

> Sobald ich ISR verwende (egal ob mit delay, reti() oder leer) bleiben
> alle LED dunkel. Bisher konnte ich es also auf die Existenz der ISR
> zurück führen.

Wartest du in der IR 1 Sekunde lang bis du zurückspringst?


 Kurt

[MWS (Gast)]

Ist doch klar, der µC  triggert die ISR schneller, als deren Laufzeit 
beträgt. Somit ist das Ding immer in der ISR und kommt vielleicht pro 
ISR-Ausführung 1 Takt in der Hauptschleife weiter. Eine Wartesekunde in 
der Hauptschleife kostet 16000000 Takte, d.h. nach ca. 185 Tagen wird 
die LED zu leuchten beginnen :D
Musst nur etwas länger aufbleiben.1

[Roman76aut (Gast)]

Hab es auch als leere routine versucht. Eigentlich ist mein problem, 
dass mit isr nix geht.

[Roman76aut (Gast)]

Also:

ISR (TIMER2_OVF_vect)
{
}

[MWS (Gast)]

Falsches Target?

[Carl D. (jcw2)]

Wenn deine ISR aktiv ist, dann wird sie praktisch dauerhaft aktiv sein.
Das delay(1000) sorgt dafür, daß nach RETI() schon der nächste Int 
ansteht. In so einem Fall führt ein AVR genau einen Befehl in der 
Hauptschleife aus, bevor wieder die ISR dran ist. Und ein delay(1000) in 
der Hauptschleife muß 1..20 Millionen Taktzyklen "vernichten", von denen 
dann aber nur einer je Sekunde ausgeführt wird. Dauert also etwas 
länger!
Zudem wird Reti() dafür sorgen, daß je ISR-Aufruf einige Bytes auf dem 
Stack liegen bleiben. Nach einiger Zeit endet das bestenfalls in einem 
"Soft-Reset".

Also:
- ISR() braucht kein reti(), das macht es schon selbst
- delay() in ISR ist böse, denn
- delay() in Mainloop erwartet, daß die geplanten Takt auch von der
  Mainloop ausgeführt werden. Werden 50% davon in der ISR ausgeführt,
  dann dauert das delay doppelt so lang. Werden 99.999% der Takte in der
  ISR ausgeführt, dann ...

[MWS (Gast)]

> reti();

Du umgehst das normale POPen der Register, nimm das raus, das ist 
Unsinn.

[Carl D. (jcw2)]

Gerade in anbetracht der Programmgröße:
Wirf einen Blick ins .lss File, da kann man sehen, wie falsch einem der 
Compiler verstanden hat. (Wobei er es in konkreten Fall sehr einfach 
hatte. Das Falschverstehen.)

[Roman G. (roman76aut)]

MWS schrieb:
> Falsches Target?

Ich habe auch eine Vermutung das irgend eine Compiler Option diesen 
Effekt bewirkt. Welches wäre das richtige Target?

Kann sich jemand mein sh ansehen bzgl. der Optionen?

MWS schrieb:
> Eine Wartesekunde in
> der Hauptschleife kostet 16000000 Takte, d.h. nach ca. 185 Tagen wird
> die LED zu leuchten beginnen :D

Mit ist klar das dieser delay Unsinn ist. Eigentlich hatte ich auch noch 
die Led zum Toggeln drinnen. Etwas so:

ISR (TIMER2_OVF_vect)
{
  _delay_ms(1000);
  PORTB ^= (1<<PB0);
}

Später hab ich auch geschrieben das ich die ISR leer hatte und trotzdem 
das Problem aufgetaucht ist.
Eigentliches Ziel ist im Interrupt einfach eine Zählervariable zu 
erhöhen.

ISR (TIMER2_OVF_vect)
{
  someCounter++;
}

lg
Roman

[Carl D. (jcw2)]

> Wenn ich die ISR auskommentiert lasse wie oben, dann funktioniert es das
> PB0 blinkt und PD3 mittels Fast-PWM in 10er Schritten alle zwei Sekunden
> hochgezählt wird.

Die "auskommentierte ISR", bei eingeschaltetem Timer2OverflowInterrupt 
sorgt für einen Neustart bei Überlauf des Timer2. Das führt auch zu 
"LED-Flackern", aber nicht zum gewünschten Programmablauf. Troll?

[Roman G. (roman76aut)]

Carl D. schrieb:
> Die "auskommentierte ISR", bei eingeschaltetem Timer2OverflowInterrupt
> sorgt für einen Neustart bei Überlauf des Timer2. Das führt auch zu
> "LED-Flackern", aber nicht zum gewünschten Programmablauf. Troll?

Led flackern hab ich auch gesehen, kann mich aber nicht mehr erinnern in 
welchem Status da ich vieles ausprobiert habe. Kann gut sein dass ich 
auch die Timer Mask auskommentiert hatte um die PWM Funktion zu sehen.
Warum geht nix wenn ich ISR aktiv habe? Zumindest flackert die Led wenn 
der Timerüberlauf resetet. Hatte ich bisher nicht gewußt aber es zeigt 
immerhin das der µC bis dahin richtig funktioniert.

[neuer PIC Freund (Gast)]

Für ein besseres Aha-Erlebnis:

Benutze das Atmel Studio mit dem Simulator.