OneWire issue?
Posted: Sun Sep 23, 2018 1:07 pmOK, so I bought a sonoff th16 with the intention of running my own firmware. I am using the DALLAS sensor ds18x20. With the original firmware, it reads temperature fine, so I know the sensor and board are fine.
I am running latest arduino IDE 1.8.7 and latest ESP arduino 2.4.2
I have the latest OneWire library 2.3 and latest DallasTemperature library (milesburton)
When I upload the firmware using DallasTemperature I only get back -127
If I use a demo that searched for addresses, I get back "No more addresses"
I checked that the io pin is gpio14, by runnning a sketch that tuens it high/low and see if it is being set. And it is, so I know it is gpio14.
I go back to the original firmware and all is fine.
I searched the forum for similar issues and in all the posts it was a wiring issue or bad pin or bad voltage. In my case it is not any of the, as original firmware works.
I want to use this sensor, as it is waterproof. But I do not want to use sonoff firmware, as I do not like cloud services and what I need this for is too important to use cloud and risk failure.
Any help/tips would be greatly appreciated
Here are the two sketches I tested with
Sketch that returns -127
This one prints "No more addresses"
I am running latest arduino IDE 1.8.7 and latest ESP arduino 2.4.2
I have the latest OneWire library 2.3 and latest DallasTemperature library (milesburton)
When I upload the firmware using DallasTemperature I only get back -127
If I use a demo that searched for addresses, I get back "No more addresses"
I checked that the io pin is gpio14, by runnning a sketch that tuens it high/low and see if it is being set. And it is, so I know it is gpio14.
I go back to the original firmware and all is fine.
I searched the forum for similar issues and in all the posts it was a wiring issue or bad pin or bad voltage. In my case it is not any of the, as original firmware works.
I want to use this sensor, as it is waterproof. But I do not want to use sonoff firmware, as I do not like cloud services and what I need this for is too important to use cloud and risk failure.
Any help/tips would be greatly appreciated
Here are the two sketches I tested with
Sketch that returns -127
Code: Select all
#include <ESP8266WiFi.h>
#include <OneWire.h>
#include <DallasTemperature.h>
#define ONE_WIRE_BUS 14
#define TEMPERATURE_PRECISION 9
#define loopmaxcount 10
OneWire oneWire(ONE_WIRE_BUS);
DallasTemperature sensors(&oneWire);
DeviceAddress ds18x20;
const int interval = 1000; // defines interval between DHT readout in sec
unsigned long currentread = 0;
unsigned long previousread = 0; // timestamp needed for DHT reading interval#
float temp = 0;
void setup() {
Serial.begin(9600);
delay(10);
sensors.begin();
// Search for devices on the bus and assign based on an index.
sensors.getAddress(ds18x20, 0);
// set the resolution to 9 bit per device
sensors.setResolution(ds18x20, TEMPERATURE_PRECISION);
}
void loop() {
currentread = millis();
if (currentread - previousread >= interval) {
previousread = currentread; // save the last time temp was read
sensors.requestTemperatures();
yield;
temp = sensors.getTempC(ds18x20);
Serial.print("Temp: ");
Serial.println(temp);
return;
}This one prints "No more addresses"
Code: Select all
#include <ESP8266WiFi.h>
#include <OneWire.h>
#define ONE_WIRE_BUS 14
// OneWire DS18S20, DS18B20, DS1822 Temperature Example
//
// http://www.pjrc.com/teensy/td_libs_OneWire.html
//
// The DallasTemperature library can do all this work for you!
// https://github.com/milesburton/Arduino-Temperature-Control-Library
OneWire ds(ONE_WIRE_BUS); // on pin 10 (a 4.7K resistor is necessary)
void setup(void) {
Serial.begin(9600);
}
void loop(void) {
byte i;
byte present = 0;
byte type_s;
byte data[12];
byte addr[8];
float celsius, fahrenheit;
if ( !ds.search(addr)) {
Serial.println("No more addresses.");
Serial.println();
ds.reset_search();
delay(250);
return;
}
Serial.print("ROM =");
for( i = 0; i < 8; i++) {
Serial.write(' ');
Serial.print(addr[i], HEX);
}
if (OneWire::crc8(addr, 7) != addr[7]) {
Serial.println("CRC is not valid!");
return;
}
Serial.println();
// the first ROM byte indicates which chip
switch (addr[0]) {
case 0x10:
Serial.println(" Chip = DS18S20"); // or old DS1820
type_s = 1;
break;
case 0x28:
Serial.println(" Chip = DS18B20");
type_s = 0;
break;
case 0x22:
Serial.println(" Chip = DS1822");
type_s = 0;
break;
default:
Serial.println("Device is not a DS18x20 family device.");
return;
}
ds.reset();
ds.select(addr);
ds.write(0x44, 1); // start conversion, with parasite power on at the end
delay(1000); // maybe 750ms is enough, maybe not
// we might do a ds.depower() here, but the reset will take care of it.
present = ds.reset();
ds.select(addr);
ds.write(0xBE); // Read Scratchpad
Serial.print(" Data = ");
Serial.print(present, HEX);
Serial.print(" ");
for ( i = 0; i < 9; i++) { // we need 9 bytes
data[i] = ds.read();
Serial.print(data[i], HEX);
Serial.print(" ");
}
Serial.print(" CRC=");
Serial.print(OneWire::crc8(data, 8), HEX);
Serial.println();
// Convert the data to actual temperature
// because the result is a 16 bit signed integer, it should
// be stored to an "int16_t" type, which is always 16 bits
// even when compiled on a 32 bit processor.
int16_t raw = (data[1] << 8) | data[0];
if (type_s) {
raw = raw << 3; // 9 bit resolution default
if (data[7] == 0x10) {
// "count remain" gives full 12 bit resolution
raw = (raw & 0xFFF0) + 12 - data[6];
}
} else {
byte cfg = (data[4] & 0x60);
// at lower res, the low bits are undefined, so let's zero them
if (cfg == 0x00) raw = raw & ~7; // 9 bit resolution, 93.75 ms
else if (cfg == 0x20) raw = raw & ~3; // 10 bit res, 187.5 ms
else if (cfg == 0x40) raw = raw & ~1; // 11 bit res, 375 ms
//// default is 12 bit resolution, 750 ms conversion time
}
celsius = (float)raw / 16.0;
fahrenheit = celsius * 1.8 + 32.0;
Serial.print(" Temperature = ");
Serial.print(celsius);
Serial.print(" Celsius, ");
Serial.print(fahrenheit);
Serial.println(" Fahrenheit");
}