Pull out settings flash calls

workspace/TS100/Core/BSP/BSP.h

@@ -2,6 +2,7 @@
 #include "stdint.h"
 #include "UnitSettings.h"
 #include "BSP_QC.h"
+#include "BSP_Flash.h"
 /*
  * BSP.h -- Board Support
  *

workspace/TS100/Core/BSP/BSP_Flash.h

@@ -0,0 +1,26 @@
+/*
+ * BSP_Flash.h
+ *
+ *  Created on: 29 May 2020
+ *      Author: Ralim
+ */
+#include "stdint.h"
+#ifndef BSP_BSP_FLASH_H_
+#define BSP_BSP_FLASH_H_
+#ifdef __cplusplus
+extern "C" {
+#endif
+/*
+ * Wrappers to allow read/writing to a sector of flash that we use to store all of the user settings
+ *
+ * Should allow reading and writing to the flash
+ */
+
+//Erase the flash, then save the buffer. Returns 1 if worked
+uint8_t flash_save_buffer(const uint8_t *buffer, const uint16_t length);
+
+void flash_read_buffer(uint8_t *buffer, const uint16_t length);
+#ifdef __cplusplus
+}
+#endif
+#endif /* BSP_BSP_FLASH_H_ */

workspace/TS100/Core/BSP/Miniware/flash.c

@@ -0,0 +1,49 @@
+/*
+ * flash.c
+ *
+ *  Created on: 29 May 2020
+ *      Author: Ralim
+ */
+
+#include "BSP_Flash.h"
+#include "BSP.h"
+#include "string.h"
+#include "stm32f1xx_hal.h"
+/*Flash start OR'ed with the maximum amount of flash - 1024 bytes*/
+/*We use the last 1024 byte page*/
+#define FLASH_ADDR (0x8000000 |0xFC00)
+uint8_t flash_save_buffer(const uint8_t *buffer, const uint16_t length) {
+	FLASH_EraseInitTypeDef pEraseInit;
+	pEraseInit.TypeErase = FLASH_TYPEERASE_PAGES;
+	pEraseInit.Banks = FLASH_BANK_1;
+	pEraseInit.NbPages = 1;
+	pEraseInit.PageAddress = FLASH_ADDR;
+	uint32_t failingAddress = 0;
+	resetWatchdog();
+	__HAL_FLASH_CLEAR_FLAG(
+			FLASH_FLAG_EOP | FLASH_FLAG_WRPERR | FLASH_FLAG_PGERR | FLASH_FLAG_BSY);
+	HAL_FLASH_Unlock();
+	HAL_Delay(10);
+	resetWatchdog();
+	HAL_FLASHEx_Erase(&pEraseInit, &failingAddress);
+	//^ Erase the page of flash (1024 bytes on this stm32)
+	// erased the chunk
+	// now we program it
+	uint16_t *data = (uint16_t*) buffer;
+	HAL_FLASH_Unlock();
+	for (uint8_t i = 0; i < (length / 2); i++) {
+		resetWatchdog();
+		HAL_FLASH_Program(FLASH_TYPEPROGRAM_HALFWORD, FLASH_ADDR + (i * 2),
+				data[i]);
+	}
+	HAL_FLASH_Lock();
+	return 1;
+}
+
+void flash_read_buffer(uint8_t *buffer, const uint16_t length) {
+
+	uint16_t *data = (uint16_t*) buffer;
+	for (uint8_t i = 0; i < (length / 2); i++) {
+		data[i] = *((uint16_t*) (FLASH_ADDR + (i * 2)));
+	}
+}

workspace/TS100/Core/Src/Settings.cpp

@@ -11,6 +11,7 @@
 #include "Settings.h"
 #include "Setup.h"
 #include "../../configuration.h"
+#include "BSP.h"
 #define FLASH_ADDR \
   (0x8000000 |     \
    0xFC00) /*Flash start OR'ed with the maximum amount of flash - 1024 bytes*/
@@ -19,39 +20,12 @@ volatile systemSettingsType systemSettings;
 
 void saveSettings() {
 	// First we erase the flash
-	FLASH_EraseInitTypeDef pEraseInit;
+	flash_save_buffer((uint8_t*) &systemSettings, sizeof(systemSettingsType));
-	pEraseInit.TypeErase = FLASH_TYPEERASE_PAGES;
-	pEraseInit.Banks = FLASH_BANK_1;
-	pEraseInit.NbPages = 1;
-	pEraseInit.PageAddress = FLASH_ADDR;
-	uint32_t failingAddress = 0;
-	HAL_IWDG_Refresh(&hiwdg);
-	__HAL_FLASH_CLEAR_FLAG(
-			FLASH_FLAG_EOP | FLASH_FLAG_WRPERR | FLASH_FLAG_PGERR | FLASH_FLAG_BSY);
-	HAL_FLASH_Unlock();
-	HAL_Delay(10);
-	HAL_IWDG_Refresh(&hiwdg);
-	HAL_FLASHEx_Erase(&pEraseInit, &failingAddress);
-	//^ Erase the page of flash (1024 bytes on this stm32)
-	// erased the chunk
-	// now we program it
-	uint16_t *data = (uint16_t*) &systemSettings;
-	HAL_FLASH_Unlock();
-
-	for (uint8_t i = 0; i < (sizeof(systemSettingsType) / 2); i++) {
-		HAL_IWDG_Refresh(&hiwdg);
-		HAL_FLASH_Program(FLASH_TYPEPROGRAM_HALFWORD, FLASH_ADDR + (i * 2),
-				data[i]);
-	}
-	HAL_FLASH_Lock();
 }
 
 bool restoreSettings() {
 	// We read the flash
-	uint16_t *data = (uint16_t*) &systemSettings;
+	flash_read_buffer((uint8_t*) &systemSettings, sizeof(systemSettingsType));
-	for (uint8_t i = 0; i < (sizeof(systemSettingsType) / 2); i++) {
-		data[i] = *((uint16_t*) (FLASH_ADDR + (i * 2)));
-	}
 
 	// if the version is correct were done
 	// if not we reset and save

workspace/TS100/Core/Src/gui.cpp

@@ -14,7 +14,7 @@
 #include "unit.h"
 #include "../../configuration.h"
 #include "Buttons.hpp"
-extern uint32_t lastButtonTime;
+
 void gui_Menu(const menuitem *menu);
 
 #ifdef MODEL_TS100

workspace/TS100/Core/Src/main.cpp

@@ -5,23 +5,14 @@
  */
 
 #include "BSP.h"
-
-#include <gui.hpp>
 #include <main.hpp>
 #include "LIS2DH12.hpp"
 #include <MMA8652FC.hpp>
-#include <history.hpp>
 #include <power.hpp>
 #include "Settings.h"
-#include "Translation.h"
 #include "cmsis_os.h"
-#include "stdlib.h"
-#include "stm32f1xx_hal.h"
-#include "string.h"
-#include "TipThermoModel.h"
 uint8_t PCBVersion = 0;
 // File local variables
-uint32_t currentTempTargetDegC = 0; // Current temperature target in C
 
 bool settingsWereReset = false;
 // FreeRTOS variables
@@ -51,6 +42,8 @@ int main(void) {
 	OLED::setFont(0);    // default to bigger font
 	// Testing for which accelerometer is mounted
 	resetWatchdog();
+	resetWatchdog();
+	settingsWereReset = restoreSettings();  // load the settings from flash
 	if (MMA8652FC::detect()) {
 		PCBVersion = 1;
 		MMA8652FC::initalize();  // this sets up the I2C registers
@@ -64,8 +57,6 @@ int main(void) {
 		systemSettings.ShutdownTime = 0;  // No accel -> disable sleep
 		systemSettings.sensitivity = 0;
 	}
-	resetWatchdog();
-	settingsWereReset = restoreSettings();  // load the settings from flash
 
 	resetWatchdog();
 

workspace/TS100/Core/Threads/PIDThread.cpp

@@ -17,6 +17,8 @@
 static TickType_t powerPulseRate = 1000;
 static TickType_t powerPulseDuration = 50;
 TaskHandle_t pidTaskNotification = NULL;
+uint32_t currentTempTargetDegC = 0; // Current temperature target in C
+
 /* StartPIDTask function */
 void startPIDTask(void const *argument __unused) {
 	/*