11 Responses

1. 

   Steffen October 27, 2018 at 18:43

   Hallo, mit den Hinweisen von Nico funktioniert es jetzt. Weis nur nicht was vorher genau der Fehler war.
   Weil ein “Main()” für die Funktionen hier gewünscht wurde hier meine Version, die LCD Ansteuerung ist allerdings natürlich individuell, aber der Rest sollte passen:

   /*******************************************************************************
    * File:     ds18b20_ready.c
    * Keine Interrups => ausschalten !
    * nach  https://pic-projekte.de/blog/ds18s20/ from Nico
    * 
    ******************************************************************************/
   #include 
   #include "main.h"
   
   //Befehle 
   #define Convert_T           0x44    //Leitet die Temperaturmessung ein
   #define Read_scratchpad     0xBE    //Liest das Scratchpad inklusive des CRC Bytes aus
   #define Write_scratchpad    0x4E    //Schreibt 3 Byte in das Scratchpad (Byte1 -> TH, Byte2 -> TL, Byte3 Config)
   #define Copy_scratchpad     0x48    //Kopiert TH und TL aus dem EEPROM zum Scratchpad
   //#define Recall_E            0xB8    //Startet einen Rückruf von TH und TL vom EEPROM zum Scratchpad
   //#define Read_Power_Supply   0xB4    //Signalisiert den Versorgungszustand zum PIC 
   #define Resolution_11bit    0x5F    //9bit 375ms wait!
   #define Resolution_12bit    0x7F    //12bit 750ms wait! Config-Reg. Bit 6,5 = 11 s. Figure 10 https://www.maximintegrated.com/en/ds/DS18B20.pdf
   
   #define Skip_ROM            0xCC    //Alle Sensoren am Bus ansprechen
   #define Read_ROM            0x33
   #define Match_ROM           0x55
   #define Alarm_Search        0xEC
   
   #define DI                        PORTBbits.RB4   //PIC Eingang!
   #define LAT_DQ              LATBbits.LATB4  //PIC Ausgang!
   #define TRIS_DQ             TRISB4
   
   volatile uint8_t tempL, tempH;
   volatile float tempC = 0; // declare a variable to store 
   
   uint8_t ow_mri(void); //Reset
   uint8_t ow_rd_bit(void);
   uint8_t ow_rd_byte(void);
   void ow_wr_bit(uint8_t val);
   void ow_wr_byte(uint8_t val);
   
   
   /*******************************************************************************
    * DS18B20 FUNCTION                                                            *
    *******************************************************************************/
   int ds18b20(void)
   {
       uns16 w_counter = 0;
       PEIE = 0; //0 für Timer INT aus
       GIE = 0; //alle INT aus
   
       lcd_1602_clear();
       lcd_1602_string("* DS18B0: *"); //LCD display for the starting part
       while(w_counter  0
           {
             /*
               ow_wr_byte(Write_scratchpad);
               ow_wr_byte(0);  //TH Reg.
               ow_wr_byte(0);  //TL reg.
               ow_wr_byte(Resolution_11bit); // Auflösung = 12bit Standard
               //Reset for Temp.
               ow_mri();
             */
               ow_wr_byte(Skip_ROM);
               ow_wr_byte(Convert_T);
               //_delay_ms(375 ); //warten auf Temp. auslesen, 11bit
               _delay_ms(750 ); //warten auf Temp. auslesen, 12bit
   
               //Temp auslesen über Scratchpad
               ow_mri(); //Master Reset Impuls
               ow_wr_byte(Skip_ROM);
               ow_wr_byte(Read_scratchpad);
               tempL = ow_rd_byte(); //read low temp value
               tempH = ow_rd_byte(); //read high temp value
   
               //Testwerte
   //            tempH = 0xFF; //-10,1°C
   //            tempL = 0x5E;
   //            tempH = 0x01;
   //            tempL = 0x91;  //+25,1°C
   
               //Anzeigen
               lcd_1602_setcursor(2, 0);
               lcd_1602_hex(tempH);
               lcd_1602_hex(tempL);
               lcd_1602_string("h        ");
               _delay_ms(1000);
               tempC = ((uns16)tempH << 8) + (uns16)tempL; //put both value in one variable
               uns16 t, t2 = 0;
               if((tempH & 0x80) != 0) //negative Temp.
               {
                   //2er Kompliment umrechnen
                   t = tempH; // Store tempH value in t = 1111 1111  .
                   t <>= 4; // t = 0000 0000 0000 1010.
                   t = t * 100; // t = 10 * 100 = 1000.
                   t2 = tempL; //Store tempL value = 0101 1110.
                   t2 = t2 - 1; // t2= 0101 1101
                   t2 = ~t2; //t2 = 1010 0010
                   t2 = t2 & 0x0f; // t2 = 1010 0010 | 0000 1111 = 0000 0010
                   t2 = (uns16)(t2 * 6.25); // 0000 00010 = 2 x 6.25 = 12.50
                   tempC = ((unsigned int)t) + (unsigned int)t2; //put both value in one variable 1000 + 12.5 = 1012.5
               }
               else //pos. Temp
               {
                   //12bit LSB je °C = 0.0625, 11bit = 0.125 mit nur 375ms Conversion Times
                   tempC = (uns16)(tempC * 6.25); ////calculations used from the table provided in the data sheet of ds18b20
               }
   
               //Anzeigen
               lcd_1602_setcursor(2, 0);
               lcd_1602_bcd(1, w_counter);
               lcd_1602_string(".Temp:");
               if(tempH & 0x80)
                   lcd_1602_string("-");
               else
                   lcd_1602_string("+");
               //
               tempC /= 10;
               //lcd_float(tempC);
               lcd_1602_bcddez((uns16)tempC); //234 => 23.4°C
               lcd_1602_string(" ");
               lcd_1602_data(DEF_GRAD); //"°"
               lcd_1602_string("C");
           } //Reset
           w_counter++;
       }
   }
   

   • 

     NicoAuthor October 27, 2018 at 20:53

     Freut mich, dass es jetzt klappt 🙂

2. 

   Steffen October 27, 2018 at 18:23

   Hallo Nico,
   die Reihenfolge ist für mich nur logisch falsch, es funktioniert natürlich auch so, das weiß ich schon. Eigentlich setzt man ja aber erst den Ausgangtyp (TRIS) und dann den Wert. Auch das man durch das DQ gleich als Eingang schalten den Ausgang hier beim 18B20 wegen der 5k auf 1 setzt, ohne den Ausgang mit 1 extra zu setzen, habe ich schon soweit verstanden.
   Ich kämpfe aber seit Tagen damit, das zum laufen zu bringen und es tut einfach nicht. Noch nicht einmal eine Temp. erhalten, nur entweder FFFFh wenn was nicht tut oder 0550h. Im Datenblatt habe ich auch die 750ms (nicht µs!) wegen der 12bit inzwischen in der Tabelle gefunden, da hat der andere Schreiber Unrecht. Kommt einen aber erstmal sehr lang vor. Ich bekomme immer nur die 0550h vom 18B20, also 80°C was einfach aber Zimmertemp. mit um die FE60h sein müsste. Auch im Datenblatt stehen die 0550h als Vorgabewert bei “DS18B20 MEMORY MAP bei byte0 und byte1. Warum bekomme ich da nicht einen anderen Wert? Das macht mich wahnsinnig, irgendwas stimmt da nicht, die Schreibe- und Lesefunktionen funktionieren anscheinend, auch das Reset, da ich eine 0 vom Sensor bekomme. Der Pegel bei DQ ist auch okay, bei diversen Breakpoints gemessen. Habe auch bei https://circuitdigest.com/microcontroller-projects/pic16f877a-digital-thermometer-using-ds18b20 alles mit meinen Code verglichen, müsste passen. Von dort auch Anzeige für neg. Werte übernommen und mit den Werten aus Datenblatt von -55 – 80°C simuliert, LCD Ausgabe passt. Auch meine Zeit mit __delay_us() und __delay_ms() stimmen, Test Blink LED und das Signal auf Oszi sind exakt richtig.
   Was kann das noch sein, das ich immer die 0550h ausgelesen bekomme? Auch mit 3 Sensoren probiert, alle defekt? Danke für einen ev. Tipp.
   Steffen

3. 

   Steffen October 26, 2018 at 20:31

   Wo wird nochmal uint8_t ds18x20[9]; benötigt?
   Un die Reihenfolge

     LAT_DQ = 0;             // set the (I)O to low level
     TRIS_DQ = 0;            // config the DQ-IO as output (-> low)
   

   ist u.a. wohl auch falsch.

   Bei mir funktioniert jedenfalls nichts. Habe auch 3 versch. Sensoren probiert, um HW Fehler zu meiden und mit Oszi die Signale und Zeiten überprüft.
   Gibt es schon eine korrigierte lauffähige Version? Auch wegen der 750ms oder µs? Kann da im Datenblatt leider nichts finden.

   Danke
   Steffen

   • 

     NicoAuthor October 27, 2018 at 10:56

     Warum sollte die Reihenfolge falsch sein? Verstehst du überhaupt was da gemacht wird? Klar, wenn es bei dir nicht funktioniert (bei anderen übrigens schon), dann muss natürlich der Code und nicht etwa deine eigene Umsetzung das Problem sein …

     Steffen ganz im Ernst überdenke doch vielleicht nochmal deine Herangehensweise. So wirst du nicht weit kommen.

4. 

   Steffen Dünnebier October 25, 2018 at 20:08

   Hallo, leider läuft das bei mir überhaupt nicht. Verwende einen PIC18F26K22 und habe auch mit 750ms und mit 750 µs getestet. Aufrufende Funktion so:

   int ds18x20_nico(void)
   {
       PEIE = 0; //0 für Timer INT aus
       GIE = 0; //alle INT aus
         if(!ow_mri()) //Master Reset Impuls, 1-wire connected = 0
           {
               ow_skip_rom(); //OW_S_Rom(); 
               ow_wr_bit(Convert_T);
               _delay_us(750); //warten auf Temp. auslesen, µs nicht ms!
   
               ow_mri(); //Master Reset Impuls
               ow_wr_byte(Skip_ROM);
               ow_wr_byte(Read_scratchpad);
               //DS18x20 antwortet mit 9Byte 
               for(int i = 0; i < 9; i++) // Antwort einlesen: 9 Byte große Scratch Pad-Inhalt einlesen
               {
                   ds18x20[i] = ow_rd_byte();
               }
   

   In ds18x20[] ist da leider nichts im Debugger MPLAB-X v4.15 zu sehen. Wo soll das sein? Deswegen Umrechnung noch nicht weiter gemacht. Aber wo "Seite 79 unter der Überschrift “Die Umrechnung der Messwerte” ??? Die Links oben verweisen nur auf die Datenblätter und die haben nur 20 und nicht 79 Seiten! Auch die unterschiedliche Schreibweise der Funktionen und des Array ist erst etwas verwirrend. Leider nicht unbedingt sehr sorgfältig erstellt und das lässt halt auch prinzipielle Fehler im Code vermuten, weswegen das so nicht geht.
   Ein anderes gefundenes Example für PIC16F877A , was aber auch nicht geht (andere Zeiten?), berechnet so

               //Umrechnung der Messwerte
               tempL = read(); //read low temp value
               tempH = read(); //read high temp value
               Temp = ((uns16)tempH << 8) + (uns16)tempL; //put both value in one variable
               Temp = (uns16)((float)Temp * 6.25); //calcuation from the table provided
   

   also irgendwie ganz andere Empfangsdaten. Mit

   unsigned char read()
   {
       char i, result = 0;
       Rx_18B20; // TRIS is input(1)
       for (i = 0; i < 8; i++)
       {
           Tx_18B20; // TRIS is output(0)
           PortTx_18B20 = 0; // genarate low pluse for 2us
           _delay_us(2);
           Rx_18B20; // TRIS is input(1) release the bus
           if (PortRx_18B20 != 0)
               result |= 1 << i;
           _delay_us(60); // wait for recovery time
       }
       return result;
   }
   

   Alles sehr verwirrend. Leider kann ich aus dem Datenblatt auch nichts genaueres deuten. Könnte man nicht Bitte einmal ein komplettes funktionierendes Programm hier ablegen?
   Mit Arduino Uno Examples und dazu passender Bibliothek natürlich alles kein Problem, wie üblich.

   • 

     NicoAuthor October 27, 2018 at 10:43

     Hallo Steffen,

     zunächst einmal: Es ist keine gute Strategie die (für dich kostenlose Arbeit) anderer auf diese Weise, wie du es tust, zu kritisieren und im selben Atemzug um Hilfe zu bitten.

     Verwende einen PIC18F26K22 und habe auch mit 750ms und mit 750 µs getestet.

     Warum? Überall (natürlich auch im Datenblatt) ist deutlich ausgewiesen, dass 750 ms für eine Konvertierung benötigt werden. Demzufolge ist diese Zeile schon mal falsch:

     _delay_us(750); //warten auf Temp. auslesen, µs nicht ms!
     

     Ist das eine von dir selber geschriebene Funktion? Die offiziell verfügbaren Bibliotheksfunktionen fangen mit __ an, nicht mit _.

     Weiter geht es hier:

     ow_wr_bit(Convert_T);
     

     Das muss dann auch schon ow_wr_byte und nicht ow_wr_bit sein. Die direkt darauffolgende Zeile mit der falschen Wartezeit habe ich eingangs schon angesprochen. Btw.: Mindestens 750. Sprich, nimm einfach 800, dann bist du hier schon mal auf der sicheren Seite. Verkleinern kannst du diese Zeit dann immer noch, wenn es erstmal funktioniert.

     Wo soll das sein? Deswegen Umrechnung noch nicht weiter gemacht. Aber wo “Seite 79 unter der Überschrift “Die Umrechnung der Messwerte” ??? Die Links oben verweisen nur auf die Datenblätter und die haben nur 20 und nicht 79 Seiten!

     Im ersten Satz des Artikels sind drei PDF’s verlinkt (1. Datenblatt zum DS18S20 2. Datenblatt zum DS18B20 und 3. Ein allg. PDF mit Beschreibung der Algorithmen). Dann habe ich zum Thema Umrechnen der Temperatur geschrieben:

     Schaut hierfür einfach in das PDF Dokument, welches ganz oben auf dieser Seite verlinkt ist, auf der Seite 79 unter der Überschrift “Die Umrechnung der Messwerte” nach.

     So und nun wirst du es schaffen in einem der drei PDFs die Seite 79 zu finden? Falls nicht, hier der Link.

     Auch die unterschiedliche Schreibweise der Funktionen und des Array ist erst etwas verwirrend. Leider nicht unbedingt sehr sorgfältig erstellt und das lässt halt auch prinzipielle Fehler im Code vermuten, weswegen das so nicht geht.

     Meine erstellen Funktionen haben alle samt die identische Schreibweise mit folgendem Präfix ow_ und nachfolgender Beschreibung der Funktion z.B.: wr_byte. Du verwendest in deinem Code hingegen Defines wie Convert_T und Skip_ROM (Hinweis: Makros werden allg. üblich / guter Programmier-Stil in Uppercase geschrieben. So und du behauptest, dass mein Code unterschiedliche Schreibweise bei Funktionen aufweist? Ja ne ist klar 😉

     Desweiteren: Im von mir abgenoteten Sourcecode gibt es ein einziges Array uint8_t ds18x20 [9]. Was genau ist daran jetzt “verwirrend”?

     Viele Grüße,
     Nico

5. 

   Ottmar January 26, 2017 at 18:31

   Hallo,

   die Wartezeit = 750ms (MILLIsekunden!) – falsche Massbezeichnung! . Muss in MIKROsekunden angegeben werden.
   Ansonsten: Ein guter Artikel, verständlich, klar formuliert, hat sogar mir geholfen die DS18S20-Routine in Assembler zu programmieren.

   Weiter so! Mit besten Grüssen
   Ottmar

   • 

     NicoAuthor January 26, 2017 at 18:31

     Hallo Ottmar,

     vielen Dank für deinen Kommentar.
     Kannst Du bitte etwas präziser werden? Wo genau soll ein Fehler sein?
     Die Zeit, die der Sensor zur Konvertierung benötigt beträgt 750 ms (nicht µs), siehe Auszug aus dem Datenblatt:

     

     Viele Grüße
     Nico

6. 

   Killian Preh January 26, 2017 at 18:29

   Sehr geehrtes PIC-Projekte Team,

   Danke für das Grundgerüst! Jedoch fehlt mir noch die “Main” Funktion im Programm. Ich habe einen DS18s20 Temperatursensor. Ich müsste die Temperatur auf einem Display ausgeben. Brauchen Sie noch weitere Informationen?

   LG, Killian Preh

   • 

     NicoAuthor January 26, 2017 at 18:29

     Hallo Killian,

     wenn Du den Artikel und/oder das PDF aufmerksam gelesen hast, dann weißt Du bereits wie die Kommunikation aufgebaut werden muss. Die Situation vereinfacht sich, wenn sich nur ein Sensor am Bus befindet (was zu Beginn zwangsweise so sein muss).

     Um nun mit diesem Sensor Kontakt aufzunehmen wird als erstes ein Master-Reset-Impuls ausgelöst. Wenn der Rückgabewert der Funktion 0 ist, dann weißt Du, dass sich mindestens ein Slave am Bus befindet und Du kannst fortfahren. Wenn jedoch eine 1 zurück kommt, dann ist irgendwas im Aufbau schief gelaufen.

     Wie gesagt, wir gehen davon aus, dass sich nur ein Sensor am Bus befindet, daher brauchen wir keine Ansprache via Adresse durchzuführen, sondern können mit dem Skip-ROM-Befehl “alle” Sensoren am Bus ansprechen (in unserem Fall also der eine, der angeschlossen ist).

     Jetzt wird als nächstes der soeben angesprochene Sensor zur Temperaturmessung aufgefordert, indem wir den Befehl Convert T an den Sensor schicken (0x44).

     Wichtig: Jetzt musst Du mindestens 750 ms warten, bis Du die Temperatur auslesen kannst… Wenn die 750 ms vorüber sind, wird der Sensor mit dem Befehl 0xBE aufgefordert das Messergebnis auf den Bus auszugeben. Das Auslesen des Messergebnisses erfolgt dann in einer for Schleife (siehe Artikel).

     So jetzt hast Du die Rohdaten schon mal im Controller und musst sie “nur” noch verarbeiten. Ich habe in diesem Artikel noch ein zusätzliches PDF verlinkt, in dem auch beschrieben wird, wie man die Daten letztendlich auswertet (ist relativ einfach).

     Viele Grüße, Nico