sei(); führt zu Absturz

Gast #774241
Lesenswert?

Hallo!

Versuche gerade einen Interrupt auf meinem Mega16 zu programmieren, aber 
leider stürzt mein Programm immer ab wenn ich sei(); aufrufe (der µC 
startet neu/hängt sich auf).

Ich habe Timer2 so initialisiert:

TCCR2 = (1<<CS22) | (1<<CS21) | (1<<CS20) | (1<<WGM21);
OCR2 = 0x0012;
TIMSK |= (1<<OCIE2);

Dazu dann dann der Interrupt:

ISR(SIG_OUTPUT_COMPARE2)
{
  lcd_writeChar('T');
}

Woran könnte das liegen?
Gast #774250
Lesenswert?

So weit ich weiß, ist OCIE2 der Compare Match für Timer 2. Also genau 
das, was ich brauche. Und im Tutorial steht:

/* Timer/Counter2 Compare Match */
#define SIG_OUTPUT_COMPARE2             _VECTOR(3)

Das passt doch bestens zusammen?
Gast #774265
Lesenswert?

> So weit ich weiß, ist OCIE2 der Compare Match für Timer 2.

Ja.

> Und im Tutorial steht:
>
> /* Timer/Counter2 Compare Match */
> #define SIG_OUTPUT_COMPARE2             _VECTOR(3)
>
> Das passt doch bestens zusammen?

Hast du auch den dazugehörigen Text darüber gelesen?
Gast #774304
Lesenswert?

Und Dein
1
lcd_writeChar('T');
ruft nicht wiederunm andere Interruptroutinen auf?

Mach doch mal:
1
ISR(SIG_OUTPUT_COMPARE2)
2
{
3
    do_something = TRUE;
4
}
und im Hauptprogramm
1
while (1)
2
{
3
  if (do_something) 
4
  {
5
      lcd_writeChar('T');
6
      do_something = FALSE;
7
  }
8
}

Rick
Gast #774480
Lesenswert?

Auch wenn
>ISR(SIG_OUTPUT_COMPARE2)

eine Mischung aus deprecated und aktuellen Makros und defines ist, weiss 
der Compiler doch, was damit zu tun ist, und auch der Rest sieht nicht 
verkehrt aus.

Insofern wie immer: Zeig mal den ganzen Code.

Oliver
#774567
Lesenswert?

Beide Namen sind definiert, also ists wurscht, welchen man nimmt.
Am Problem ändert das nichts, das muß also woanders liegen.


Mir ist kürzlich auch ein Programm in den Wald gelaufen und ich hab mich 
dumm und dusslig gesucht.

Der Grund war, ich hab für nen ATtiny85 compiliert, aber nur nen 
ATtiny45 eingesteckt.
Damit gehts nach dem ersten RET(I) ab ins Nirwana.


Peter
Gast #779634
Lesenswert?

Heh, sorry dass ich dich so schockiere. Aber das habe ich per copy & 
paste aus einem anderen Projekt übernommen, an dem ich gerade arbeite. 
Und da funktioniert der Timerinterrupt tadellos. So falsch kann das also 
nicht sein...
#779647
Lesenswert?

Bernd wrote:
> So falsch kann das also
> nicht sein...
Es gibt zwei Möglichkeiten:
1.: Normaler Interrupt-Handler, Deklaration mit ISR(VEKTORNAME_vect)

2.: Unterbrechbarer Handler, Prototyp mit __attribute__((interrupt)), 
dann Definition wie in Deinem Code (aber wenn's geht mit aktuellem 
Vektornamen und nicht mit dem veralteten SIGNAL-Zeugs). Bei Dir fehlt 
also mindestens das __attribute__((interrupt))...

Unterbrechbare Handler sollten aber eh nur dann eingesetzt werden, wenn 
es unbedingt sein muss. Also halte Dich besser an die Version unter 1.

EDIT:
Rick Dangerus hat schon mal (07.02., 13:45) eine Version gepostet, die 
zumindest funktionieren müsste und bis auf die Tatsache, dass er auch 
einen veralteten Vektornamen verwendet, auch OK ist.
#779706
Lesenswert?

Jörg Wunsch wrote:
> Johannes M. wrote:
>
>> 2.: Unterbrechbarer Handler, Prototyp mit __attribute__((interrupt)),
>
> Schnee von gestern. ;-)
Hach, schon wieder was Neues... Schlimm. Seit welcher Version ist das 
so? Ich dachte ja eigentlich, ich hätte wenigstens ne einigermaßen 
aktuelle Version hier auf dem Rechner...
Gast #780268
Lesenswert?

Also ich habe jetzt folgende Notationen ausprobiert. Und keine davon 
funktioniert! Die LCD Funktion wird nicht erreicht, und der µC startet 
neu...

void SIG_OUTPUT_COMPARE2( void )
{
  lcd_writeChar('T');
}


ISR(TIMER2_COMP_vect)
{
  lcd_writeChar('T');
}

ISR(SIG_OUTPUT_COMPARE2)
{
  lcd_writeChar('T');
}
#780294
Lesenswert?

Erstens kann es ziemlich riskant sein, in einer ISR einen Aufruf einer 
Ausgabe-Funktion zu tätigen (je nachdem, wie die Funktion beschaffen 
ist), und zweitens müssten (vorausgesetzt, die avr/interrupt.h ist 
korrekt eingebunden!) die beiden letzten Versionen zumindest ohne die 
LCD-Ausgabe funktionieren.
Gast #780320
Lesenswert?

Die Interrupts.h ist korrekt eingebunden.

  "und zweitens müssten die beiden letzten Versionen [...] 
funktionieren."
Genau das ist ja das Problem ^^



Zur LCD-Ausgabe: Mir ist schon klar, dass das wenig sinnvoll ist. Aber 
mir geht es nur darum, ob diese Zeile überhaupt erreicht wird (habe dort 
einen Breakpoint gesetzt). Und das ist nicht der Fall. Der µC startet 
neu, bevor er überhaupt in die ISR springt...

Antwort schreiben

Bitte melde dich an, um einen Beitrag zu schreiben.

oder

Mit Google-Account einloggen

Die Registrierung ist kostenlos und dauert nur eine Minute.

Jetzt registrieren