Timer stört Temperaturmessung

[Praktikant (Gast)]

Hallo zusammen.

Ich habe ein Problem und keinen Schimmer was da falsch läuft.

Mein Timer1 wird so konfiguriert, dass er jede Sekunde ein Interrupt 
auslöst:

  // Timer 0 konfigurieren
 // TCCR1A = (1<<WGM11); // CTC Modus
  TCCR1B |= (1<<WGM12)|(1<<CS12)|(1<<CS10); // Prescaler 8
  // ((4000000/64)/1000) = 62,5
  OCR1A = F_CPU/1024-1;
  // Compare Interrupt erlauben
  TIMSK |= (1<<OCIE1A);
sei();

In der Interrupt-Routine mache ich nichts weiter, als ein Flag zu 
setzten und eine Variable hochzuzählen:

ISR (TIMER1_COMPA_vect)
{
  sekunde++;  
  bSekunde=1;
}

In meiner While-Schleife wird auf das Flag reagiert.

if(bSekunde==1)
    {
      cli();
        bSekunde=0;
        if(sekunde == 60)
        {
          minute++;
          sekunde = 0;
          LCD_String(4,13,itoa(minute,Zeit,10));
          LCD_String(4,16,"  ");
        }
        if(minute == 60)
        {
          stunde++;
          minute = 0;
          LCD_String(4,10,itoa(stunde,Zeit,10));
          LCD_String(4,13,"  ");
          LCD_String(4,16,"  ");
        }
        if(stunde == 24)
        {
          stunde = 0;
          LCD_String(4,10,itoa(stunde,Zeit,10));
          LCD_String(4,16,"  ");
        }
    
        if(sekunde%5==0)
        {
  
        Temperature1=DS18B20_Read_Addressed_Temperature(rom1,byte);
        Temperature2=DS18B20_Read_Addressed_Temperature(rom2,byte);
        LCD_String(1,13,dtostrf(Temperature1,3,1,buf));
        LCD_String(2,13,dtostrf(Temperature2,3,1,buf));
        }
    
    LCD_String(4,16,itoa(sekunde,Zeit,10));    
    sei();
    }

Da ich weiß, dass LCD-Ansteuerung und Temperatursensor auslesen lange 
dauert habe ich mal ein cli() und ein sei()-Befehl eingebaut.
Trotz allem wird mir nach dem 3. Lesevorgang und nur bei Temperatur 1 
-0.1 °C angezeigt.
Temperatur 2 liefert immer einen korrekten Wert.

Wenn ich den Timer deaktiviere und in der While-Schleife nur

Temperature1=DS18B20_Read_Addressed_Temperature(rom1,byte);
Temperature2=DS18B20_Read_Addressed_Temperature(rom2,byte);
LCD_String(1,13,dtostrf(Temperature1,3,1,buf));
LCD_String(2,13,dtostrf(Temperature2,3,1,buf));

ausführe, dann erhalte ich von beiden Fühlern korrekte Werte.

Warum also schießt mir der Timer, trotz cli(), in die Auswertung der 
Fühler?

Jemand eine Idee?

Gruß

Der Praktikant

[Peter II (Gast)]

zeige uns mal etwa mehr vom code, es fehlten alle variablen 
Deklarationen.

[Karl H. (kbuchegg)]

-0.1 das dürfte Hex als 0xFFFF über die Leitung gehen.

Wie versorgst du den DS18B20, bzw. wie sieht die Abfrage des Sensors 
aus?

Der UNterschied dürfte darin liegen, dass du in dem einen Fall nur alle 
5 Sekunden den Sensor abfrägst, während du im anderen Fall den Sensor 
mehr oder weniger ständig abfrägst. So wie das aussieht, bist du im 5 
Sekunden Fall aber zu schnell - der erste Sensor ist noch nicht fertig.


Mit dem Timer, schätze ich mal, hat das ganze nur indirekt zu tun. Wenn 
du in deinen 2-ten Fall das 5 Sekunden Intervall mittels delay_ms(5000) 
einrichtest, kriegst du dann dasselbe Ergebnis?

[Praktikant (Gast)]

Die Sensoren werden mit 5 V versorgt. Nicht im parasitären Modus. 4,7k 
Pull Up.

Gelesen wird wie im Datenblatt angegeben

float DS18B20_Read_Addressed_Temperature(uint8_t *address, uint8_t *buffer)
  {
    uint8_t i;

    W1_Address_Device(address);
    W1_Byte_Write(CONVERT); // Start Conversion
    // für neuen Befehl bereit machen
    W1_DQ_High();
    _delay_ms(750);
    W1_Address_Device(address);
    W1_Byte_Write(READ_SP);
    // jetzt werden 9 Byte empfangen
    for(i=0;i<9;i++)
    {
      buffer[i] = W1_Byte_Read();
    }
    buffer[9]=0;
    //temp_msb = byte[1]; // Byte 1 enthält das Vorzeichen, 0 = +, 1 = -
    //temp_lsb = byte[0]; // Byte 0 enthält die Temperatur
    return((buffer[1]<<8 | buffer[0])*0.0625);
    
    
  }

Das mit dem delay_ms(5000) bringt nichts, da hier nur gewartet wird.

Wie? Ich bin im 5 Sekunden Fall zu schnell? Vielleicht habe ich das 
falsch formuliert. Statt alle 5 Sekunden kann ich auch jede Sekunde, 
oder alle 30 s die asuwertung machen. Das Ergebnis ist das Gleiche und 
damit hat der Timer sehr wohl etwas damit zu tun


Fall 1: Der Timer ist Aktiv -> TIMSK |= (1<<OCIE1A);

  while(1)
    {
  
    if(bSekunde==1)
    {
      cli();
        bSekunde=0;
        if(sekunde == 60)
        {
          minute++;
          sekunde = 0;
          LCD_String(4,13,itoa(minute,Zeit,10));
          LCD_String(4,16,"  ");
        }
        if(minute == 60)
        {
          stunde++;
          minute = 0;
                LCD_String(4,10,itoa(stunde,Zeit,10));
          LCD_String(4,13,"  ");
          LCD_String(4,16,"  ");
        }
        if(stunde == 24)
        {
          stunde = 0;
          LCD_String(4,10,itoa(stunde,Zeit,10));
          LCD_String(4,16,"  ");
        }
    
        if(sekunde%5==0)
        {
  
        Temperature1=DS18B20_Read_Addressed_Temperature(rom1,byte);
        Temperature2=DS18B20_Read_Addressed_Temperature(rom2,byte);
        LCD_String(1,13,dtostrf(Temperature1,3,1,buf));
        LCD_String(2,13,dtostrf(Temperature2,3,1,buf));
        }
    
    LCD_String(4,16,itoa(sekunde,Zeit,10));    
    sei();
    }
}

Fall 2: Der Timer wurde deaktiviert -> //TIMSK |= (1<<OCIE1A);

  while(1)
    {
  Temperature1=DS18B20_Read_Addressed_Temperature(rom1,byte);
  Temperature2=DS18B20_Read_Addressed_Temperature(rom2,byte);
  LCD_String(1,13,dtostrf(Temperature1,3,1,buf));
  LCD_String(2,13,dtostrf(Temperature2,3,1,buf));
    }

Im zweiten Fall findet die Auswertung permanent statt. Wie lange das 
dauert wird durch die einzelenen Wartezeiten bestimmt.

Im ersten Fall haut mir der Interrupt jede Sekunde dazwischen. D. h. 
wenn er gerade beim Lesen des Fühlers ist, unterrbricht der Timer.
ABER.
Ich habe ein cli() eingebaut, damit mir der Timer nicht mehr 
dazwischkloppen kann. Nur geht das irgendwie nicht.

[Pratikant (Gast)]

Ich wollte nur kurz mitteilen, dass das Problem gelöst wurde.

Ähem. Scheint wohl so, dass der Programmierer den Buffer der 
itoa-Funktion zu, öhm zu kleine dimensioniert hat.

Ich hasse diese ständigen Layer 8 Probleme


Einen guten Rutsch an alle, die das hier noch lesen.

[ha. (Gast)]

Ein LCD in einem Timer anzusteuern ist grottenschlechter Stil. Der 
Anfang vom Ende.

[Praktikant (Gast)]

Ähhh? Das LCD wird nicht im Timer angesteuert.

Wie ganz oben beschrieben setze ich im Interrupt nur ein Flag das ich in 
meiner Hauptschleife auswerte.

Nicht den Anfangsbeitrag lesen ist ein grottenschlechter Stil. ;-)