ISR - optimale Programmstruktur (MPS430 C-Code)

[Jürgen R. (duke)]

Hallo zusammen,

wie sieht eine optimale Programmstruktur mit Interrupts / ISR aus?

Ich lese oft, dass die ISR so kurz wie möglich gehalten werden soll.
Meistens sind jedoch noch weitere Berechnungen durchzuführen.

Ist die unten gezeigte Struktur ok oder gibt es noch 
Optimierungspotential?

Weiterhin suche ich eine Möglichkeit die Interruptauslastung 
auszumessen. Hat jemand eine Software Lösung?

Beispiel C-code für MSP430

// #includes, #defines etc.

void main(void) //wird nur einmal nach power on reset durchlaufen
{
// Initialisierung BasicTimer1
// erzeugt alle 976,5µs einen Interrupt

// sowie sonstige Initialisierungen
goto_lpm();
}


void goto_lpm(void)
{
// µC in Low Power Mode (LPM) versetzen
}


#pragma vector=BASICTIMER_VECTOR
__interrupt void basic_timer_ISR(void) //Aufruf alle 976,5µs
{
_DINT(); // keine weiteren Interrupts zulassen

if (10mal aufgerufen)   // dann 10ms_Flag setzen
if (1024mal aufgerufen) // dann 1000ms_Flag setzen

after_ISR();

_EINT(); // Interrupts wieder zulassen
}


// wird nach jedem ISR durchlaufen
// max. Bearbeitungszeit 976µs
void after_ISR() 
{
if (10ms Flag gesetzt?)
  {
    // hier alles rein was alle 10ms erledigt werden muss
    // z.B. Schalter abfragen, Sensoren abtasten etc. 
  }
if (1000ms Flag gesetzt?)
  {
    // hier alles rein was jede Sekunde erledigt werden muss
    // z.B Uhr aktualisieren (Sekunde++) etc.
  }
// hier geht der µC wieder automatisch in LPM
}

[Peter D. (peda)]

Nach goto_lpm läuft Dein Programm aus dem Main raus - ganz schlecht!
Da gehört ne Endlosschleife drumrum.


"goto" ist ein Schlüsselwort, besser nicht als Namen verwenden.


Wenn Du im Interrupt trotzdem die Funktionen aufrufst, kannst Du Dir das 
Flag setzen schenken.
Flags machen nur Sinn, wenn die langwierige Behandlung im Main erfolgt.


Peter

[Ampfing (Gast)]

Hallo Jürgen,

wie Peter schon sagte, das Flag-Setzen macht nur anders Sinn. In etwa 
so:

void main()
{
  //Initialisieren der HW

  while(1)
  {
    if (10ms_Flag == 1)
    {
       //alles was nach 10 ms getan werden soll tun
    }
    if (1000ms_Flag == 1)
    {
       //alles was nach 1000ms getan werden soll
    }
    schlafen_legen();
   }
}

Wenn mich nicht alles täuscht kannst Du Dir das _DINT() und _EINT() in 
der ISR sparen, das macht der MSP430 automatisch.

Viele Grüße

[Falk B. (falk)]

@ Jürgen R. (duke)

>Weiterhin suche ich eine Möglichkeit die Interruptauslastung
>auszumessen. Hat jemand eine Software Lösung?

Nee, aber Hardware (bin Hardwerker ;-)
Einfach zum Beginn der Interruptroutine ein IO-Pin auf HIGH setzen, am 
Ende wieder auf Low. Damit sieht man am Oszi schön, wann und wie oft der 
Interrupt auftritt. Und über eien RC-Filter + Multimeter kann man damit 
auch die Prozessorlast messen, quasi [PWM]].

MFG
Falk

[Christian R. (supachris)]

Lies dir mal diese AppNote durch: http://www.ti.com/litv/pdf/slaa294a

[Jürgen R. (duke)]

Hallo zusammen,

vielen Dank für die zahlreichen Tipps!

@Peter
"Nach goto_lpm läuft Dein Programm aus dem Main raus - ganz schlecht!"
Die haupt_routine() ist immer noch ausserhalb der main(),
ist das im unten gezeigten Code noch ein Problem?

@Falk
danke für die HW Lösung, die war mir schon bekannt.:-)
Wie kann ich sicherstellen (messen) dass ich keinen Interrupt auslasse?
Angenommen die haupt_routine() dauert so lange, dass schon wieder ein 
Interrupt ansteht.


Ich habe meine Struktur wie folgt angepasst.
Es würde mich freuen, wenn Ihr mir hiezu noch euren Kommentar geben 
könnt.

// #includes, #defines etc.

void main(void) //wird nur einmal nach power on reset durchlaufen
{
// Initialisierung BasicTimer1
// erzeugt alle 976,5µs einen Interrupt

// sowie sonstige Initialisierungen
// alles was vor der Hautproutine nur einmalig passiert

haupt_routine();
}

void haupt_routine(void)
{
  for (;;)  // Hauptschleife forever
  {
    // hier wird auch nach der ISR weitergemacht
    // ggf. µC-Pin setzen für Laufzeitmessung

    if (10ms Flag gesetzt?)
    {
      // hier alles rein was alle 10ms erledigt werden muss
      // z.B. Schalter abfragen, Sensoren abtasten etc. 
    }
    if (1000ms Flag gesetzt?)
    {
      // hier alles rein was jede Sekunde erledigt werden muss
      // z.B Uhr aktualisieren (Sekunde++) etc.
    }

    // ggf. µC-Pin rücksetzen für Laufzeitmessung

    LPM3; // µC in Low Power Mode (LPM) versetzen
  }
}


#pragma vector=BASICTIMER_VECTOR
__interrupt void basic_timer_ISR(void) 
{
 // Aufruf alle 976,5µs
 // In der ISR NUR Flags setzen!

 if (10mal aufgerufen)   // dann 10ms_Flag setzen
 if (1024mal aufgerufen) // dann 1000ms_Flag setzen

 LPM3_EXIT; // erst mal wach bleiben
}