Added isReadBusy() for Serial interfaces

src/helpers/ArduinoSerialInterface.cpp

@@ -17,6 +17,11 @@ bool ArduinoSerialInterface::isConnected() const {
   return true;   // no way of knowing, so assume yes
 }
 
+// PowerSaving variant
+bool ArduinoSerialInterface::isReadBusy() const {
+  return false;
+}
+
 bool ArduinoSerialInterface::isWriteBusy() const {
   return false;
 }

src/helpers/ArduinoSerialInterface.h

@@ -28,6 +28,9 @@ public:
 
   bool isConnected() const override;
 
+  // PowerSaving variant
+  bool isReadBusy() const override;
+
   bool isWriteBusy() const override;
   size_t writeFrame(const uint8_t src[], size_t len) override;
   size_t checkRecvFrame(uint8_t dest[]) override;

src/helpers/BaseSerialInterface.h

@@ -16,6 +16,9 @@ public:
   virtual bool isConnected() const = 0;
   virtual void loop() {};
 
+  // PowerSaving variant
+  virtual bool isReadBusy() const = 0;
+
   virtual bool isWriteBusy() const = 0;
   virtual size_t writeFrame(const uint8_t src[], size_t len) = 0;
   virtual size_t checkRecvFrame(uint8_t dest[]) = 0;

src/helpers/MultiSerialInterface.h

@@ -140,6 +140,23 @@ public:
     }
   }
 
+  bool isReadBusy() const override {
+    // not busy when disabled
+    if (!_enabled) {
+      return false;
+    }
+
+    // check if any interface is busy
+    for (auto iface : _interfaces) {
+      if (iface.instance && iface.instance->isEnabled() && iface.instance->isReadBusy()) {
+        return true;
+      }
+    }
+
+    // nothing busy
+    return false;
+  }
+
   bool isWriteBusy() const override {
     // not busy when disabled
     if(!_enabled){

src/helpers/esp32/SerialBLEInterface.cpp

@@ -188,6 +188,11 @@ size_t SerialBLEInterface::writeFrame(const uint8_t src[], size_t len) {
 
 #define  BLE_WRITE_MIN_INTERVAL   60
 
+// PowerSaving variant
+bool SerialBLEInterface::isReadBusy() const {
+  return uxQueueMessagesWaiting(recv_queue) > 0;
+}
+
 bool SerialBLEInterface::isWriteBusy() const {
   return millis() < _last_write + BLE_WRITE_MIN_INTERVAL;   // still too soon to start another write?
 }

src/helpers/esp32/SerialBLEInterface.h

@@ -82,6 +82,9 @@ public:
 
   bool isConnected() const override;
 
+  // PowerSaving variant
+  bool isReadBusy() const override;
+
   bool isWriteBusy() const override;
   size_t writeFrame(const uint8_t src[], size_t len) override;
   size_t checkRecvFrame(uint8_t dest[]) override;

src/helpers/nrf52/SerialBLEInterface.cpp

@@ -405,6 +405,11 @@ bool SerialBLEInterface::isConnected() const {
   return _isDeviceConnected && Bluefruit.connected() > 0;
 }
 
+// PowerSaving variant
+bool SerialBLEInterface::isReadBusy() const {
+  return (recv_queue_len > 0);
+}
+
 bool SerialBLEInterface::isWriteBusy() const {
   return send_queue_len >= (FRAME_QUEUE_SIZE * 2 / 3);
 }

src/helpers/nrf52/SerialBLEInterface.h

@@ -66,6 +66,7 @@ public:
   void disable() override;
   bool isEnabled() const override { return _isEnabled; }
   bool isConnected() const override;
+  bool isReadBusy() const override;
   bool isWriteBusy() const override;
   size_t writeFrame(const uint8_t src[], size_t len) override;
   size_t checkRecvFrame(uint8_t dest[]) override;

src/helpers/wifi/SerialWifiInterface.cpp

@@ -39,6 +39,11 @@ size_t SerialWifiInterface::writeFrame(const uint8_t src[], size_t len) {
   return 0;
 }
 
+// PowerSaving variant
+bool SerialWifiInterface::isReadBusy() const {
+  return false;
+}
+
 bool SerialWifiInterface::isWriteBusy() const {
   return false;
 }

src/helpers/wifi/SerialWifiInterface.h

@@ -52,6 +52,10 @@ public:
   bool isEnabled() const override { return _isEnabled; }
 
   bool isConnected() const override;
+
+  // PowerSaving variant
+  bool isReadBusy() const override;
+  
   bool isWriteBusy() const override;
 
   size_t writeFrame(const uint8_t src[], size_t len) override;