Release v6.0.1

Author: ALAASTM

Interfaces/Patterns/COMMON/common_interface.c

@@ -82,7 +82,7 @@ FlagStatus Common_GetProtectionStatus(void)
 
 /**
   * @brief  Register a callback function to be called at the end of commands processing.
-  * @retval None
+  * @retval None.
   */
 void Common_SetPostProcessingCallback(Function_Pointer Callback)
 {
@@ -91,7 +91,7 @@ void Common_SetPostProcessingCallback(Function_Pointer Callback)
 
 /**
   * @brief  Start post processing task.
-  * @retval None
+  * @retval None.
   */
 void Common_StartPostProcessing()
 {

Interfaces/Patterns/COMMON/common_interface.h

@@ -20,6 +20,10 @@
 #ifndef COMMON_INTERFACE_H
 #define COMMON_INTERFACE_H
 
+#ifdef __cplusplus
+extern "C" {
+#endif
+
 /* Includes ------------------------------------------------------------------*/
 /* Exported types ------------------------------------------------------------*/
 typedef void (*Function_Pointer)(void);
@@ -34,4 +38,8 @@ FlagStatus Common_GetProtectionStatus(void);
 void Common_SetPostProcessingCallback(Function_Pointer Callback);
 void Common_StartPostProcessing(void);
 
+#ifdef __cplusplus
+}
+#endif
+
 #endif /* COMMON_INTERFACE_H */

Interfaces/Patterns/ENGI BYTES/engibytes_interface.c

@@ -33,7 +33,7 @@ OPENBL_MemoryTypeDef EB_Descriptor =
 {
   EB_START_ADDRESS,
   EB_END_ADDRESS,
-  (64),
+  EB_SIZE,
   EB_AREA,
   OPENBL_EB_Read,
   NULL,

Interfaces/Patterns/ENGI BYTES/engibytes_interface.h

@@ -20,11 +20,19 @@
 #ifndef ENGIBYTES_INTERFACE_H
 #define ENGIBYTES_INTERFACE_H
 
+#ifdef __cplusplus
+extern "C" {
+#endif
+
 /* Includes ------------------------------------------------------------------*/
 /* Exported types ------------------------------------------------------------*/
 /* Exported constants --------------------------------------------------------*/
 /* Exported macro ------------------------------------------------------------*/
 /* Exported functions ------------------------------------------------------- */
 uint8_t OPENBL_EB_Read(uint32_t Address);
 
+#ifdef __cplusplus
+}
+#endif
+
 #endif /* ENGIBYTES_INTERFACE_H */

Interfaces/Patterns/FDCAN/fdcan_interface.c

@@ -18,20 +18,20 @@
 
 /* Includes ------------------------------------------------------------------*/
 #include "platform.h"
+#include "interfaces_conf.h"
 #include "openbl_core.h"
 #include "openbl_fdcan_cmd.h"
 #include "fdcan_interface.h"
 #include "iwdg_interface.h"
-#include "interfaces_conf.h"
 
 /* Private typedef -----------------------------------------------------------*/
 /* Private define ------------------------------------------------------------*/
 /* Private macro -------------------------------------------------------------*/
 /* Private variables ---------------------------------------------------------*/
-FDCAN_HandleTypeDef hfdcan;
-FDCAN_FilterTypeDef sFilterConfig;
-FDCAN_TxHeaderTypeDef TxHeader;
-FDCAN_RxHeaderTypeDef RxHeader;
+static FDCAN_HandleTypeDef hfdcan;
+static FDCAN_FilterTypeDef sFilterConfig;
+static FDCAN_TxHeaderTypeDef TxHeader;
+static FDCAN_RxHeaderTypeDef RxHeader;
 static uint8_t FdcanDetected = 0U;
 
 /* Exported variables --------------------------------------------------------*/
@@ -42,7 +42,10 @@ uint8_t RxData[FDCAN_RAM_BUFFER_SIZE];
 static void OPENBL_FDCAN_Init(void);
 
 /* Private functions ---------------------------------------------------------*/
-
+/**
+ * @brief  This function is used to initialize the used FDCAN instance.
+ * @retval None.
+ */
 static void OPENBL_FDCAN_Init(void)
 {
   /*                Bit time configuration:
@@ -116,9 +119,7 @@ void OPENBL_FDCAN_Configuration(void)
 {
   /* Enable all resources clocks --------------------------------------------*/
   /* Enable used GPIOx clocks */
-  __HAL_RCC_GPIOD_CLK_ENABLE();
-  /* Enable FDCAN clock */
-  __HAL_RCC_FDCAN_CLK_ENABLE();
+  FDCANx_GPIO_CLK_ENABLE();
 
   OPENBL_FDCAN_Init();
 }
@@ -143,7 +144,7 @@ void OPENBL_FDCAN_DeInit(void)
 
 /**
  * @brief  This function is used to detect if there is any activity on FDCAN protocol.
- * @retval None.
+ * @retval Returns 1 if interface is detected else 0..
  */
 uint8_t OPENBL_FDCAN_ProtocolDetection(void)
 {
@@ -196,7 +197,7 @@ uint8_t OPENBL_FDCAN_GetCommandOpcode(void)
   */
 uint8_t OPENBL_FDCAN_ReadByte(void)
 {
-  uint8_t byte = 0x0;
+  uint8_t byte;
 
   /* check if FIFO 0 receive at least one message */
   while (HAL_FDCAN_GetRxFifoFillLevel(&hfdcan, FDCAN_RX_FIFO0) < 1)
@@ -212,7 +213,7 @@ uint8_t OPENBL_FDCAN_ReadByte(void)
 
 /**
   * @brief  This function is used to read bytes from FDCAN pipe.
-  * @retval Returns the read byte.
+  * @retval None.
   */
 void OPENBL_FDCAN_ReadBytes(uint8_t *Buffer, uint32_t BufferSize)
 {

Interfaces/Patterns/FDCAN/fdcan_interface.h

@@ -20,6 +20,10 @@
 #ifndef FDCAN_INTERFACE_H
 #define FDCAN_INTERFACE_H
 
+#ifdef __cplusplus
+extern "C" {
+#endif
+
 /* Includes ------------------------------------------------------------------*/
 #include <stdint.h>
 #include "openbl_core.h"
@@ -39,4 +43,8 @@ void OPENBL_FDCAN_SendByte(uint8_t Byte);
 void OPENBL_FDCAN_SendBytes(uint8_t *Buffer, uint32_t BufferSize);
 void OPENBL_FDCAN_SpecialCommandProcess(OPENBL_SpecialCmdTypeDef *Frame);
 
+#ifdef __cplusplus
+}
+#endif
+
 #endif /* FDCAN_INTERFACE_H */