I merge two codes, display module and temperature sensor, but it gives error in the temperature reading.
#include <OneWire.h>
#include <TM74HC595Display.h>
#include <TimerOne.h>
int DS18S20_Pin = 2; //DS18S20 Signal pin on digital 2
int SCLK = 7;
int RCLK = 6;
int DIO = 5;
TM74HC595Display disp(SCLK, RCLK, DIO);
//Temperature chip i/o
OneWire ds(DS18S20_Pin); // on digital pin 2
void setup(void) {
Timer1.initialize(1500); // set a timer of length 1500
Timer1.attachInterrupt(timerIsr); // attach the service routine here
Serial.begin(9600);
}
void loop(void) {
float temperature = getTemp(); //will take about 750ms to run
Serial.println(temperature);
disp.dispFloat(temperature, 2); //send float indicators
delay(10000);
//disp.clear(); //clear display
}
float getTemp(){
//returns the temperature from one DS18S20 in DEG Celsius
byte data[12];
byte addr[8];
if ( !ds.search(addr)) {
//no more sensors on chain, reset search
ds.reset_search();
return -1000;
}
if ( OneWire::crc8( addr, 7) != addr[7]) {
Serial.println("CRC is not valid!");
return -1000;
}
if ( addr[0] != 0x10 && addr[0] != 0x28) {
Serial.print("Device is not recognized");
return -1000;
}
ds.reset();
ds.select(addr);
ds.write(0x44,1); // start conversion, with parasite power on at the end
delay(750); // Wait for temperature conversion to complete
byte present = ds.reset();
ds.select(addr);
ds.write(0xBE); // Read Scratchpad
for (int i = 0; i < 9; i++) { // we need 9 bytes
data[i] = ds.read();
}
ds.reset_search();
byte MSB = data[1];
byte LSB = data[0];
float tempRead = ((MSB << 8) | LSB); //using two's compliment
float TemperatureSum = tempRead / 16;
return TemperatureSum;
}
void timerIsr()
{
disp.timerIsr();
}
Not sure which Arduino you're using, and what the libraries exactly do.
I might be wrong, but I would stay away from pin D2 and D3 (interrupt pins on an Uno).
Leo..