Fixes inPeriodic function

m5stack · Sep 28, 2024 · 5f0fac5 · 5f0fac5
1 parent 8b62e6b
commit 5f0fac5
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/M5UnitComponent.cpp b/src/M5UnitComponent.cpp
@@ -126,7 +126,9 @@ bool Component::assign(TwoWire& wire)
 
 bool Component::selectChannel(const uint8_t ch)
 {
-    M5_LIB_LOGV("%s:%u", deviceName(), ch);
+    if (ch != 255) {
+        M5_LIB_LOGV("%s:%u", deviceName(), ch);
+    }
     bool ret{true};
     if (hasParent()) {
         ret = _parent->selectChannel(channel());

diff --git a/src/M5UnitComponent.hpp b/src/M5UnitComponent.hpp
@@ -166,7 +166,7 @@ class Component {
     /*! @brief In periodic measurement? */
     inline bool inPeriodic() const
     {
-        return _periodic;
+        return in_periodic();
     }
     //! @brief Periodic measurement data updated?
     inline bool updated() const
@@ -345,6 +345,10 @@ class Component {
     virtual const char* unit_device_name() const = 0;
     virtual types::uid_t unit_identifier() const = 0;
     virtual types::attr_t unit_attribute() const = 0;
+    inline virtual bool in_periodic() const
+    {
+        return _periodic;
+    }
 
     // Duplicate the adapter for children
     // Note that ownership of the return pointer is delegated to the destination