search

Nodo-Domotica / Nodo-Core

Arduino based Domotica platform
http://www.nodo-domotica.nl
4 stars 8 forks source link

Dallas sensor type #629

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Berekening van temperatuur is verschillend voor de DS1820 en de DS18B20 
sensoren: /2 of *0,0625.
Verschil is te zien aan byte 0 van het adres: 0x10 is DS1820, 0x28 is DS18B20.
In de huidige code zie ik geen address door de rom-skip. En de berekening is nu 
hard coded....

Original issue reported on code.google.com by j.h....@hccnet.nl on 27 Nov 2012 at 11:31

GoogleCodeExporter commented 9 years ago 
Wil ik om redenen van performance en geheugen zo laten. De Dallas sensoren zijn 
traag en het uitvragen van de productcode vereist dat je de hele rom-scan 
uitvoert. Dit is een huge brok met code t.o.v. De rest. 

Dit is dus geen bug maar een bewuste keuze. De Dallas 1820 wordt in de 
fabrieksdocumentatie ook niet maar aangeraden voor nieuwe toepassingen (EOL). 
Daarom zal ik de routine default voor de DS18B20 maken. 

Als je de comment leest wordt ook duidelijk hoe het werkt. Dit is ook een 
voorbeeldimplemementatie voor een sensor.

Original comment by P.K.Ton...@gmail.com on 27 Nov 2012 at 12:28

GoogleCodeExporter commented 9 years ago 
Dan stel ik toch voor om de default inderdaad DS18B20 te maken, en via een 
#define de mogelijkheid van de DS1820 te bieden.
Doet de user helemaal niks, dan is het de default. Heeft hij nog een oude 
DS1820 liggen, dan kan hij die met de #define gebruiken.
Mogelijke code wordt dan:
    #ifdef DS1820    // use old DS1820 model
      return float(DSTemp)/2; // DS1820 variant
    #else            // use newer DS18B20 model
      return float(DSTemp)*0.0625; // DS18-B-20 variant
    #endif

Original comment by j.h....@hccnet.nl on 27 Nov 2012 at 3:01

GoogleCodeExporter commented 9 years ago 
Goed voorstel.

Original comment by P.K.Ton...@gmail.com on 27 Nov 2012 at 3:51

GoogleCodeExporter commented 9 years ago

Original comment by P.K.Ton...@gmail.com on 28 Nov 2012 at 10:08

GoogleCodeExporter commented 9 years ago

Original comment by P.K.Ton...@gmail.com on 28 Nov 2012 at 10:09

GoogleCodeExporter commented 9 years ago

Original comment by P.K.Ton...@gmail.com on 28 Nov 2012 at 10:10

GoogleCodeExporter commented 9 years ago 
zie ook issue 630

Original comment by j.h....@hccnet.nl on 29 Nov 2012 at 6:23

GoogleCodeExporter commented 9 years ago 
doen we niet, ds1820 not supported.

Original comment by j.h....@hccnet.nl on 30 Nov 2012 at 3:42

GoogleCodeExporter commented 9 years ago

Original comment by P.K.Ton...@gmail.com on 26 May 2013 at 8:55