Mittelwertbildung von Temperatur

[Dieter B. (Gast)]

Moin Moin ^^

Ich bin dabei eine Temperaturanzeige zu programmieren. Das klappt soweit 
auch alles. Was mir nicht gefällt, ist das der Wert sich viel zu oft im 
Display ( 7- Segment) ändert. Dies habe ich versuch mit hilfe einer 
"Mittelwertbildung" zu unterdrücken. Der Sensor braucht ca. eine Sekunde 
bis er ein neues Ergebniss liefern kann. In meine ISR (wo ich den Wert 
abfrage vom DS1621 & umrechne) springe ich ca. jede 2 Sekunden rein. 
Wenn ich den Wert jetzt 16 x auf summiere, müsste es doch eigentlich "32 
Sekunden" dauern bis ich meinen neuen Wert auf dem Display sehen kann 
oder ?
Aktuell ändert sich der Wert aber fast jede 5 Sekunden.

Hat jemand eine Idee, sieht was ich falsch mache ?

#define F_CPU 8000000

#define DS1621_Write  0x9E
#define DS1621_Read   0x9F

#define Segment_1_AN PORTB &= ~(1<<PB3)
#define Segment_1_AUS PORTB |= (1<<PB3)

#define Segment_2_AN PORTB &= ~(1<<PB2)
#define Segment_2_AUS PORTB |= (1<<PB2)

#define Segment_3_AN PORTB &= ~(1<<PB1)
#define Segment_3_AUS PORTB |= (1<<PB1)

#define Segment_4_AN PORTB &= ~(1<<PB0)
#define Segment_4_AUS PORTB |= (1<<PB0)

#define Mittelwert 16

int Zahlen[13]= {
          0b00111111, // 0
          0b00000110, // 1
          0b01011011, // 2
          0b01001111, // 3
          0b01100110, // 4
          0b01101101, // 5
          0b01111101, // 6
          0b00100111, // 7
          0b01111111, // 8
          0b01101111, // 9
            0b01100011, // *
            0b01000000, // Minus
          0b00000000  // alle Aus
        };

#include <avr/io.h>
#include <util/delay.h>
#include "i2cmaster.h"
#include <avr/interrupt.h>

volatile unsigned char  TempH, TempL = 0;
volatile int16_t    Wert, Summe, Ergebnis;
volatile int8_t      TempAbs = 0;
volatile uint8_t    IsNegative, I, Done;
 

int main(void)
{
  DDRB |= ((1<<PB0) | (1<<PB1) | (1<<PB2) | (1<<PB3));
  DDRD |= ((1<<PD0) | (1<<PD1) | (1<<PD2) | (1<<PD3) | (1<<PD4) | (1<<PD5) | (1<<PD6));
  
    OCR1A   = 15624;            // Legt den Wert ZUM Überlaufes fest ( ca. jede 2 Sec. )
  TCCR1A |= (1<<WGM11);          // CTC Modus
    TIMSK  |= (1<<OCIE1A);          // Compare Match Interrupt enable
    TCCR1B |= ((1<<CS12) | (1<<CS10));    // Prescaler auf 1024 setzen (F_CPU/1024)
  sei();
  

  i2c_init();
  
  i2c_start(DS1621_Write); // Starten der Umwandlung (Messung)
  i2c_write(0xEE);
  i2c_stop();
       
  i2c_start(DS1621_Write); // Starten der Auslesung (Temperaturregister)
  i2c_write(0xAA);
  i2c_stop();

  Segment_1_AUS; Segment_2_AUS; Segment_3_AUS; Segment_4_AUS;
  
while (1)
{
  
  Segment_1_AN;
  PORTD = Zahlen[Wert & 0x000F];
  _delay_ms(4);
  Segment_1_AUS;

  Segment_2_AN;
  PORTD = Zahlen[(Wert & 0x00F0) >> 4];
  _delay_ms(4);
  Segment_2_AUS;
  
  Segment_3_AN;
  PORTD = Zahlen[(Wert & 0x0F00) >> 8];
  _delay_ms(4);
  Segment_3_AUS;
  
  Segment_4_AN;
  PORTD = Zahlen[(Wert & 0xF000) >> 12];
  _delay_ms(4);
  Segment_4_AUS;

} // Ende While
} // Ende Main

ISR (TIMER1_COMPA_vect)
{

  if (I <= Mittelwert)
  {  
    I++;  
    i2c_start(DS1621_Read);
    TempH = i2c_readAck();
    TempL = i2c_readNak();
    i2c_stop();

/*    if ((TempL & 0b10000000) == (0b10000000))
    {
      TempH++;
    }
*/      
      Summe += TempH;
    
  }


  if (I == Mittelwert)
  {
    Ergebnis = Summe / Mittelwert;
    
    I = 0;
    Summe = 0;
    
    
      if (Ergebnis < 0)
      {
        IsNegative = 1;
        TempAbs = ~(Ergebnis) + 1;
      }
      else
      {
        IsNegative = 0;
        TempAbs = Ergebnis;
      }

      Wert = ((((TempAbs / 10) % 10) << 8) | ((TempAbs % 10) << 4) | (0x0A)); // Wert entspricht dem "Wert" aus dem Array (Zahlen)

      if (TempAbs == 0)
      {
        Wert = 0xCC0A;
      }
      else if (TempAbs >= 100)
      {
        Wert = (Wert | 0x1000);
      }
      else if (TempAbs < 10)
      {
        Wert = (Wert | 0xCC00);
      }
      else if (TempAbs < 100)
      {
        Wert = (Wert | 0xC000);
      }

      if (IsNegative == 1)
      {
        Wert = (Wert & 0x0FFF) | (0xB000);
      }
    
    
  }
  

  
  
}// Ende ISR

[Gerald G. (gerald_g)]

Ocr1a hat normalerweise 8 bit. 15624 ist etwas mehr. Für Interrupts im 
Sekunden Bereich nehme ich normal einfach den watchdog

[Dieter B. (Gast)]

Gerald G. schrieb:
> Ocr1a hat normalerweise 8 bit. 15624 ist etwas mehr. Für
> Interrupts im
> Sekunden Bereich nehme ich normal einfach den watchdog

Bei nem TINY2313 ist das doch nen 16 Bit Register ?!

[Timmo H. (masterfx)]

Ich habe mir deinen Code jetzt nicht wirklich angesehen.
Aber ich würde da einfach so eine Art Tiefpass drüber machen...

#define FILTER_SHIFT  6

filtered_temp = filtered_temp - (filtered_temp >> FILTER_SHIFT) + aktuelle_temp;  
ausgabe_temp = filtered_temp >> FILTER_SHIFT;

[Gerald G. (gerald_g)]

Dieter B. schrieb:
> Bei nem TINY2313 ist das doch nen 16 Bit Register ?!

Ja, da hast du recht

Dann toggle doch einfach bei jedem isr Aufruf eine deiner Segmente.

[Dieter B. (Gast)]

Gerald G. schrieb:
> Dieter B. schrieb:
>> Bei nem TINY2313 ist das doch nen 16 Bit Register ?!
>
> Ja, da hast du recht
>
> Dann toggle doch einfach bei jedem isr Aufruf eine deiner Segmente.

What soll mir das bringen ?

[Gerald G. (gerald_g)]

So debugge ich meinen code.
Du willst wissen warum die Aktualisierung nur 5 Sekunden dauert anstatt 
30? Schaue wie schnell die isr aufgerufen wird. Passt das? Dann pack das 
togglen in die if's rein. Passt das? ...
Das ist ne Sache von Minuten da so einen toggle Befehl an verschiedenen 
stellen nacheinander einzufügen.

[holger (Gast)]

>  TCCR1A |= (1<<WGM11);          // CTC Modus

Der Kommentar ist falsch. WGM11 auf 1 ist kein CTC.

[Dieter B. (Gast)]

holger schrieb:
>>  TCCR1A |= (1<<WGM11);          // CTC Modus
>
> Der Kommentar ist falsch. WGM11 auf 1 ist kein CTC.

Ahh,

Das müsste dann der WaveGeneratorMode sein oder ?
Welche Bits sind denn für den CTC zuständig ?! Finde die gerade nicht

[holger (Gast)]

>Welche Bits sind denn für den CTC zuständig ?! Finde die gerade nicht

Die Tabelle im Datenblatt ist so gross wie ein LKW.
Die kann man nicht übersehen.