ISR mit Zustandsmaschine

Jens K. schrieb:
> Das Problem ist, dass die LEDs immer eingeschaltet sind.

Logisch, das erste, was du in der ISR auch machst, ist "Zustand" auf 
"LED_Ein" zu setzen.

Ersetze

1

    static U8 Zustand;

2

    Zustand = LED_Ein;

durch

1

    static U8 Zustand = LED_Ein;

Diese Dinge hier

1

        AT91C_BASE_PIOA->PIO_CODR = LED_AUSGABE; // LED on

2

3

        AT91C_BASE_PIOA->PIO_SODR = LED_AUSGABE; // LED off

sind korrekt?

Die ISR wird regelmässig aufgerufen?
Wie schnell wird die eigentlich aufgerufen? (*)

Wenn die mehr als, sagen wir mal 30 mal in der Sekunde aufgerufen wird, 
wirstdu das Blinken nicht mehr sehen.

(*) Ich zieh die Frage zurück und stelle eine Neue
> Dazu benutze ich einen Timer, der alle 0,5s einen Interrupt auslöst.
Hast du das kontrolliert?

Jens K. schrieb:
> Karl heinz Buchegger schrieb:
>> Diese Dinge hier
>>         AT91C_BASE_PIOA->PIO_CODR = LED_AUSGABE; // LED on
>>
>>         AT91C_BASE_PIOA->PIO_SODR = LED_AUSGABE; // LED off
>> sind korrekt?
>
> das ist korrekt.
> Der Timer ist so eingestellt, dass die ISR-Routine alle 0,5s aufgerufen
> werden sollte.
> Wie kann ich denn überprüfen, wie oft uns schnell sie aufgerufen wird?

Led blinken lassen :-)
(Jetzt haben wir ein Henne/Ei Problem)

Probier mal sowas

1

enum {LED_Ein, LED_Aus};

2

3

__ramfunc void ISR_Clock_Ausgabe (void)

4

{

5

    static U8 Zustand = LED_Ein;

6

    static U8 Counter = 0;

7

8

    Counter++;

9

    if( Counter == 10 ) {

10

      Counter = 0;

11

12

      switch (Zustand)

13

      {

14

      case LED_Ein:

15

        AT91C_BASE_PIOA->PIO_CODR = LED_AUSGABE; // LED on

16

        Zustand = LED_Aus;

17

        break;

18

19

      case LED_Aus:

20

        AT91C_BASE_PIOA->PIO_SODR = LED_AUSGABE; // LED off

21

        Zustand = LED_Ein;

22

        break;

23

24

      default:

25

        break;

26

    }

27

  }

28

}

und dann schraubst du die 10 hoch, bis
* du bei einer sehr großen Zahl angelangt bist
* oder die LED irgendwann zu flimmern und dann zu blinken anfängt

In deinem Code ist jedenfalls nichts zu entdecken, warum das kein 
Blinken stattfinden sollte.

Hast du vielleicht noch eine 2-te LED zur Verfügung?
Wenn ja, dann schalte die mal hier

1

    if( Counter == 10 ) {

2

      Counter = 0;

3

4

      .... hier LED einschalten

zusätzlich ein und sieh nach, ob das passiert (dann weist du, dass die 
ISR aufgerufen wird.
Auch den Gegentest machen: dort die LED ausschalten

Jens K. schrieb:

> Nachdem ich den Counter-Wert bis auf 300.000 hochgeschraubt habe

Ich hoffe du hast auch berücksichtigt, dass ein U8 (8 Bit Datentyp?) von 
Haus aus nur bis 255 zählen kann :-)

> Es scheint also an der Timereinstellung zu liegen!?

Sieht wohl so aus :-)

Kann es sein, dass deine 0.5 sich nicht auf Sekunden sondern auf 
Millisekunden bezieht?

Jens K. schrieb:
> Karl heinz Buchegger schrieb:
>> Ich hoffe du hast auch berücksichtigt, dass ein U8 (8 Bit Datentyp?) von
>> Haus aus nur bis 255 zählen kann :-)
>
> Natürlich (habe auf 32 Bit angepasst)!

Brav.
War nur so, dass mir 300000 ein wenig hoch vorgekommen ist.

> Ich habe was Lustiges festgestellt. Ändere ich die Timereinstellungen,
> dann ändert sich nichts an dem Takt; das heißt, die LED blinkt mit dem
> gleichen Takt weiter. Wie kann das denn sein???

Richtiger Timer benutzt?
(Keine Ahnung. Ich hab ja noch nicht einmal rausgefunden welchen µC du 
benutzt. Ein AVR-Mega oder Tiny ist es nicht. Und von anderen µC versteh 
ich nichts)

gastlich schrieb:
> Stefan Ernst schrieb:
>> Logisch, das erste, was du in der ISR auch machst, ist "Zustand" auf
>> "LED_Ein" zu setzen.
>
> hier stehts geschrieben !!!!
>
> nimm die definition der Variable "Zustand" aus der Interrupt Routine
> raus !!!

Nö.
Dieser Punkt ist schon längst abgehakt.

Die Variable ist static und anstatt der Zuweisung hat er jetzt eine 
Initialisierung.
Das ist soweit alles korrekt.

gastlich schrieb:
> Karl heinz Buchegger schrieb:
>> In deinem Code ist jedenfalls nichts zu entdecken, warum das kein
>> Blinken stattfinden sollte.
>
> sorry da kann ich nicht schweigen ....
>
> natürlich ist da was faul ...

Jungs.
Lernt C!

Ich weiß, das static immer wieder für Verwirrungen sorgt. Aber das ist 
alles korrekt, so wie er das macht.

> varliablen die in ISR verändert werden müssen als volatile declariert

Und wenn du schon dabei bist, lies auch noch mal über volatile nach und 
wann man diesen Modifier tatsächlich benötigt und warum.

gastlich schrieb:
> initialisierung ? wo ?
>
> innerhalb der ISR nutzt die nichts ...

Oh Mann.
Lern C!

Du verzapfst hier einen Unsinn nach dem anderen.

Jens K. schrieb:
> Noch eine Frage Karl Heinz:
>
> Könnte es sein, dass die ISR nach dem ersten Durchlauf sofort wieder
> aufgerufen wird, weil der Interrupt nicht zurückgesetzt wurde?

Hmm.
Kann ich dir nicht sagen. Da musst du das Datenblatt zum Prozessor 
befragen. Es wär zwar unüblich, aber möglich wärs.

gastlich schrieb:
> soll ichs noch ausführen ?
>
> ISR {
>
>  //initialisierung der varibale Zustand
>
>  //verwender der varibale zustand
>
>  //verändern der variable zustand
>
> }
>
> meiner meinung nach wir beim verwenden der varibale immer der gleiche
> wert drin stehen ...

Deine Meinung steht in starkem Kontrast zu dem was die ISO-Norm für C 
dem Compiler vorschreibt was zu passieren hat.

> es sei denn der compiler verschiebt wie durch ein
> wunder die intialisierung aus der isr raus .....

(Hinter vorgehaltener Hand:
letzten Endes bleibt dem Compiler nichts anderes übrig. Das definieren 
der static Variablen in einer Funktion limitiert lediglich den Scope 
unter dem die Variable sichtbar ist, auf die Funktion. Im Grunde hat man 
damit eine File-globale Variable erzeugt, die allerdings nur in dieser 
Funktion zugänglich ist. Und genau genommen findet die Intialisierung 
beim ersten Betreten der Funktion statt. Findet die Intialisierung aber 
mit einer Konstanten statt, kann kein Mensch den Unterschied 
herausfinden, denn wie gesagt: Ausserhalb der Funktion existiert die 
Variable offiziell gar nicht)

>
> und komm jetzt nicht mit static und so ... die initialisierung findet in
> der isr statt ...

Es ist im C-Standard exakt geregelt, was mit der Initialisierung einer 
Variablen in einer Funktion zu geschehen hat.

Jens K. schrieb:
> gastlich schrieb:
>> hab mal in meinen Routinen nachgeschaut, da kommt diese zeile in der
>> timer ISR vor :
>> tmp = AT91C_BASE_TC0->TC_SR;
>
> das hat leider auch nichts gebracht...
>
> Ich poste nochmal die aktuelle ISR.

Die ISR wird nicht viel bringen.
Du musst im Datenblatt nachlesen:
  Timereinstellung
  Wie ist das Interrupt Flag zu handhaben