Code anpassen (DS18S20)

[Tobias N. (silberkristall)]

Hallo,

ich habe mir etwas zum auslesen eines DS18S20 geschrieben. Das auslesen 
mittels SkipRom klappt perfekt. Nur würde ich gerne noch mittels ROm 
Code auslesen können.

Hier mal die Codes:

// SkipRom

unsigned char Read_Skip_Temp(void)
{
    char get[10];
    int k;
    char temp_c;
      
    ow_reset();
    ow_wr_byte(0xCC); //Skip ROM
    ow_wr_byte(0x44); // Start Conversion
    _delay_ms(800);
    ow_reset();
    ow_wr_byte(0xCC); // Skip ROM
    ow_wr_byte(0xBE); // Read Scratch Pad
    for (k=0;k<9;k++){get[k]=ow_rd_byte();}

    temp_c = get[0]; // Temp data plus lsb
    temp_c = temp_c/2;

  return (temp_c);
}

// MatchRom

unsigned char Read_Match_Temp(unsigned char* romaddr)
{
    int get[10];
    int k;
    int i;
    int temp_c;
      
    ow_reset();
    ow_wr_byte(0x55);
    for (i=0; i<8; i++)
      {
        ow_wr_byte(romaddr[i]);
      }

    ow_wr_byte(0x44); // Start Conversion
    _delay_ms(800);
    ow_reset();
    
    ow_wr_byte(0x55);
    for (i=0; i<8; i++)
      {
        ow_wr_byte(romaddr[i]);
      }
    
    ow_wr_byte(0xBE); // Read Scratch Pad
    for (k=0;k<9;k++){get[k]=ow_rd_byte();}
    
    temp_c = get[0];
    temp_c = temp_c / 2;

  return (temp_c);
}

Den RomCode habe ich wie folgt in der c datei abgelegt:

*** Devices ***/
unsigned char sensor1[8] = {16,157,172,77,2,8,0,86};
/*** Devices ende ***/

Der MatchRom code klappt garnicht, ich dachte mir an einen aufruf wie

temp = Read_Match_Temp(sensor1);

dabei soll die temperatur dann in temp stehen. Mit SkipRom klappt das 
wunderbar.

Dann hätte ich gerne noch nachkomma stellen und "minus" Temperaturen. 
Leider verlässt mich da mein wissen.

Enweder so das temp dann temp[0] die vorkomma und temp[1] die nachkomma 
temperatur enthält oder noch besser das temp bereits die temperatur 
inkl. nachkomma enthält. Bei Minus dann noch das - (minus Zeichen davor) 
und vielleicht so das man als wert noch eine 1 für plus und eine 0 für 
minus oder so in der art hat um das im Code dann weiter verarbeiten zu 
können.

Wäre toll wenn mir das jemand so anpassen könnte.

Danke euch

Tobias

[Chris (Gast)]

Hi,

was mir zumindest mal aufgefallen ist:

Deine Funktion Read_Match_Temp ist vom Typ insigned char:

unsigned char Read_Match_Temp(unsigned char* romaddr)


Sie gibt bei dir allerdings einen int zurück, da du temp_c als int 
deklariert hast:

int temp_c;

Soweit ich weiß, hat beim AVR-GCC ein int standardmäßig 16bit,

Eventuell liegt da der Fehler.

lg
Chris

[Tobias N. (silberkristall)]

Jau, also jetzt klappt das auch mit dem RomCode, also ich kann jetzt die 
Temperatur mittels temp = Read_Match_Temp(sensor1); auslesen, aber halt 
noch ohne nachkomma und minus werten, wäre toll wenn das auch noch 
klappen würde :)

Danke schonmal :)

[Cyblord -. (cyblord)]

Tobiiiiii....Nicht schon wieder...Wie lange kann man sich an sowas 
eigentlich aufhalten? Vor allem wenn man bedenkt dass du einen 
kompletten Bordcomputer mit Sensoren in 6-8 Wochen gebaut hast ;-)

Aber gut:
Das Problem der Nachkommastellen hat nichts mit dem auslesen zu tun, 
sondern mit der Darstellung. Und die hängt davon ab, wie und worauf du 
den Wert darstellen willst. Der Wert den du ausliest hat bereits alle 
Infos, Vorzeichen und Nachkommastellen enthalten.

Ist dir klar dass die Temperatur in 16tel Grad angegeben wird? Also eine 
Ganzahldivision durch 16 gibt dir die ganzen Grad Celcius. Eine 
Modulo-Division durch 16 gibt dir die Nachkommastellen. Jetzt musst du 
nur noch die ganzen Grade in einen String, ein Komma daneben, dann die 
Nachkommastellen anhängen. Fertig.

Wenn du die 2 Bytes für die Temp in C in eine int16_t Variable steckst, 
hast du automatisch eine Variable mit korrektem Vorzeichen. Da musst du 
sonst gar nichts mehr machen. Wenn die dann mittels itoa in einen String 
umgewandelt wird dann ist auch dort das Vorzeichen korrekt.

gruß cyblord

[Tobias N. (silberkristall)]

egal ob ich int16_t double float was auch immer nehme, ich kriege immer 
nur GANZzahlen, egal was ich mache und versuche.

[Karl H. (kbuchegg)]

Tobias N. schrieb:
> egal ob ich int16_t double float was auch immer nehme, ich kriege immer
> nur GANZzahlen, egal was ich mache und versuche.

Tja.
Dann solltest du spätestens jetzt mal herzeigen, was du jetzt machst 
oder versuchst.

[Cyblord -. (cyblord)]

Tobias N. schrieb:
> egal ob ich int16_t double float was auch immer nehme, ich kriege immer
> nur GANZzahlen, egal was ich mache und versuche.

Du liest ja auch nicht was ich dir schreibe. Die int16_t gibt dir die 
negativen Zahlen. Komma musst du selber ausrechnen und einfügen.

gruß cyblord

[Tobias N. (silberkristall)]

ok, also ich habe jetzt mal die berechnung aus einem anderen code 
kopiert aber trotzdem nur ganzzahlen, ich vermute schon es liegt an der 
ausgabe selbst, hier mal die codes:


Das auslesen:

unsigned char Read_Match_Temp(unsigned char* romaddr)
{
    int get[10];
    int k;
    int i;
    float temp_c;
      
    ow_reset();
    ow_wr_byte(0x55);
    for (i=0; i<8; i++)
      {
        ow_wr_byte(romaddr[i]);
      }

    ow_wr_byte(0x44); // Start Conversion
    _delay_ms(800);
    ow_reset();
    
    ow_wr_byte(0x55);
    for (i=0; i<8; i++)
      {
        ow_wr_byte(romaddr[i]);
      }
    
    ow_wr_byte(0xBE); // Read Scratch Pad
    for (k=0;k<9;k++){get[k]=ow_rd_byte();}
    
    temp_c=(get[0]-0.25f+(((float)get[7]-(float)get[6])/(float)get[7]));
    temp_c = (floor(temp_c*10.0+0.5)/10);  
    
    //temp_c = get[8];
    //temp_c = temp_c / get[7];
    //temp_c = temp_c * 10;
    //temp_c = temp_c / 2;
    //temp_c = temp_c / 10;
    
  return (temp_c);
}

Und hier wie ich das in der main.c aufrufe:

int main (void)
{

// Timer
  // Global Interrupts aktivieren
  sei();
      
    TIMSK |= (1<<TOIE1);
    TCCR1B |= (1<<CS11) |  (1<<CS10);     //Prescaler = 64
    TCNT1 = 0xFFFF;         //Zaehlregister vorladen mit FFFF zum Sofortstart

  char Buffer[20];
  float temp;
    //init lcd
    lcd_init();
    lcd_clear();
  
  //lcd_draw_fullscreen_bmp(img);
  
  while(1)
  {
  
  uhr(minute, stunde);
  
  ow_reset();
  temp = Read_Match_Temp(sensor1);
  
  itoa(temp, Buffer, 10);
  lcd_set_cursor(10,LINE7);
  lcd_puts(small_font, "Temperatur: ");
  lcd_puts(small_font, Buffer);
  }
}

[Karl H. (kbuchegg)]

cyblord ---- schrieb:
> Tobias N. schrieb:
>> egal ob ich int16_t double float was auch immer nehme, ich kriege immer
>> nur GANZzahlen, egal was ich mache und versuche.
>
> Du liest ja auch nicht was ich dir schreibe. Die int16_t gibt dir die
> negativen Zahlen. Komma musst du selber ausrechnen und einfügen.

@TO
Und wenn 0.5°C Auflösung reichen (es gibt selten einen Grund warum das 
nicht reichen sollte), dann ist das auch besonders einfach:
Einfach das MSB und das LSB aus dem Scratchpad zusammenfügen und man 
erhält das Doppelte der Gradzahl vorzeichenrichtig. Bei der Ausgabe kann 
man dann immer noch die Division durch 2 machen und so dann für die 
Ausgabe die Kommastelle generieren.
Aber das alles hat cyblord ja schon geschrieben. Wo also ist dein 
Problem?

[Tobias N. (silberkristall)]

Naja, er gibt mir bei get[0] 45 aus, wenn ich das durch 2 teile würde 
ich 22,5 bekommen, er gibt mir aber nur 22 auf meinem display aus. laut 
meinem multimeter sind hier 22,7C also 22,5 naja würde ja dann schon 
passen, aber ich erhalte in der ausgabe nie eine nachkomma zahl.

[Karl H. (kbuchegg)]

temp_c=(get[0]

Schon falsch.
In der Doku steht eindeutig, dass die Temperatur um das 0.5°C Bit, also 
das LSBit bereinigt werden soll.

Sieh dir doch wenigstens die Doku an. So schwer ist das nicht.

Und ja. Du musst erst mal das Scratchpad Byte 0 und 1 zu einem 16 Bit 
Wert zusammenfügen, um die Temperatur zu erhalten. Die wird dann durch 2 
divisiert und mit der erweiterten Formel um 1/16 Grad ergänzt.
Aber eigentlich ist das nur Augenauswischerei. Denn die Nachkommastelle 
hat mit einer realen Temperatur sowieso nicht viel zu tun. Oder hat es 
in deinem Zimmmer an allen Stellen exakt 24.7 Grad?

[Karl H. (kbuchegg)]

Tobias N. schrieb:
> Naja, er gibt mir bei get[0] 45 aus, wenn ich das durch 2 teile würde
> ich 22,5 bekommen, er gibt mir aber nur 22 auf meinem display aus.

Logisch.
Wiew soll denn in einem Integer eine Nachkommastelle herkommen?

itoa wandelt nun mal einen Integer in seine Textrepräsentierung. Egal ob 
temp ein float ist oder nicht.

[Karl H. (kbuchegg)]

Und wie soll da

unsigned char Read_Match_Temp(unsigned char* romaddr)


ein float mit Nachkommastellen zurückkommen, wenn der Returnwert der 
Funktion ein unsigned char ist?

[Karl H. (kbuchegg)]

Und PS:
Wie soll in einem 'unsigned' ein Vorzeichen enthalten sein?

Schon alleine das Wort 'unsigned' impliziert, dass es eben kein 
Vorzeichen gibt.

[Karl H. (kbuchegg)]

Lassen wir mal das ganze Techtelmechtel mit den 1/16 Grad weg:

int16_t Read_Match_Temp(unsigned char* romaddr)
{
  uint8_t get[10];
  uint8_t i;
  int16_t temp;
      
  ow_reset();
  ow_wr_byte(0x55);
  for (i=0; i<8; i++)
  {
    ow_wr_byte(romaddr[i]);
  }

  ow_wr_byte(0x44); // Start Conversion
  _delay_ms(800);
  ow_reset();
    
  ow_wr_byte(0x55);
  for (i=0; i<8; i++)
  {
    ow_wr_byte(romaddr[i]);
  }
    
  ow_wr_byte(0xBE); // Read Scratch Pad
  
  for (k=0;k<9;k++)
  {
    get[k] = ow_rd_byte();
  }

  temp = (int16_t)( ( get[1] << 8 ) | get[0] ); 

  return temp;
}

...

int main (void)
{

// Timer
  // Global Interrupts aktivieren
  sei();
      
    TIMSK |= (1<<TOIE1);
    TCCR1B |= (1<<CS11) |  (1<<CS10);     //Prescaler = 64
    TCNT1 = 0xFFFF;         //Zaehlregister vorladen mit FFFF zum Sofortstart

  char Buffer[20];
  int16_t temp;

  //init lcd
  lcd_init();
  lcd_clear();
  
  while(1)
  {
    uhr(minute, stunde);
  
    temp = Read_Match_Temp(sensor1);

    lcd_set_cursor(10,LINE7);
    lcd_puts(small_font, "Temperatur: ");

    sprintf( Buffer, "%+03d.%1d", temp/2, ( temp%2) * 5 ); 
    lcd_puts(small_font, Buffer);

    /* oder
      itoa( temp/2, Buffer, 10 );
      lcd_puts(small_font, Buffer);
      lcd_puts(small_font, ".");
      itoa( (temp%2)*5, Buffer, 10 );
      lcd_puts(small_font, Buffer);
 
      aber die sprintf Lösung hat auch noch andere Vorzüge. Unter anderem
      den, dass sie mit führenden 0-en nach Bedarf auffüllt. Sie hat aller-
      dings auch einen Nachteil: sie braucht Platz und ist vergleichsweise
      langsam. Wenn das keine Rolle spielt, dann ist sie aber perfekt.
    */
  }
}

[eProfi (Gast)]

Dass sich überhaupt noch jemand mit diesem lernresistenten Ignoranten 
und Voll....... abgibt??

Am 17.01.2013 habe ich ihm bereits die Lösung vor die Nase gehalten:
Beitrag "Re: DS18S20 auslesen (finde den fehler nicht)"


Und dass mindestens 100 User die Ausgabe vor ihm auch hinbekommen UND 
dokumentiert haben, ist an ihm ebenso vorübergegangen.