RPC: introduce SerialRPC abstraction over normal (non raw) channels

Author: facchinm

Diff for: libraries/RPC/SerialRPC.cpp

@@ -0,0 +1,3 @@
+#include "SerialRPC.h"
+
+arduino::SerialRPCClass SerialRPC;

Diff for: libraries/RPC/SerialRPC.h

@@ -0,0 +1,69 @@
+#include "RPC_internal.h"
+#include "Arduino.h"
+
+namespace arduino {
+
+class SerialRPCClass : public Stream {
+
+public:
+	SerialRPCClass() {};
+	void end() {};
+	int available(void) {
+		return rx_buffer.available();
+	};
+	int peek(void) {
+		return rx_buffer.peek();
+	}
+	int read(void) {
+		return rx_buffer.read_char();
+	}
+	void flush(void) {};
+
+	using Print::write; // pull in write(str) and write(buf, size) from Print
+
+	void onWrite(std::vector<uint8_t> vec) {
+	  for (int i = 0; i < vec.size(); i++) {
+	  	rx_buffer.store_char(vec[i]);
+	  }
+	  // call attached function
+	  if (_rx) {
+	    _rx.call();
+	  }
+	}
+
+	size_t write(uint8_t c) {
+		write(&c, 1);
+	}
+
+	size_t write(uint8_t* buf, size_t len) {
+		tx_buffer.clear();
+		for (int i=0; i<len; i++) {
+			tx_buffer.push_back(buf[i]);
+		}
+		RPC1.call("on_write", tx_buffer);
+	}
+
+	int begin() {
+		RPC1.begin();
+		RPC1.bind("on_write", mbed::callback(this, &SerialRPCClass::onWrite));
+	}
+
+	operator bool() {
+		return RPC1;
+	}
+
+    void attach(void (*fptr)(void))
+    {
+        if (fptr != NULL) {
+            _rx = mbed::Callback<void()>(fptr);
+        }
+    }
+
+private:
+   	mbed::Callback<void()> _rx;
+	RingBufferN<1024> rx_buffer;
+	std::vector<uint8_t> tx_buffer;
+};
+}
+
+extern arduino::SerialRPCClass SerialRPC;

Diff for: libraries/ThreadDebug/src/ThreadDebug.cpp

@@ -1188,10 +1188,10 @@ void UsbDebugCommInterface::attach(void (*pCallback)())
 
 #if defined(STM32H747xx) && defined(CORE_CM4)
 
-RPCDebugCommInterface::RPCDebugCommInterface(arduino::RPC* pSerial) :
+RPCDebugCommInterface::RPCDebugCommInterface(arduino::SerialRPCClass* pSerial) :
     _pSerial(pSerial)
 {
-    _pSerial->begin();
+    //_pSerial->begin();
 }
 
 RPCDebugCommInterface::~RPCDebugCommInterface()

Diff for: libraries/ThreadDebug/src/ThreadDebug.h

@@ -36,7 +36,7 @@
 
 #if defined(STM32H747xx) && defined(CORE_CM4)
 // include RPC out of arduino namespace
-#include "RPC_internal.h"
+#include "SerialRPC.h"
 #endif
 
 namespace arduino {
@@ -102,7 +102,7 @@ class UsbDebugCommInterface : public DebugCommInterface {
 // Use the RPC interface to communicate with GDB from M4 core
 class RPCDebugCommInterface : public DebugCommInterface {
 public:
-    RPCDebugCommInterface(arduino::RPC*);
+    RPCDebugCommInterface(arduino::SerialRPCClass*);
     virtual ~RPCDebugCommInterface();
 
     virtual bool readable();
@@ -112,7 +112,7 @@ class RPCDebugCommInterface : public DebugCommInterface {
     virtual void attach(void (*pCallback)());
 
 protected:
-    arduino::RPC*  _pSerial;
+    arduino::SerialRPCClass*  _pSerial;
 };
 #endif