Unterprogramme in Interrupt-Routine

[Philipp (Gast)]

Hallo
ich möchte Unterprogramme in eine Timer Interrupt-Routine einbauen, doch 
es funktioniert nicht. Es bleibt vermutlich im ersten Unterprogramm 
stecken und das Interrupt kommt nicht.
Beides getrennt von einander funktioniert. Also wenn ich die 
Unterprogramme im void main() teste funktioniert es und wenn ich den 
Interrupt alleine teste funktioniert er auch.

#include <33FJ256MC710A.h>
#include <stdio.h>
#include <stdlib.h>

#fuses XT,PR_PLL,NOWDT
#use delay (crystal=8M, clock=80M)
#INT_TIMER2

int eins(int a)
{
    int b=3,v=2;
    a=b+v;
    return(a);
}

int zwei(int d)
{
    int e=4,a,f;
    f=eins(a);
    d=e+f;
    return(d);
}

void TIMER2interrupt()
{
    int d,c=0;
        c=zwei(d);
        if(c==9)
        {
            output_high(PIN_E5);
        }
        else
            output_low(PIN_E5);
   clear_interrupt(INT_TIMER2);
}

void main()
{
    setup_timer2(TMR_DISABLED);
    setup_timer2(TMR_INTERNAL | TMR_DIV_BY_1,4000);

    enable_interrupts(INT_TIMER2);
    enable_interrupts(INTR_GLOBAL);
    clear_interrupt(INT_TIMER2);

    while(1)
    {
    }
}

Ich hoffe mir kann jemand helfen

mfg

[Philipp (Gast)]

PS:
Ich verwende den dsPIC33FJ256MC710A, das dsPICDEM MCLV, einen CCS 
Compiler

[Karl H. (kbuchegg)]

grundsätzlich:

sag nicht "es funktioniert nicht".
Sag "ich erwarte das und das. Aber statt dessen passiert dieses und 
jenes"


dann

int zwei(int d)
{
    int e=4,a,f;
    f=eins(a);
    d=e+f;
    return(d);
}

welchen Wert hat hier a, wenn du die FUnktion eins aufrufst?

Wenn du eine lokale Variable nicht ausdrücklich initialisierst, kannst 
du dich auf keinen Wert verlassen. Es ist zufällig, welchen Wert a hier 
hat.

[Karl H. (kbuchegg)]

Selbiges hier

void TIMER2interrupt()
{
    int d,c=0;
        c=zwei(d);

welchen Wert hat d?
der ist völlig undefiniert. zufällig.

Gut, so wie zwei geschrieben ist, spielt das keine Rolle. Es ist 
trotzdem schlechtes Programmieren, wenn man die Internals von zwei 
kennen muss, um zu wissen, ob es an dieser Stelle egal ist oder nicht. 
Hier wird der Inhalt einer Variablen an eine Funktion übergeben und über 
diesen Inhalt wissen wir nur eines: der ist zufällig.

[NurEinGast (Gast)]

Deine Funktionen sind - sorry - ziemlicher Unsinn.
Die übergeben Parmeter werden nie verwendet.

Wenn ich's im Kopf richtig überschlagen habe, kommt aus "zwei(d)" immer 
9 heraus.
Wobei der Parameter "d" für zwei(d) nicht intialisuert wird. Ist aber 
egal, weil er in den Funktione sowieso nicht benutzt wird.

Dieses Konstrukt ist vollkommen unklar.

[Karl H. (kbuchegg)]

Irgendwie fehlt mir in deinem Code das Gegenstück zu

    setup_timer2(TMR_DISABLED);

du gibst zwar im Nachhinein Interrupts frei. Das ist alles schön und 
gut.
ABer wird dadurch auch der Timer enabled? Arbeitet der?

[Philipp (Gast)]

Es soll die LED am Pin E5 leuchten.

Den Wert der Variable a soll er aus dem ersten Unterprogramm übernehmen

[Philipp (Gast)]

@ NurEinGast:
Ich weiß das es keine großartiges Programm ist, es geht mir darum einen 
Wert von einem Unterprogramm in ein zweites zu übergeben und dann in 
einer Interrupt-Routine eine Aktion durchzuführen. Wenn ich das geschaft 
habe werde ich es natürlich noch Sinnvoll ausbauen.

Ja eigentlich sollte immer 9 heraus kommen aber trotzdem leuchtet die 
LED nicht.

@ Karl Hein:
Wenn ich bei anderen Programmen ohne Unterprogramme die 
Timer-Interrupt-Routine verwende funktioniert sie einwandfrei.

[Rufus Τ. F. (rufus)]

Philipp schrieb:
> Den Wert der Variable a soll er aus dem ersten Unterprogramm übernehmen

Wer ist "er"?

Und hast Du Dir überhaupt schon mal angesehen, wie in C die 
Parameterübergabe und -Rückgabe mit Funktionen (so nennt man das, was 
Du als "Unterprogramm" bezeichnest) funktioniert?

Was sagt denn Dein C-Buch dazu?


Mit Interrupts jedenfalls hat Dein Problem überhaupt nichts zu tun.

[profi (Gast)]

schreib mal das "#INT_TIMER2" über die Funktion für den Timer IRQ

also:

#INT_TIMER2
void TIMER2interrupt()

[Amateur (Gast)]

Da "C", außer bei einfachsten Befehlen, sowieso Funktionen einbindet, 
warum nicht?

Normalerweise kannst Du innerhalb von Unterbrechungen machen was Du 
willst.

Wichtig ist nur, dass Du garantiert fertig wirst, mit etwas Reserve, 
bevor ein erneuter Aufruf erfolgt.
Schön wäre es auch, wenn noch etwas Zeit, für die Hauptroutine und 
eventuelle andere Unterbrechungen, übrig bleibt.

Alles in allem ein zeitliches Problem. Es besteht ja auch kein 
Unterschied zwischen einem linearen Programmablauf und einem mit 
Unterprogrammen.
Natürlich kosten der Aufruf (call), ein eventuelles Registersichern 
(push), die Restauration von Registern (pop) und der Rücksprung ihre 
Zeit (ret).
Bei längeren Routinen ist das prozentual Vertretbar. Für kürzere 
Sequenzen bietet sich die Anweisung Inline an.
Ob damit mehr als das call und ret gespart werden ist eine andere Sache.

[nein? (Gast)]

Man sollte keine solchen Spielchen machen. Was sollen Unterprogramme in 
einem interrupt? Vergiss solches Zeug. Eine schlechte Idee.

[Philipp (Gast)]

"Er" ist die zweite Funktion

Ein Beispiel ist darin:

int example(int a, int b)
{
 return a+b;
}

.
.
.

void main()
{
int a=9;
int b=15;
int c=example(a,b)
}

[Philipp (Gast)]

Danke Profi, dass war das Problem...

Danke an alle

[Karl H. (kbuchegg)]

?
Sagtest du nicht, du hättest Interrupts alleine getestet?

[Philipp (Gast)]

Ja hab ich und weil ich beide Funktionen gelöscht habe war auch das 
#INT_TIMER2 genau vor der Interrupt-Routine

[Profi (Gast)]

kein Problem
Rechnung folgt....... ;-)