Address PR feedback from @brentru

Author: tyeth

.github/workflows/build-clang-doxy.yml

@@ -38,14 +38,6 @@ jobs:
           ]
 
     steps:
-      - name: "skip if unwanted"
-        continue-on-error: true
-        if: |
-          github.event_name == 'workflow_dispatch' &&
-          github.event.inputs.board != '' &&
-          matrix.arduino-platform != github.event.inputs.board
-        run: |
-          echo "don't build this one!"; exit 1
       - uses: actions/setup-python@v5
         with:
           python-version: "3.x"
@@ -70,7 +62,7 @@ jobs:
       - name: Install extra Arduino libraries
         run: |
           git clone --quiet https://github.com/pstolarz/Arduino-Temperature-Control-Library.git /home/runner/Arduino/libraries/Arduino-Temperature-Control-Library
-          git clone --quiet --branch 2.2.54 https://github.com/adafruit/SdFat.git /home/runner/Arduino/libraries/SdFat
+          git clone --quiet https://github.com/adafruit/SdFat.git /home/runner/Arduino/libraries/SdFat
           git clone --quiet https://github.com/pstolarz/OneWireNg.git /home/runner/Arduino/libraries/OneWireNg
           git clone --quiet https://github.com/adafruit/Adafruit_HX8357_Library.git /home/runner/Arduino/libraries/Adafruit_HX8357_Library
           git clone --quiet https://github.com/adafruit/Adafruit_ILI9341.git /home/runner/Arduino/libraries/Adafruit_ILI9341

src/components/i2c/drivers/drvD6t1a.h

@@ -39,11 +39,6 @@ class drvD6t1a : public drvBase {
   drvD6t1a(TwoWire *i2c, uint16_t sensorAddress, uint32_t mux_channel,
            const char *driver_name)
       : drvBase(i2c, sensorAddress, mux_channel, driver_name) {
-    _i2c = i2c;
-    _address = sensorAddress;
-    _i2c_mux_channel = mux_channel;
-    strncpy(_name, driver_name, sizeof(_name) - 1);
-    _name[sizeof(_name) - 1] = '\0';
     _deviceTemp = NAN;
     _objectTemp = NAN;
     _lastRead = 0;
@@ -61,9 +56,7 @@ class drvD6t1a : public drvBase {
   */
   bool begin() override {
     _d6t1a = new OmronD6T(OmronD6T::D6T_1A, _i2c);
-    if (!_d6t1a->begin(_address))
-      return false;
-    return true;
+    return _d6t1a->begin(_address);
   }
 
   /*!

src/components/i2c/drivers/drvIna219.h

@@ -100,7 +100,7 @@ class drvIna219 : public drvBase {
   }
 
 protected:
-  Adafruit_INA219 *_ina219; ///< Pointer to INA219 sensor object
+  Adafruit_INA219 *_ina219 = nullptr; ///< Pointer to INA219 sensor object
 };
 
 #endif // drvIna219

src/components/i2c/drivers/drvIna228.h

@@ -83,7 +83,7 @@ class drvIna228 : public drvBase {
   void ConfigureDefaultSensorTypes() override;
 
 protected:
-  Adafruit_INA228 *_ina228; ///< Pointer to INA228 sensor object
+  Adafruit_INA228 *_ina228 = nullptr; ///< Pointer to INA228 sensor object
 };
 
 #endif // DRV_INA228_H

src/components/i2c/drivers/drvIna237.h

@@ -83,7 +83,7 @@ class drvIna237 : public drvBase {
   void ConfigureDefaultSensorTypes() override;
 
 protected:
-  Adafruit_INA237 *_ina237; ///< Pointer to INA237 sensor object
+  Adafruit_INA237 *_ina237 = nullptr; ///< Pointer to INA237 sensor object
 };
 
 #endif // DRV_INA237_H

src/components/i2c/drivers/drvIna238.h

@@ -83,7 +83,7 @@ class drvIna238 : public drvBase {
   void ConfigureDefaultSensorTypes() override;
 
 protected:
-  Adafruit_INA238 *_ina238; ///< Pointer to INA238 sensor object
+  Adafruit_INA238 *_ina238 = nullptr; ///< Pointer to INA238 sensor object
 };
 
 #endif // DRV_INA238_H

src/components/i2c/drivers/drvIna260.h

@@ -83,7 +83,7 @@ class drvIna260 : public drvBase {
   void ConfigureDefaultSensorTypes() override;
 
 protected:
-  Adafruit_INA260 *_ina260; ///< Pointer to INA260 sensor object
+  Adafruit_INA260 *_ina260 = nullptr; ///< Pointer to INA260 sensor object
 };
 
 #endif // DRV_INA260_H