eRPC updates 05/2021

-- Update erpcgen and unit tests to cover the case when enum member value is negative (Github #159 PR / #41 issue solving).
-- Update (D)SPI transport layers to avoid busy loops in rtos environments, introduce semaphores.
-- Introduced new transports: i2c_slave_transport (C/C++), LIBUSBSIOSPITransport (Python), LIBUSBSIOI2CTransport (Python).
-- Remove const qualifier from InterThreadBufferTransport::send method parameter to align with parent Transport class (GitHub issue #172).
-- Allow conditional compilation of message_loggers and pre_post_action to help to reduce the code size when compiler does not correctly strip unused code.
-- Minor documentation updates.
-- Version changed to 1.8.1.

Author: MichalPrincNXP

README.md

@@ -194,4 +194,4 @@ Repository on Github contains two main branches. __Master__ and __develop__. Cod
 ---
 Copyright 2014-2016 Freescale Semiconductor, Inc.
 
-Copyright 2016-2020 NXP
+Copyright 2016-2021 NXP

doxygen/Doxyfile.erpc

@@ -38,7 +38,7 @@ PROJECT_NAME           = "eRPC API Reference"
 # could be handy for archiving the generated documentation or if some version
 # control system is used.
 
-PROJECT_NUMBER         = "Rev. 1.8.0"
+PROJECT_NUMBER         = "Rev. 1.8.1"
 
 # Using the PROJECT_BRIEF tag one can provide an optional one line description
 # for a project that appears at the top of each page and should give viewer a

doxygen/Doxyfile.erpcgen

@@ -38,7 +38,7 @@ PROJECT_NAME           = "eRPC Generator (erpcgen)"
 # could be handy for archiving the generated documentation or if some version
 # control system is used.
 
-PROJECT_NUMBER         = "Rev. 1.8.0"
+PROJECT_NUMBER         = "Rev. 1.8.1"
 
 # Using the PROJECT_BRIEF tag one can provide an optional one line description
 # for a project that appears at the top of each page and should give viewer a

erpc_c/infra/erpc_message_buffer.h

@@ -385,7 +385,7 @@ class MessageBufferFactory
     virtual MessageBuffer create(void) = 0;
 
     /*!
-     * @brief This function inform server if it has to create buffer for received message.
+     * @brief This function informs server if it has to create buffer for received message.
      *
      * @return Has to return TRUE when server need create buffer for receiving message.
      */

erpc_c/infra/erpc_message_loggers.cpp

@@ -1,11 +1,13 @@
 /*
- * Copyright 2017 NXP
+ * Copyright 2017-2021 NXP
  * Copyright 2021 ACRIOS Systems s.r.o.
  * All rights reserved.
  *
  *
  * SPDX-License-Identifier: BSD-3-Clause
  */
+#include "erpc_config_internal.h"
+#if ERPC_MESSAGE_LOGGING
 
 #include "erpc_message_loggers.h"
 
@@ -82,3 +84,4 @@ erpc_status_t MessageLoggers::logMessage(MessageBuffer *msg)
 
     return err;
 }
+#endif /* ERPC_MESSAGE_LOGGING */

erpc_c/infra/erpc_pre_post_action.cpp

@@ -1,5 +1,5 @@
 /*
- * Copyright 2020 NXP
+ * Copyright 2020-2021 NXP
  * Copyright 2020 ACRIOS Systems s.r.o.
  * Copyright 2021 ACRIOS Systems s.r.o.
  * All rights reserved.
@@ -8,9 +8,10 @@
  * SPDX-License-Identifier: BSD-3-Clause
  */
 
-#include "erpc_pre_post_action.h"
-
 #include "erpc_config_internal.h"
+#if ERPC_PRE_POST_ACTION
+
+#include "erpc_pre_post_action.h"
 #if ERPC_PRE_POST_ACTION_DEFAULT
 #include "erpc_setup_extensions.h"
 #endif
@@ -49,3 +50,4 @@ void PrePostAction::addPostCB(pre_post_action_cb postCB)
     }
 #endif
 }
+#endif /* ERPC_PRE_POST_ACTION */

erpc_c/infra/erpc_transport_arbitrator.h

@@ -119,7 +119,7 @@ class TransportArbitrator : public Transport
      *
      * @param[in] crcImpl Object containing crc-16 compute function.
      */
-    virtual void setCrc16(Crc16 *crcImpl);
+    virtual void setCrc16(Crc16 *crcImpl) override;
 
 protected:
     Transport *m_sharedTransport; //!< Transport being shared through this arbitrator.

erpc_c/infra/erpc_version.h

@@ -1,6 +1,6 @@
 /*
  * Copyright (c) 2016, Freescale Semiconductor, Inc.
- * Copyright 2016-2020 NXP
+ * Copyright 2016-2021 NXP
  * All rights reserved.
  *
  *
@@ -20,9 +20,9 @@
 ////////////////////////////////////////////////////////////////////////////////
 
 //! @brief String version of eRPC.
-#define ERPC_VERSION "1.8.0"
+#define ERPC_VERSION "1.8.1"
 //! @brief Integer version of eRPC.
-#define ERPC_VERSION_NUMBER 10800
+#define ERPC_VERSION_NUMBER 10801
 
 /*! @} */
 

erpc_c/setup/erpc_setup_i2c_slave.cpp

@@ -0,0 +1,30 @@
+/*
+ * Copyright 2021 NXP
+ * All rights reserved.
+ *
+ *
+ * SPDX-License-Identifier: BSD-3-Clause
+ */
+
+#include "erpc_i2c_slave_transport.h"
+#include "erpc_manually_constructed.h"
+#include "erpc_transport_setup.h"
+
+using namespace erpc;
+
+////////////////////////////////////////////////////////////////////////////////
+// Variables
+////////////////////////////////////////////////////////////////////////////////
+
+static ManuallyConstructed<I2cSlaveTransport> s_transport;
+
+////////////////////////////////////////////////////////////////////////////////
+// Code
+////////////////////////////////////////////////////////////////////////////////
+
+erpc_transport_t erpc_transport_i2c_slave_init(void *baseAddr, uint32_t baudRate, uint32_t srcClock_Hz)
+{
+    s_transport.construct((I2C_Type *)baseAddr, baudRate, srcClock_Hz);
+    s_transport->init();
+    return reinterpret_cast<erpc_transport_t>(s_transport.get());
+}

erpc_c/setup/erpc_transport_setup.h

@@ -1,6 +1,6 @@
 /*
  * Copyright (c) 2014-2016, Freescale Semiconductor, Inc.
- * Copyright 2016-2020 NXP
+ * Copyright 2016-2021 NXP
  * Copyright 2019 ACRIOS Systems s.r.o.
  * All rights reserved.
  *
@@ -356,6 +356,23 @@ erpc_transport_t erpc_transport_usb_cdc_init(void *serialHandle, void *serialCon
                                              uint8_t *usbRingBuffer, uint32_t usbRingBufferLength);
 //@}
 
+//! @name I2C transport setup
+//@{
+
+/*!
+ * @brief Create an I2C slave transport.
+ *
+ * Create I2C slave transport instance, to be used at slave core.
+ *
+ * @param[in] baseAddr Base address of I2C peripheral used in this transport layer.
+ * @param[in] baudRate SPI baud rate.
+ * @param[in] srcClock_Hz I2C source clock in Hz.
+ *
+ * @return Return NULL or erpc_transport_t instance pointer.
+ */
+erpc_transport_t erpc_transport_i2c_slave_init(void *baseAddr, uint32_t baudRate, uint32_t srcClock_Hz);
+//@}
+
 //@}
 
 #ifdef __cplusplus