uh, some stuff

CMakeLists.txt

@@ -9,17 +9,17 @@ set(SOURCE_FILES
         gex/serial/serial.h
         gex/gex_client.c
         gex/gex_client.h
-        gex/hexdump.c
+        gex/utils/hexdump.c
-        gex/hexdump.h
+        gex/utils/hexdump.h
-        gex/TF_Integration.c
+        gex/protocol/TF_Integration.c
-        gex/protocol/payload_builder.c
+        gex/utils/payload_builder.c
-        gex/protocol/payload_builder.h
+        gex/utils/payload_builder.h
-        gex/protocol/payload_parser.c
+        gex/utils/payload_parser.c
-        gex/protocol/payload_parser.h
+        gex/utils/payload_parser.h
         gex/protocol/TF_Config.h
         gex/protocol/TinyFrame.c
         gex/protocol/TinyFrame.h
-        gex/protocol/type_coerce.h
+        gex/utils/type_coerce.h
 )
 
 include_directories(

gex/gex_client.c

@@ -7,17 +7,16 @@
 #include <unistd.h>
 #include <signal.h>
 #include  <stdlib.h>
-#include <protocol/TinyFrame.h>
 #include <string.h>
 #include <errno.h>
 
+#include "TinyFrame.h"
 #include "gex_client.h"
 #include "serial.h"
-#include "hexdump.h"
 
 int gex_serial_fd = -1;
 
-
+/** ^C handler to close it gracefully */
 static void sigintHandler(int sig)
 {
 	if (gex_serial_fd != -1) {

gex/protocol/TF_Integration.c

@@ -6,24 +6,24 @@
 #include <assert.h>
 #include <errno.h>
 
-void TF_WriteImpl(const uint8_t *buff, size_t len)
+void TF_WriteImpl(TinyFrame *tf, const uint8_t *buff, size_t len)
 {
-	assert(gex_serial_fd != 0);
+	assert(gex_serial_fd != 0); // TODO update after TF has instances
 
-    ssize_t rv = write(gex_serial_fd, buff, len);
+	ssize_t rv = write(gex_serial_fd, buff, len);
 	if (rv != len) {
 		fprintf(stderr, "ERROR %d in TF write: %s\n", errno, strerror(errno));
 	}
 }
 
 /** Claim the TX interface before composing and sending a frame */
-void TF_ClaimTx(void)
+void TF_ClaimTx(TinyFrame *tf)
 {
     //
 }
 
 /** Free the TX interface after composing and sending a frame */
-void TF_ReleaseTx(void)
+void TF_ReleaseTx(TinyFrame *tf)
 {
     //
 }

gex/protocol/TinyFrame.h

@@ -10,12 +10,13 @@
  * Upstream URL: https://github.com/MightyPork/TinyFrame
  */
 
-#define TF_VERSION "1.2.0"
+#define TF_VERSION "2.0.0"
 
 //---------------------------------------------------------------------------
 #include <stdint.h>  // for uint8_t etc
 #include <stdbool.h> // for bool
-#include <stdlib.h>  // for NULL
+#include <stddef.h>  // for NULL
+#include <string.h>  // for memset()
 //---------------------------------------------------------------------------
 
 // Select checksum type (0 = none, 8 = ~XOR, 16 = CRC16 0x8005, 32 = CRC32)
@@ -24,54 +25,54 @@
 #define TF_CKSUM_CRC16 16
 #define TF_CKSUM_CRC32 32
 
-#include <TF_Config.h>
+#include "TF_Config.h"
 
 //region Resolve data types
 
 #if TF_LEN_BYTES == 1
-	typedef uint8_t TF_LEN;
+typedef uint8_t TF_LEN;
 #elif TF_LEN_BYTES == 2
-	typedef uint16_t TF_LEN;
+typedef uint16_t TF_LEN;
 #elif TF_LEN_BYTES == 4
-	typedef uint32_t TF_LEN;
+typedef uint32_t TF_LEN;
 #else
-	#error Bad value of TF_LEN_BYTES, must be 1, 2 or 4
+#error Bad value of TF_LEN_BYTES, must be 1, 2 or 4
 #endif
 
 
 #if TF_TYPE_BYTES == 1
-	typedef uint8_t TF_TYPE;
+typedef uint8_t TF_TYPE;
 #elif TF_TYPE_BYTES == 2
-	typedef uint16_t TF_TYPE;
+typedef uint16_t TF_TYPE;
 #elif TF_TYPE_BYTES == 4
-	typedef uint32_t TF_TYPE;
+typedef uint32_t TF_TYPE;
 #else
-	#error Bad value of TF_TYPE_BYTES, must be 1, 2 or 4
+#error Bad value of TF_TYPE_BYTES, must be 1, 2 or 4
 #endif
 
 
 #if TF_ID_BYTES == 1
-	typedef uint8_t TF_ID;
+typedef uint8_t TF_ID;
 #elif TF_ID_BYTES == 2
-	typedef uint16_t TF_ID;
+typedef uint16_t TF_ID;
 #elif TF_ID_BYTES == 4
-	typedef uint32_t TF_ID;
+typedef uint32_t TF_ID;
 #else
-	#error Bad value of TF_ID_BYTES, must be 1, 2 or 4
+#error Bad value of TF_ID_BYTES, must be 1, 2 or 4
 #endif
 
 
 #if TF_CKSUM_TYPE == TF_CKSUM_XOR || TF_CKSUM_TYPE == TF_CKSUM_NONE
-	// ~XOR (if 0, still use 1 byte - it won't be used)
+// ~XOR (if 0, still use 1 byte - it won't be used)
-	typedef uint8_t TF_CKSUM;
+typedef uint8_t TF_CKSUM;
 #elif TF_CKSUM_TYPE == TF_CKSUM_CRC16
-	// CRC16
+// CRC16
-	typedef uint16_t TF_CKSUM;
+typedef uint16_t TF_CKSUM;
 #elif TF_CKSUM_TYPE == TF_CKSUM_CRC32
-	// CRC32
+// CRC32
-	typedef uint32_t TF_CKSUM;
+typedef uint32_t TF_CKSUM;
 #else
-	#error Bad value for TF_CKSUM_TYPE, must be 8, 16 or 32
+#error Bad value for TF_CKSUM_TYPE, must be 8, 16 or 32
 #endif
 
 //endregion
@@ -86,26 +87,27 @@
 
 /** Peer bit enum (used for init) */
 typedef enum {
-	TF_SLAVE = 0,
+    TF_SLAVE = 0,
-	TF_MASTER,
+    TF_MASTER = 1,
 } TF_Peer;
 
+/** Response from listeners */
 typedef enum {
-	TF_NEXT = 0,   //!< Not handled, let other listeners handle it
+    TF_NEXT = 0,   //!< Not handled, let other listeners handle it
-	TF_STAY = 1,   //!< Handled, stay
+    TF_STAY = 1,   //!< Handled, stay
-	TF_RENEW = 2,  //!< Handled, stay, renew - useful only with listener timeout
+    TF_RENEW = 2,  //!< Handled, stay, renew - useful only with listener timeout
-	TF_CLOSE = 3,  //!< Handled, remove self
+    TF_CLOSE = 3,  //!< Handled, remove self
 } TF_Result;
 
 /** Data structure for sending / receiving messages */
 typedef struct _TF_MSG_STRUCT_ {
-	TF_ID frame_id;       //!< message ID
+    TF_ID frame_id;       //!< message ID
-	bool is_response;     //!< internal flag, set when using the Respond function. frame_id is then kept unchanged.
+    bool is_response;     //!< internal flag, set when using the Respond function. frame_id is then kept unchanged.
-	TF_TYPE type;         //!< received or sent message type
+    TF_TYPE type;         //!< received or sent message type
-	const uint8_t *data;  //!< buffer of received data or data to send. NULL = listener timed out, free userdata!
+    const uint8_t *data;  //!< buffer of received data or data to send. NULL = listener timed out, free userdata!
-	TF_LEN len;           //!< length of the buffer
+    TF_LEN len;           //!< length of the buffer
-	void *userdata;       //!< here's a place for custom data; this data will be stored with the listener
+    void *userdata;       //!< here's a place for custom data; this data will be stored with the listener
-	void *userdata2;
+    void *userdata2;
 } TF_Msg;
 
 /**
@@ -113,15 +115,11 @@ typedef struct _TF_MSG_STRUCT_ {
  */
 static inline void TF_ClearMsg(TF_Msg *msg)
 {
-	msg->frame_id = 0;
+    memset(msg, 0, sizeof(TF_Msg));
-	msg->is_response = false;
-	msg->type = 0;
-	msg->data = NULL;
-	msg->len = 0;
-	msg->userdata = NULL;
-	msg->userdata2 = NULL;
 }
 
+typedef struct TinyFrame_ TinyFrame;
+
 /**
  * TinyFrame Type Listener callback
  *
@@ -131,21 +129,117 @@ static inline void TF_ClearMsg(TF_Msg *msg)
  * @param len - number of bytes in the buffer
  * @return listener result
  */
-typedef TF_Result (*TF_Listener)(TF_Msg *msg);
+typedef TF_Result (*TF_Listener)(TinyFrame *tf, TF_Msg *msg);
+
+// -------------------------------------------------------------------
+
+// region Internal
+
+enum TFState_ {
+    TFState_SOF = 0,      //!< Wait for SOF
+    TFState_LEN,          //!< Wait for Number Of Bytes
+    TFState_HEAD_CKSUM,   //!< Wait for header Checksum
+    TFState_ID,           //!< Wait for ID
+    TFState_TYPE,         //!< Wait for message type
+    TFState_DATA,         //!< Receive payload
+    TFState_DATA_CKSUM    //!< Wait for Checksum
+};
+
+struct TF_IdListener_ {
+    TF_ID id;
+    TF_Listener fn;
+    TF_TICKS timeout;     // nr of ticks remaining to disable this listener
+    TF_TICKS timeout_max; // the original timeout is stored here
+    void *userdata;
+    void *userdata2;
+};
+
+struct TF_TypeListener_ {
+    TF_TYPE type;
+    TF_Listener fn;
+};
+
+struct TF_GenericListener_ {
+    TF_Listener fn;
+};
+
+/**
+ * Frame parser internal state.
+ */
+struct TinyFrame_ {
+    /* Public user data */
+    void *userdata;
+    uint32_t usertag;
+
+    // --- the rest of the struct is internal, do not access directly ---
+
+    /* Own state */
+    TF_Peer peer_bit;       //!< Own peer bit (unqiue to avoid msg ID clash)
+    TF_ID next_id;          //!< Next frame / frame chain ID
+
+    /* Parser state */
+    enum TFState_ state;
+    TF_TICKS parser_timeout_ticks;
+    TF_ID id;               //!< Incoming packet ID
+    TF_LEN len;             //!< Payload length
+    uint8_t data[TF_MAX_PAYLOAD_RX]; //!< Data byte buffer
+    TF_LEN rxi;             //!< Field size byte counter
+    TF_CKSUM cksum;         //!< Checksum calculated of the data stream
+    TF_CKSUM ref_cksum;     //!< Reference checksum read from the message
+    TF_TYPE type;           //!< Collected message type number
+    bool discard_data;      //!< Set if (len > TF_MAX_PAYLOAD) to read the frame, but ignore the data.
+
+    /* --- Callbacks --- */
+
+    /* Transaction callbacks */
+    struct TF_IdListener_ id_listeners[TF_MAX_ID_LST];
+    struct TF_TypeListener_ type_listeners[TF_MAX_TYPE_LST];
+    struct TF_GenericListener_ generic_listeners[TF_MAX_GEN_LST];
+
+    // Those counters are used to optimize look-up times.
+    // They point to the highest used slot number,
+    // or close to it, depending on the removal order.
+    TF_COUNT count_id_lst;
+    TF_COUNT count_type_lst;
+    TF_COUNT count_generic_lst;
+
+    // Buffer for building frames
+    uint8_t sendbuf[TF_SENDBUF_LEN];
+};
+
+// endregion
+
+// -------------------------------------------------------------------
 
 /**
  * Initialize the TinyFrame engine.
- * This can also be used to completely reset it (removing all listeners etc)
+ * This can also be used to completely reset it (removing all listeners etc).
+ *
+ * The field .userdata (or .usertag) can be used to identify different instances
+ * in the TF_WriteImpl() function etc. Set this field after the init.
+ *
+ * This function is a wrapper around TF_InitStatic that calls malloc() to obtain
+ * the instance.
  *
  * @param peer_bit - peer bit to use for self
  */
-void TF_Init(TF_Peer peer_bit);
+TinyFrame *TF_Init(TF_Peer peer_bit);
+
+
+/**
+ * Initialize the TinyFrame engine using a statically allocated instance struct.
+ *
+ * The .userdata / .usertag field is preserved when TF_InitStatic is called.
+ *
+ * @param peer_bit - peer bit to use for self
+ */
+void TF_InitStatic(TinyFrame *tf, TF_Peer peer_bit);
 
 /**
  * Reset the frame parser state machine.
  * This does not affect registered listeners.
  */
-void TF_ResetParser(void);
+void TF_ResetParser(TinyFrame *tf);
 
 /**
  * Register a frame type listener.
@@ -155,14 +249,14 @@ void TF_ResetParser(void);
  * @param timeout - timeout in ticks to auto-remove the listener (0 = keep forever)
  * @return slot index (for removing), or TF_ERROR (-1)
  */
-bool TF_AddIdListener(TF_Msg *msg, TF_Listener cb, TF_TICKS timeout);
+bool TF_AddIdListener(TinyFrame *tf, TF_Msg *msg, TF_Listener cb, TF_TICKS timeout);
 
 /**
  * Remove a listener by the message ID it's registered for
  *
  * @param frame_id - the frame we're listening for
  */
-bool TF_RemoveIdListener(TF_ID frame_id);
+bool TF_RemoveIdListener(TinyFrame *tf, TF_ID frame_id);
 
 /**
  * Register a frame type listener.
@@ -171,14 +265,14 @@ bool TF_RemoveIdListener(TF_ID frame_id);
  * @param cb - callback
  * @return slot index (for removing), or TF_ERROR (-1)
  */
-bool TF_AddTypeListener(TF_TYPE frame_type, TF_Listener cb);
+bool TF_AddTypeListener(TinyFrame *tf, TF_TYPE frame_type, TF_Listener cb);
 
 /**
  * Remove a listener by type.
  *
  * @param type - the type it's registered for
  */
-bool TF_RemoveTypeListener(TF_TYPE type);
+bool TF_RemoveTypeListener(TinyFrame *tf, TF_TYPE type);
 
 /**
  * Register a generic listener.
@@ -186,14 +280,14 @@ bool TF_RemoveTypeListener(TF_TYPE type);
  * @param cb - callback
  * @return slot index (for removing), or TF_ERROR (-1)
  */
-bool TF_AddGenericListener(TF_Listener cb);
+bool TF_AddGenericListener(TinyFrame *tf, TF_Listener cb);
 
 /**
  * Remove a generic listener by function pointer
  *
  * @param cb - callback function to remove
  */
-bool TF_RemoveGenericListener(TF_Listener cb);
+bool TF_RemoveGenericListener(TinyFrame *tf, TF_Listener cb);
 
 /**
  * Send a frame, no listener
@@ -201,17 +295,12 @@ bool TF_RemoveGenericListener(TF_Listener cb);
  * @param msg - message struct. ID is stored in the frame_id field
  * @return success
  */
-bool TF_Send(TF_Msg *msg);
+bool TF_Send(TinyFrame *tf, TF_Msg *msg);
 
 /**
  * Like TF_Send, but without the struct
  */
-bool TF_SendSimple(TF_TYPE type, const uint8_t *data, TF_LEN len);
+bool TF_SendSimple(TinyFrame *tf, TF_TYPE type, const uint8_t *data, TF_LEN len);
-
-/**
- * Like TF_Query, but without the struct
- */
-bool TF_QuerySimple(TF_TYPE type, const uint8_t *data, TF_LEN len, TF_Listener listener, TF_TICKS timeout, void *userdata);
 
 /**
  * Send a frame, and optionally attach an ID listener.
@@ -221,7 +310,13 @@ bool TF_QuerySimple(TF_TYPE type, const uint8_t *data, TF_LEN len, TF_Listener l
  * @param timeout - listener expiry time in ticks
  * @return success
  */
-bool TF_Query(TF_Msg *msg, TF_Listener listener, TF_TICKS timeout);
+bool TF_Query(TinyFrame *tf, TF_Msg *msg, TF_Listener listener, TF_TICKS timeout);
+
+/**
+ * Like TF_Query, but without the struct
+ */
+bool TF_QuerySimple(TinyFrame *tf, TF_TYPE type, const uint8_t *data, TF_LEN len,
+                    TF_Listener listener, TF_TICKS timeout);
 
 /**
  * Send a response to a received message.
@@ -229,15 +324,15 @@ bool TF_Query(TF_Msg *msg, TF_Listener listener, TF_TICKS timeout);
  * @param msg - message struct. ID is read from frame_id. set ->renew to reset listener timeout
  * @return success
  */
-bool TF_Respond(TF_Msg *msg);
+bool TF_Respond(TinyFrame *tf, TF_Msg *msg);
 
 /**
- * Renew ID listener timeout
+ * Renew an ID listener timeout externally (as opposed to by returning TF_RENEW from the ID listener)
  *
  * @param id - listener ID to renew
  * @return true if listener was found and renewed
  */
-bool TF_RenewIdListener(TF_ID id);
+bool TF_RenewIdListener(TinyFrame *tf, TF_ID id);
 
 /**
  * Accept incoming bytes & parse frames
@@ -245,21 +340,14 @@ bool TF_RenewIdListener(TF_ID id);
  * @param buffer - byte buffer to process
  * @param count - nr of bytes in the buffer
  */
-void TF_Accept(const uint8_t *buffer, size_t count);
+void TF_Accept(TinyFrame *tf, const uint8_t *buffer, size_t count);
 
 /**
  * Accept a single incoming byte
  *
  * @param c - a received char
  */
-void TF_AcceptChar(uint8_t c);
+void TF_AcceptChar(TinyFrame *tf, uint8_t c);
-
-/**
- * 'Write bytes' function that sends data to UART
- *
- * ! Implement this in your application code !
- */
-extern void TF_WriteImpl(const uint8_t *buff, size_t len);
 
 /**
  * This function should be called periodically.
@@ -269,12 +357,21 @@ extern void TF_WriteImpl(const uint8_t *buff, size_t len);
  *
  * (suggestion - call this in a SysTick handler)
  */
-void TF_Tick(void);
+void TF_Tick(TinyFrame *tf);
+
+// --- TO BE IMPLEMENTED BY USER ---
+
+/**
+ * 'Write bytes' function that sends data to UART
+ *
+ * ! Implement this in your application code !
+ */
+extern void TF_WriteImpl(TinyFrame *tf, const uint8_t *buff, size_t len);
 
 /** Claim the TX interface before composing and sending a frame */
-extern void TF_ClaimTx(void);
+extern void TF_ClaimTx(TinyFrame *tf);
 
 /** Free the TX interface after composing and sending a frame */
-extern void TF_ReleaseTx(void);
+extern void TF_ReleaseTx(TinyFrame *tf);
 
 #endif