[EEPROM emulation] Clean up code

Clean up also the old references to the original author/version/date,
since we already refactored this code quite well.
The copyright notice of course is still intact.

Signed-off-by: Nils Hasenbanck <[email protected]>

Author: Nils Hasenbanck

cores/arduino/stm32/stm32_eeprom.c

@@ -1,11 +1,7 @@
 /**
   ******************************************************************************
-  * @file    eeprom.c
-  * @author  WI6LABS
-  * @version V1.0.0
-  * @date    01-August-2016
-  * @brief   provide emulated eeprom from flash
-  *
+  * @file    stm32_eeprom.c
+  * @brief   Provides emulated eeprom from flash
   ******************************************************************************
   * @attention
   *
@@ -35,48 +31,24 @@
   *
   ******************************************************************************
   */
-/** @addtogroup CMSIS
-  * @{
-  */
 
-/** @addtogroup stm32f4xx_system
-  * @{
-  */
-
-/** @addtogroup STM32F4xx_System_Private_Includes
-  * @{
-  */
 #include "stm32_eeprom.h"
 #include <string.h>
 
 #ifdef __cplusplus
  extern "C" {
 #endif
 
-/**
-  * @}
-  */
-
-/** @addtogroup STM32F4xx_System_Private_TypesDefinitions
-  * @{
-  */
-
-/**
-  * @}
-  */
-
-/** @addtogroup STM32F4xx_System_Private_Defines
-  * @{
-  */
-// We use the last page of the flash to store data (to prevent code overwritten).
+/* Use the last page of the flash to store data in order to prevent overwritting
+   program data */
 #if defined (STM32F0xx) || defined (STM32F1xx) || defined(STM32L1xx)
 #if defined (FLASH_BANK2_END)
 #define FLASH_BASE_ADDRESS  ((uint32_t)((FLASH_BANK2_END + 1) - FLASH_PAGE_SIZE))
 #elif defined (FLASH_BANK1_END)
 #define FLASH_BASE_ADDRESS  ((uint32_t)((FLASH_BANK1_END + 1) - FLASH_PAGE_SIZE))
 #else
 #define FLASH_BASE_ADDRESS  ((uint32_t)((FLASH_END + 1) - FLASH_PAGE_SIZE))
-#endif // FLASH_BANK2_END
+#endif /* FLASH_BANK2_END */
 #elif defined (STM32F2xx) || defined (STM32F4xx) || defined (STM32F7xx)
 #define FLASH_BASE_ADDRESS  ((uint32_t)(FLASH_END + 1) - FLASH_PAGE_SIZE)
 #define FLASH_DATA_SECTOR   ((uint32_t)(FLASH_SECTOR_TOTAL - 1))
@@ -116,76 +88,48 @@ static inline uint32_t get_flash_end(void) {
 #define FLASH_BANK_NUMBER   FLASH_BANK_1
 #else
 #define FLASH_BANK_NUMBER   FLASH_BANK_2
-#endif // FLASH_BANK_2
-// Flash base address
+#endif /* FLASH_BANK_2 */
+/* Flash base address */
 #define FLASH_PAGE_NUMBER   ((uint32_t)((FLASH_SIZE / FLASH_PAGE_SIZE) - 1))
 #define FLASH_BASE_ADDRESS  ((uint32_t)(FLASH_BASE + (FLASH_PAGE_NUMBER * FLASH_PAGE_SIZE)))
 #endif
-/**
-  * @}
-  */
-
-/** @addtogroup STM32F4xx_System_Private_Macros
-  * @{
-  */
-
-/**
-  * @}
-  */
 
-/** @addtogroup STM32F4xx_System_Private_Variables
-  * @{
-  */
 static uint8_t tmpEE[E2END] = {0};
 
-/**
-  * @}
-  */
-
-/** @addtogroup STM32F4xx_System_Private_FunctionPrototypes
-  * @{
-  */
 void get_data_from_flash(void);
 void set_data_to_flash(void);
 
-/**
-  * @}
-  */
-
 /**
   * @brief  Function read a byte from eeprom
-  * @param  __p : address to read
+  * @param  pos : address to read
   * @retval byte : data read from eeprom
   */
-uint8_t eeprom_read_byte(const uint16_t __p)
-{
+uint8_t eeprom_read_byte(const uint16_t pos) {
   uint8_t byte = 0;
 
   get_data_from_flash();
-  byte = tmpEE[__p];
+  byte = tmpEE[pos];
 
   return byte;
 }
 
 /**
-  * @brief  Function write a byte to eeprom
-  * @param  __p : address to write
-  * @param  __value : value to write
+  * @brief  Function writes a byte to emulated eeprom (flash)
+  * @param  pos : address to write
+  * @param  value : value to write
   * @retval none
   */
-void eeprom_write_byte(uint16_t __p, uint8_t __value)
-{
-  tmpEE[__p] = __value;
+void eeprom_write_byte(uint16_t pos, uint8_t value) {
+  tmpEE[pos] = value;
   set_data_to_flash();
 }
 
 /**
-  * @brief  The function read into the flash.
+  * @brief  Function reads a byte from emulated eeprom (flash)
   * @param  none
   * @retval none
   */
-void get_data_from_flash(void)
-{
+void get_data_from_flash(void) {
   memcpy(tmpEE, (uint8_t*)(FLASH_BASE_ADDRESS), E2END);
 }
 
@@ -194,9 +138,8 @@ void get_data_from_flash(void)
   * @param  none
   * @retval none
   */
-void set_data_to_flash(void)
-{
-  //copy in flash
+void set_data_to_flash(void) {
+  /* Copy in flash */
   FLASH_EraseInitTypeDef EraseInitStruct;
   uint32_t offset = 0;
   uint32_t address = FLASH_BASE_ADDRESS;
@@ -206,12 +149,12 @@ void set_data_to_flash(void)
   uint32_t pageError = 0;
   uint64_t data = 0;
 
-  // ERASING page
+  /* ERASING page */
   EraseInitStruct.TypeErase = FLASH_TYPEERASE_PAGES;
 #ifdef STM32L4xx
   EraseInitStruct.Banks = FLASH_BANK_NUMBER;
   EraseInitStruct.Page = FLASH_PAGE_NUMBER;
-#else // STM32F4xx
+#else /* STM32F4xx */
 #ifdef STM32F1xx
   EraseInitStruct.Banks       = FLASH_BANK_1;
 #endif
@@ -262,16 +205,15 @@ void set_data_to_flash(void)
   uint32_t SectorError = 0;
   uint32_t data = 0;
 
-  // ERASING page
+  /* ERASING page */
   EraseInitStruct.TypeErase = FLASH_TYPEERASE_SECTORS;
   EraseInitStruct.VoltageRange = FLASH_VOLTAGE_RANGE_3;
   EraseInitStruct.Sector = FLASH_DATA_SECTOR;
   EraseInitStruct.NbSectors = 1;
 
   HAL_FLASH_Unlock();
 
-  if(HAL_FLASHEx_Erase(&EraseInitStruct, &SectorError) == HAL_OK)
-  {
+  if(HAL_FLASHEx_Erase(&EraseInitStruct, &SectorError) == HAL_OK) {
     while(address < address_end) {
       memcpy(&data, tmpEE + offset, sizeof(uint32_t));
       if(HAL_FLASH_Program(FLASH_TYPEPROGRAM_WORD, address, data) == HAL_OK) {
@@ -286,18 +228,6 @@ void set_data_to_flash(void)
 #endif
 }
 
-
-/**
-  * @}
-  */
-
-/**
-  * @}
-  */
-
-/**
-  * @}
-  */
 #ifdef __cplusplus
 }
 #endif

cores/arduino/stm32/stm32_eeprom.h

@@ -1,9 +1,6 @@
 /**
   ******************************************************************************
   * @file    stm32_eeprom.h
-  * @author  WI6LABS
-  * @version V1.0.0
-  * @date    01-August-2016
   * @brief   Header for eeprom module
   ******************************************************************************
   * @attention
@@ -50,16 +47,16 @@
 /* Exported constants --------------------------------------------------------*/
 
 #if defined (STM32F2xx) || defined (STM32F4xx) || defined (STM32F7xx)
-//FLASH_SECTOR_SIZE
-#define FLASH_PAGE_SIZE     ((uint32_t)(16*1024)) //16kB page
+/* FLASH_SECTOR_SIZE */
+#define FLASH_PAGE_SIZE     ((uint32_t)(16*1024)) /* 16kB page */
 #endif
 #define E2END FLASH_PAGE_SIZE
 
 /* Exported macro ------------------------------------------------------------*/
 /* Exported functions ------------------------------------------------------- */
 
-uint8_t eeprom_read_byte(const uint16_t __p);
-void eeprom_write_byte(uint16_t __p, uint8_t __value);
+uint8_t eeprom_read_byte(const uint16_t pos);
+void eeprom_write_byte(uint16_t pos, uint8_t value);
 
 #ifdef __cplusplus
 }