Added support for the TMP102 sensor. Unfortunately, this will almost

certainly cause the ADS1015 chip to be incorrectly detected and treated
as a TMP102 sensor.  Ideally I'll find a way to distinguish between
the two based upon their behavior in a future commit.

Author: summetj

CHANGELOG.md

@@ -1,5 +1,7 @@
 Change Log
 ======================
+v2.9:
+* Adds support for the TMP102 temperature sensor
 
 v2.8:
 ---------

Firmware/OpenLog_Artemis/OpenLog_Artemis.ino

@@ -18,6 +18,7 @@
    10Hz logging IMU, no Qwiic, SD, no USB, no Power LED: 9-27mA
 
   TODO:
+  Add support for TMP102 temperature sensor (preferably while still working with the ADS1015 (also at address 0x48)
   (done) Create settings file for sensor. Load after qwiic bus is scanned.
   (done on larger Strings) Remove String dependencies.
   (done) Bubble sort list of devices.
@@ -160,10 +161,12 @@
     The charsReceived debug print ("Total chars received: ") now excludes the length of the timestamps
     Consistent use of File32/ExFile/FsFile/File. Don't use SdFile for temporary files
 
+  v2.9
+    Adds support for TMP102 low(er) cost temperature sensor
 */
 
 const int FIRMWARE_VERSION_MAJOR = 2;
-const int FIRMWARE_VERSION_MINOR = 8;
+const int FIRMWARE_VERSION_MINOR = 9;
 
 //Define the OLA board identifier:
 //  This is an int which is unique to this variant of the OLA and which allows us
@@ -315,6 +318,7 @@ icm_20948_DMP_data_t dmpData; // Global storage for the DMP data - extracted fro
 #include "SparkFun_SGP30_Arduino_Library.h" //Click here to get the library: http://librarymanager/All#SparkFun_SGP30
 #include "SparkFun_VCNL4040_Arduino_Library.h" //Click here to get the library: http://librarymanager/All#SparkFun_VCNL4040
 #include "SparkFun_MS5637_Arduino_Library.h" //Click here to get the library: http://librarymanager/All#SparkFun_MS5637
+#include "SparkFunTMP102.h" //Click here to get the library: http://librarymanager/All#SparkFun_TMP102
 #include "SparkFun_TMP117.h" //Click here to get the library: http://librarymanager/All#SparkFun_TMP117
 #include "SparkFun_u-blox_GNSS_Arduino_Library.h" //http://librarymanager/All#SparkFun_u-blox_GNSS
 #include "SparkFun_Qwiic_Scale_NAU7802_Arduino_Library.h" //Click here to get the library: http://librarymanager/All#SparkFun_NAU7802

Firmware/OpenLog_Artemis/Sensors.ino

@@ -434,6 +434,20 @@ void gatherDeviceValues(char * sdOutputData, size_t lenData)
             }
           }
           break;
+        case DEVICE_TEMPERATURE_TMP102:
+          {
+            TMP102 *nodeDevice = (TMP102 *)temp->classPtr;
+            struct_TMP102 *nodeSetting = (struct_TMP102 *)temp->configPtr;
+
+            if (nodeSetting->log == true)
+            {
+                olaftoa(nodeDevice->readTempC(), tempData1, 4, sizeof(tempData1) / sizeof(char)); //Resolution to 0.0625°C, ACCURACY: 0.5°C (–25°C to +85°C)
+                sprintf(tempData, "%s,", tempData1);
+                strlcat(sdOutputData, tempData, lenData);
+              
+            }
+          }
+          break;     
         case DEVICE_TEMPERATURE_TMP117:
           {
             TMP117 *nodeDevice = (TMP117 *)temp->classPtr;
@@ -1465,6 +1479,15 @@ static void getHelperText(char* helperText, size_t lenText)
             }
           }
           break;
+        case DEVICE_TEMPERATURE_TMP102:
+          {
+            struct_TMP102 *nodeSetting = (struct_TMP102 *)temp->configPtr;
+            if (nodeSetting->log)
+            {
+                strlcat(helperText, "TMP102 degC,", lenText);
+            }
+          }
+          break;  
         case DEVICE_TEMPERATURE_TMP117:
           {
             struct_TMP117 *nodeSetting = (struct_TMP117 *)temp->configPtr;

Firmware/OpenLog_Artemis/autoDetect.ino

@@ -147,6 +147,12 @@ bool addDevice(deviceType_e deviceType, uint8_t address, uint8_t muxAddress, uin
         temp->configPtr = new struct_VCNL4040;
       }
       break;
+    case DEVICE_TEMPERATURE_TMP102:
+      {
+        temp->classPtr = new TMP102;
+        temp->configPtr = new struct_TMP102;
+      }
+      break;
     case DEVICE_TEMPERATURE_TMP117:
       {
         temp->classPtr = new TMP117;
@@ -402,6 +408,14 @@ bool beginQwiicDevices()
           temp->online = tempDevice->begin(qwiic); //Wire port
         }
         break;
+      case DEVICE_TEMPERATURE_TMP102:
+        {
+          TMP102 *tempDevice = (TMP102 *)temp->classPtr;
+          struct_TMP102 *nodeSetting = (struct_TMP102 *)temp->configPtr; //Create a local pointer that points to same spot as node does
+          if (nodeSetting->powerOnDelayMillis > qwiicPowerOnDelayMillis) qwiicPowerOnDelayMillis = nodeSetting->powerOnDelayMillis; // Increase qwiicPowerOnDelayMillis if required
+          temp->online = tempDevice->begin(temp->address, qwiic); //Address, Wire port
+        }
+        break;
       case DEVICE_TEMPERATURE_TMP117:
         {
           TMP117 *tempDevice = (TMP117 *)temp->classPtr;
@@ -819,6 +833,18 @@ void configureDevice(node * temp)
         sensor->setAmbientIntegrationTime(sensorSetting->ambientIntegrationTime);
       }
       break;
+
+    case DEVICE_TEMPERATURE_TMP102:
+      {
+        TMP102 *sensor = (TMP102 *)temp->classPtr;
+        struct_TMP102 *sensorSetting = (struct_TMP102 *)temp->configPtr;
+
+        // JWS TODO replace with initalization code for TMP102 if needed?
+        //sensor->setConversionAverageMode(sensorSetting->conversionAverageMode);
+        //sensor->setConversionCycleBit(sensorSetting->conversionCycle);
+        //sensor->setContinuousConversionMode();
+      }
+      break;
     case DEVICE_TEMPERATURE_TMP117:
       {
         TMP117 *sensor = (TMP117 *)temp->classPtr;
@@ -1106,6 +1132,9 @@ FunctionPointer getConfigFunctionPtr(uint8_t nodeNumber)
     case DEVICE_PROXIMITY_VCNL4040:
       ptr = (FunctionPointer)menuConfigure_VCNL4040;
       break;
+    case DEVICE_TEMPERATURE_TMP102:
+      ptr = (FunctionPointer)menuConfigure_TMP102;
+      break;
     case DEVICE_TEMPERATURE_TMP117:
       ptr = (FunctionPointer)menuConfigure_TMP117;
       break;
@@ -1324,6 +1353,7 @@ void swap(struct node * a, struct node * b)
 #define ADR_MS8607 0x40 //Humidity portion of the MS8607 sensor
 #define ADR_UBLOX 0x42 //But can be set to any address
 #define ADR_ADS122C04 0x45 //Alternates: 0x44, 0x41 and 0x40
+#define ADR_TMP102 0x48 //Alternates: 0x49, 0x4A, and 0x4B
 #define ADR_TMP117 0x48 //Alternates: 0x49, 0x4A, and 0x4B
 #define ADR_ADS1015 0x48 //Alternates: 0x49, 0x4A, and 0x4B
 #define ADR_BIO_SENSOR_HUB 0x55
@@ -1488,6 +1518,17 @@ deviceType_e testDevice(uint8_t i2cAddress, uint8_t muxAddress, uint8_t portNumb
         if (sensor.begin(i2cAddress, qwiic) == true) //Address, Wire port
           return (DEVICE_TEMPERATURE_TMP117);
 
+        //Confidence: Low - only does a simple isConnected. Not a problem with TMP117, but it
+        // will probably detect a ADS1015 as a TMP102.  We need to figure out a way to discriminate between
+        // those two devices (when a TMP102 is misdetected as a ADS1015, all 4 values are always equal, maybe this will help?)
+        // For development purposes, listing before the ADS1015 so that I can develop/test and make my TMP102's work.
+        TMP102 sensor2;
+        if (sensor2.begin(i2cAddress, qwiic) == true) //Address, Wire port
+          return (DEVICE_TEMPERATURE_TMP102);
+
+        // JWS WARNING - shadowed by above TMP102 test, need to find out a way to discriminate between the two chips via their behavior, OR
+        // allow user to select which it is via menu.
+        //
         //Confidence: Low - only does a simple isConnected
         ADS1015 sensor1;
         if (sensor1.begin(i2cAddress, qwiic) == true) //Address, Wire port
@@ -1889,6 +1930,9 @@ const char* getDeviceName(deviceType_e deviceNumber)
     case DEVICE_PROXIMITY_VCNL4040:
       return "Proximity-VCNL4040";
       break;
+    case DEVICE_TEMPERATURE_TMP102:
+      return "Temperature-TMP102";
+      break;
     case DEVICE_TEMPERATURE_TMP117:
       return "Temperature-TMP117";
       break;

Firmware/OpenLog_Artemis/menuAttachedDevices.ino

@@ -297,6 +297,9 @@ void menuAttachedDevices()
           case DEVICE_PROXIMITY_VCNL4040:
             SerialPrintf3("%s VCNL4040 Proximity Sensor %s\r\n", strDeviceMenu, strAddress);
             break;
+          case DEVICE_TEMPERATURE_TMP102:
+            SerialPrintf3("%s TMP102 Temperature Sensor %s\r\n", strDeviceMenu, strAddress);
+            break;
           case DEVICE_TEMPERATURE_TMP117:
             SerialPrintf3("%s TMP117 High Precision Temperature Sensor %s\r\n", strDeviceMenu, strAddress);
             break;
@@ -1643,6 +1646,35 @@ void menuConfigure_VCNL4040(void *configPtr)
 
 }
 
+
+void menuConfigure_TMP102(void *configPtr)
+{
+  struct_TMP102 *sensorSetting = (struct_TMP102*)configPtr;
+  while (1)
+  {
+    SerialPrintln(F(""));
+    SerialPrintln(F("Menu: Configure TMP102 Temperature Sensor"));
+
+    SerialPrint(F("1) Sensor Logging: "));
+    if (sensorSetting->log == true) SerialPrintln(F("Enabled"));
+    else SerialPrintln(F("Disabled"));
+
+    SerialPrintln(F("x) Exit"));
+
+    byte incoming = getByteChoice(menuTimeout); //Timeout after x seconds
+
+    if (incoming == '1')
+      sensorSetting->log ^= 1;
+    else if (incoming == 'x')
+      break;
+    else if (incoming == STATUS_GETBYTE_TIMEOUT)
+      break;
+    else
+      printUnknown(incoming);
+  }
+
+}
+
 void menuConfigure_TMP117(void *configPtr)
 {
   struct_TMP117 *sensorSetting = (struct_TMP117*)configPtr;

Firmware/OpenLog_Artemis/nvm.ino

@@ -587,11 +587,17 @@ void recordDeviceSettingsToFile()
             settingsFile.println((String)base + "resolution=" + nodeSetting->resolution);
           }
           break;
+        case DEVICE_TEMPERATURE_TMP102:
+          {
+            struct_TMP102 *nodeSetting = (struct_TMP102 *)temp->configPtr;
+            settingsFile.println((String)base + "log=" + nodeSetting->log);
+          }
+          break;
         case DEVICE_TEMPERATURE_TMP117:
           {
             struct_TMP117 *nodeSetting = (struct_TMP117 *)temp->configPtr;
             settingsFile.println((String)base + "log=" + nodeSetting->log);
-            settingsFile.println((String)base + "logTemperature=" + nodeSetting->logTemperature);
+            settingsFile.println((String)base + "logTemperature=" + nodeSetting->logTemperature); // JWS: I don't think this value is ever changed?
           }
           break;
         case DEVICE_PRESSURE_MS5637:
@@ -1145,13 +1151,22 @@ bool parseDeviceLine(char* str) {
             SerialPrintf2("Unknown device setting: %s\r\n", deviceSettingName);
         }
         break;
+      case DEVICE_TEMPERATURE_TMP102:
+        {
+          struct_TMP102 *nodeSetting = (struct_TMP102 *)deviceConfigPtr; //Create a local pointer that points to same spot as node does
+          if (strcmp(deviceSettingName, "log") == 0)
+            nodeSetting->log = d;
+          else
+            SerialPrintf2("Unknown device setting: %s\r\n", deviceSettingName);
+        }
+        break;
       case DEVICE_TEMPERATURE_TMP117:
         {
           struct_TMP117 *nodeSetting = (struct_TMP117 *)deviceConfigPtr; //Create a local pointer that points to same spot as node does
           if (strcmp(deviceSettingName, "log") == 0)
             nodeSetting->log = d;
           else if (strcmp(deviceSettingName, "logTemperature") == 0)
-            nodeSetting->logTemperature = d;
+            nodeSetting->logTemperature = d;  // JWS: I don't think this value is ever used??
           else
             SerialPrintf2("Unknown device setting: %s\r\n", deviceSettingName);
         }

Firmware/OpenLog_Artemis/settings.h

@@ -8,6 +8,7 @@ typedef enum
   DEVICE_DISTANCE_VL53L1X,
   DEVICE_GPS_UBLOX,
   DEVICE_PROXIMITY_VCNL4040,
+  DEVICE_TEMPERATURE_TMP102,
   DEVICE_TEMPERATURE_TMP117,
   DEVICE_PRESSURE_MS5637,
   DEVICE_PRESSURE_LPS25HB,
@@ -170,12 +171,18 @@ struct struct_VL53L1X {
   unsigned long powerOnDelayMillis = minimumQwiicPowerOnDelay; // Wait for at least this many millis before communicating with this device. Increase if required!
 };
 
+struct struct_TMP102 {
+  bool log = true;
+  unsigned long powerOnDelayMillis = minimumQwiicPowerOnDelay; // Wait for at least this many millis before communicating with this device. Increase if required!
+};
+
+
 #define TMP117_MODE_CONTINUOUS 0
 #define TMP117_MODE_SHUTDOWN 1
 #define TMP117_MODE_ONESHOT 2
 struct struct_TMP117 {
   bool log = true;
-  bool logTemperature= true;
+  bool logTemperature= true;      // JWS : This value is set to true here, and then ever changed ever again???
   int conversionMode = TMP117_MODE_CONTINUOUS;
   int conversionAverageMode = 0; //Setup for 15.5ms reads
   int conversionCycle = 0;

README.md

@@ -36,6 +36,7 @@ The OpenLog Artemis automatically scans, detects, configures, and logs various Q
 * [SDP3X Differential Pressure Sensor](https://www.sparkfun.com/products/17874)
 * [MS8607 Pressure Humidity Temperature Sensor](https://www.sparkfun.com/products/16298)
 * [MPR0025PA MicroPressure Sensor](https://www.sparkfun.com/products/16476)
+* [TMP102 Temperature Sensor] (https://www.sparkfun.com/products/13314)
 * [TMP117 High Precision Temperature Sensor](https://www.sparkfun.com/products/15805)
 * [AHT20 Humidity and Temperature Sensor](https://www.sparkfun.com/products/16618)
 * [SHTC3 Humidity and Temperature Sensor](https://www.sparkfun.com/products/16467)

SENSOR_UNITS.md

@@ -49,6 +49,7 @@ This document summarizes the units used for each sensor measurement.
 - [MCP9600 thermocouple amplifier](#MCP9600-thermocouple-amplifier)
 - [Qwiic PT100 ADS122C04 platinum resistance sensor](#Qwiic-PT100-ADS122C04-platinum-resistance-sensor)
 - [TMP117 precision temperature sensor](#TMP117-precision-temperature-sensor)
+- [TMP102 temperature sensor](#TMP102-temperature-sensor)
 
 ### Weight:
 
@@ -350,6 +351,13 @@ Fan operational status:
 | Temperature Internal | degC | Degrees Centigrade |
 | Raw Voltage | V*2.048/2^23 | Volts * 2.048 / 2<sup>23</sup> |
 
+---
+## TMP102 temperature sensor
+
+| []() | | |
+|---|---|---|
+| Temperature | degC | Degrees Centigrade |
+
 ---
 ## TMP117 precision temperature sensor