Timerinterrupt und UART

[Praktikant (Gast)]

Hallo.

Ich habe mal eine Frage.

Mein Programm hat eine Uhr eingebaut. Diese wird über einen Timer im 
Sekundentakt inkrementiert. Der Timer ist so eingestellt, dass er jede 
Sekunde einen Interrupt auslöst. In der Routine wird nur ein Flag 
gesetzt.
Die Zeit wird in der while(1)-Schleife gebildet wenn sich das Flag 
ändert.

Zugegeben ist die Uhr nicht die Genauste, aber für meine Zwecke reicht 
das.

Nun frage ich in regelmäßigen Abständen (alle 2 Sekunden) drei Werte von 
dem µC über die RS232 mit 9600 Baud ab.

Wenn ich nach einem Tag eine Auswertung in Excel mache, dann kann man 
erkennen, dass irgendwann ein Wert unterschlagen wurde.

Meine Vermutung ist, dass der Timerinterrupt dazwischenhaut. Wie mache 
ich es am geschickteste, dass dieses Problem nicht mehr auftritt?

[Mike (Gast)]

Praktikant schrieb:
> Meine Vermutung ist, dass der Timerinterrupt dazwischenhaut. Wie mache
> ich es am geschickteste, dass dieses Problem nicht mehr auftritt?

Îndem du die serielle Kommunikation auch über eine ISR laufen läßt und 
in der ISR nicht "stundenlang" den µC blockierst.

> Zugegeben ist die Uhr nicht die Genauste,
Und was fehlt ihr?
Beitrag "Die genaue Sekunde / RTC"

[Praktikant (Gast)]

Mike schrieb:
> in der ISR nicht "stundenlang" den µC blockierst

?Versteh ich nicht. Was willst du mir sagen?

Mike schrieb:
>> Zugegeben ist die Uhr nicht die Genauste,
> Und was fehlt ihr?
> Beitrag "Die genaue Sekunde / RTC"

Den Artikel kenn ich. Für mein Projekt aber nicht von Belang.


Welche serielle Kommunikation soll ich in einer ISR ablaufen lassen?
Senden, oder Empfangen?

[Thomas E. (thomase)]

Praktikant schrieb:
> Meine Vermutung ist, dass der Timerinterrupt dazwischenhaut. Wie mache
> ich es am geschickteste, dass dieses Problem nicht mehr auftritt?
Vielleicht.
Deswegen beschreib nicht in Prosa, was das Programm machen soll, sondern 
zeig deinen Quelltext als angehängtes C-File. Sonst kann man nur im 
Nebel stochern.

mfg.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

1. Tue dir einen Gefallen und rechne die Uhr in der ISR fertig. Dann 
musst du nicht im Hauptprogramm oder der Anzeige Routine auch noch an 
der Uhr rumfummeln.
Uhren sind ein simples 60:60:24 Getriebe und in ein paar Zeilen 
erledigt.

2. Wenn du die USART per Receive Interrupt betreibst, ist sicher 
gestellt, das sie sofort am Ende eines etwaigen anderen Interrupts 
behandelt wird.
 Wenn dein MC im Hauptprogramm rumtrödelt, wird sie sogar sofort 
gezündet und empfängt sicher das Zeichen. Entweder zeigst du per Flag, 
das da was neues im Buffer ist, oder du machst den ganzen Stringkram in 
der ISR. Das hängt davon ab, wie komplex die Auswertung ist.

[Praktikant (Gast)]

Entschuldigung für die späte Reaktion, aber mir ist gestern was 
dazwischen gekommen. Sonst hätte ich mich schon eher gemeldet.

Hier der Auszug meines Quelltextes:
Die While Schleife:

    while(1)
    {
    
    //Timer 1 Auswertung
    if(bSekunde==1)  //Nur bearbeiten, wenn der Sekunden-Interrupt ausgelöst hat
    {
      bSekunde=0;  //Flag zurücksetzen
      sekunde++;
      
      if(State.bPumpOnOff==1){Laufzeit++;}    //Laufzeit der Pumpe in Sekunden hochzählen
      if(State.bTimerDeltaT==1){ZeitDeltaT++;}//Laufzeit der delta T Erfassung in Sekunden hochzählen
      
      if(sekunde == 60)
      {
        minute++;

        if(State.bPumpDelay==1){ZeitPumpDelay++;}    //Laufzeit der Pumpe in Minuten hochzählen
        sekunde = 0;
        LCD_String(1,13,itoa(minute,Zeit,10));
      }
      if(minute == 60)
      {
        stunde++;
        minute = 0;
        LCD_String(1,10,itoa(stunde,Zeit,10));
        LCD_String(1,13,"  ");
      }
      if(stunde == 24)
      {
        stunde = 0;
        LCD_String(1,10,"  ");
        LCD_String(1,10,"0");
      }

    Temperature1=DS18B20_Read_Addressed_Temperature(rom1,byte);
    Temperature2=DS18B20_Read_Addressed_Temperature(rom2,byte);
    }

    if(sekunde%2==0)    //Alle 2 Sekunden die Temperatur ausgeben
    {
      LCD_String(3,13,dtostrf(Temperature1,3,1,buf));
      LCD_String(4,13,dtostrf(Temperature2,3,1,buf));
      LCD_String(2,13,dtostrf(Temperaturold,3,1,buf));
    }    
    
    
    if((Temperature1>Temperaturold) && State.bPumpDelay==0)  //Nur machen wenn ein neuer Pumpzyklus möglich ist und die Temperatur größer ist als der letzte Wert
    {
      State.bTimerDeltaT=1;
    }
    if(Temperature1<=Temperaturold)
    {
      ZeitDeltaT=0;
      Temperaturold=Temperature1;
      State.bTimerDeltaT=0;
    }
  
    if(ZeitDeltaT>=10)
    {
      if(Temperature1>Temperaturold+DELTATEMP)
      {
      State.bPumpOnOff=1;    //Pumpe anschalten
      PumpeAn;
      ZeitDeltaT=0;
      State.bTimerDeltaT=0;
      }      
    }
  
    if(Laufzeit>=LAUFZEITPUMPE)      //Nach Laufzeit Pumpe aus
    {
      PumpeAus;
      State.bPumpOnOff=0;  //Pumpe aus
      Laufzeit=0;
      Temperaturold=Temperature1;
      State.bPumpDelay=1;      //Pumpe hat Zyklus beendet und braucht die nächste Zeit nicht gestartet werden.
    }      
    
    if(ZeitPumpDelay==PUMPDELAY){State.bPumpDelay=0;}  //Ein neuer Pumpzyklus kann gestartet werden
        
    //RS232 Auswertung
    if(uart_str_complete == 1)  //Nur abarbeiten, wenn etwas empfangen wurde
    {
      if(0==strcmp(uart_string,"Read Romcode"))    //Romcode eines einzelnen Fühlers auslesen
      {
        DS18B20_Read_RomCode(byte);
        for(i=0;i<8;i++){Send_Byte_UART(byte[i]);}  //Rom Code muss Byteweise gesendet werden, da NULL enthalten sein kann        
        uart_str_complete=0;  //Flag zurücksetzen
      }
      if(0==strcmp(uart_string,"Messung"))
      {
        //"<MS1>xx.x<MS2>yy.y<PS>z<EOD>";

        strcpy(strSendBuffer,"<MS1>");
        strcat(strSendBuffer,dtostrf(Temperature1,3,1,buf));
        strcat(strSendBuffer,"<MS2>");
        strcat(strSendBuffer,dtostrf(Temperature2,3,1,buf));
        strcat(strSendBuffer,"<PS>");
        strcat(strSendBuffer,"x");
        strcat(strSendBuffer,"<EOD>");
        strSendBuffer[22]='0'+State.bPumpOnOff%10;
        Send_String_UART(strSendBuffer);
        uart_str_complete=0;  //Flag zurücksetzen
      }
      else
      {
        for(i=0;i<8;i++){strBuffer[i]=uart_string[i];}
        strBuffer[8]='\0';
        if(0==strcmp(strBuffer,"Set time"))      //Setzen der Uhrzeit per RS232. Befehl lautet 'Set time hh:mm:ss'
        {
          
          Zeit[0]=uart_string[9];
          Zeit[1]=uart_string[10];
          Zeit[2]='\0';
          stunde=atoi(Zeit);
          LCD_String(1,10,Zeit);

          Zeit[0]=uart_string[12];
          Zeit[1]=uart_string[13];
          Zeit[2]='\0';
          minute=atoi(Zeit);
          LCD_String(1,13,Zeit);
      
          Zeit[0]=uart_string[15];
          Zeit[1]=uart_string[16];
          Zeit[2]='\0';
          sekunde=atoi(Zeit);
        }        
          uart_str_complete=0;    //Flag zurücksetzen      
      }      
    }  
  
  }

Die Interrupt Routinen

ISR(USART_RXC_vect)
{
  
  uint8_t nextChar;
  
  // Daten aus dem Puffer lesen
  nextChar = UDR;
  if( uart_str_complete == 0 ) {  // wenn uart_string gerade in Verwendung, neues Zeichen verwerfen
  
  // Daten werden erst in uart_string geschrieben, wenn nicht String-Ende/max Zeichenlänge erreicht ist/string gerade verarbeitet wird
  if( nextChar != '\n' )//|| nextChar != '\r' || uart_str_count < 10 )
  {
    uart_string[uart_str_count] = nextChar;
    uart_str_count++;
  }
  else
  {
    uart_string[uart_str_count] = '\0';
    uart_str_count = 0;
    uart_str_complete = 1;
  }
}

}



ISR (TIMER1_COMPA_vect)
{
  bSekunde=1;
}

[STK500.-Besitzer (Gast)]

Du solltest dir einen Uhrzeit-String zusammenbasteln und den am Ende der 
Auswertung an das LCD schicken.
Die Kommunikation mit der Anzeige bremst das ganze Programm aus.

[Praktikant (Gast)]

@STK500.-Besitzer
Versteh ich nicht. Die Uhrzeit wird nur jede Minute und jede Stunde 
aktualisiert.

Womit ich ein Problem habe ist scheinbar dieser Teil:

  if(0==strcmp(uart_string,"Messung"))
      {
        //"<MS1>xx.x<MS2>yy.y<PS>z<EOD>";

        strcpy(strSendBuffer,"<MS1>");
        strcat(strSendBuffer,dtostrf(Temperature1,3,1,buf));
        strcat(strSendBuffer,"<MS2>");
        strcat(strSendBuffer,dtostrf(Temperature2,3,1,buf));
        strcat(strSendBuffer,"<PS>");
        strcat(strSendBuffer,"x");
        strcat(strSendBuffer,"<EOD>");
        strSendBuffer[22]='0'+State.bPumpOnOff%10;
        Send_String_UART(strSendBuffer);
        uart_str_complete=0;  //Flag zurücksetzen
      }

Wenn hier der Timerinterrupt dazwischenhaut, dann habe ich den Murks.
Ich könnte auch die Interrupts vorm Senden deaktivieren und danach 
wieder aktivieren.
Aber mache ich mir damit die Uhr nicht noch ungenauer?

[holger (Gast)]

>Wenn hier der Timerinterrupt dazwischenhaut, dann habe ich den Murks.

Wie sieht dieser Murks aus?
Welchen Controller verwendest du?

[Paule H. (stk500-besitzer)]

Praktikant schrieb:
> Versteh ich nicht. Die Uhrzeit wird nur jede Minute und jede Stunde
> aktualisiert.

Was macht denn die Funktion "LCDString"?

[Prakitkant (Gast)]

Paule H. schrieb:
> Was macht denn die Funktion "LCDString"?

Sie schreibt an LCDString(Zeile,Spalte,String).
Aber das tut sie NUR wenn eine neue Minute berechnet wurde, bzw. NUR 
wenn eine neue Stunde berechnet wurde.

Oder anders formuliert:
NUR einmal pro Minute wird die Uhrzeit aktualisiert.

holger schrieb:
> Wie sieht dieser Murks aus?

Ich übertrage folgenden String:
<MS1>xx.x<MS2>yy.y<PS>z<EOD> (Wobei x, y und z durch Werte ersetz 
werden)

Auf meinem PC kommt dann irgendwann folgendes an

EOD><MS1>xx.x<MS2>yy.y<PS>z<

Die Software die diesen String auswertet verschluckt sich daran.

Ich verwende einen Atmega 8 mit 4 Mhz Quarzoszillator. Ich weiss, dass 
ein "Baudratenquarz" besser wäre. Ist aber nicht vorhanden

[Paule H. (stk500-besitzer)]

Prakitkant schrieb:
> Oder anders formuliert:
> NUR einmal pro Minute wird die Uhrzeit aktualisiert.

und wie lange braucht sie dafür?
Selbst wenn nur alle paar Stunden die Anzeige auf diese Art aktualisiert 
wird, braucht es eine gewisse Zeit, bis das Prozedere abgearbeitet 
wurde.
Da kann es schon sein, dass ein, zwei oder (Uhren-)Interrupts "verloren" 
gehen.

Kannst du mal bitte das ganze Programm als Anhang hier posten.
Programme in Prosa geschrieben, sind viel länger als Quellcode...

[holger (Gast)]

>Auf meinem PC kommt dann irgendwann folgendes an
>
>EOD><MS1>xx.x<MS2>yy.y<PS>z<
>
>Die Software die diesen String auswertet verschluckt sich daran.

Woher weisst du das nicht deine PC Software diesen Murks baut?
Vieleicht ist der Atmega ja diesmal unschuldig;)

[Prakitkant (Gast)]

holger schrieb:
> Woher weisst du das nicht deine PC Software diesen Murks baut?

Weil dann zusätzlich in meinem Display totaler Müll steht.
Nebenbei. Wenn mein String so aussieht "EOD><MS1>xx.x<MS2>yy.y<PS>z<"
dann sieht man schon, dass noch Reste "EOD>" vom alten String 
übermittelt wurden und an den neuen String angehängt wurde. So als ob 
die vorherige  Übertragung durch den Timerinterrupt unterbrochen wurde.

Paule H. schrieb:
> Selbst wenn nur alle paar Stunden die Anzeige auf diese Art aktualisiert
> wird, braucht es eine gewisse Zeit, bis das Prozedere abgearbeitet
> wurde.

Versteh ich wieder nicht.
Zu irgendeiner Zeit muss ich doch die Uhrzeit ausgeben. Mit meiner 
Methode tausche ich pro Minute nur die Minuten bzw. Stunden aus. Das ist 
weniger Aufwand als den kompletten Zeitstring auszugeben.

[holger (Gast)]

>> Woher weisst du das nicht deine PC Software diesen Murks baut?
>
>Weil dann zusätzlich in meinem Display totaler Müll steht.

DAS wäre noch mal ne wichtige Zusatzinformation gewesen.

>So als ob die vorherige  Übertragung durch den
>Timerinterrupt unterbrochen wurde.

Das dauert nicht lange genug um die Übertragung
wirklich zu unterbrechen.

Hier ist eine undichte Stelle in deinem Programm:

  if( nextChar != '\n' )//|| nextChar != '\r' || uart_str_count < 10 )

das kann zu Bufferoverflow führen falls mal kein \n auftaucht.
Dabei steht die Lösung schon daneben. Ts, ts.

Was ist das für eine Pumpe die du da schaltest?
Das könnte auch der Übeltäter sein wenn die Schalterei
nicht richtig entstört ist. Das würde auch den Müll
im Display erklären.

[Praktikant (Gast)]

holger schrieb:
> DAS wäre noch mal ne wichtige Zusatzinformation gewesen.

Tschuldigung.

holger schrieb:
> Was ist das für eine Pumpe die du da schaltest?

Im Moment noch Keine.
Meine Pumpe ist zur Zeit nur eine LED.

holger schrieb:
> das kann zu Bufferoverflow führen falls mal kein \n auftaucht.
> Dabei steht die Lösung schon daneben. Ts, ts.

Sehe ich ein. Allerdings überträgt mein Programm ein \n.
Die Lösung die daneben steht habe ich auch schon probiert (Deswegen 
steht es noch daneben). Allerdings besteht auch hier das Problem.

Kurioserweise läuft mein Programm heute schon seit 8h 30 min, ohne Murks 
im Display. Jedoch auch ohne PC, der ständig die Werte abfragt.

Hmmm. Liegt wohl doch irgendwie an der seriellen Übertragung