rusefi
/
rusefi-1
mirror of https://github.com/rusefi/rusefi-1.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Flash fix erase check (#2750) 

* Fix flash erase error formating

* Flash F7: do magic calculation with local copy of sector number

* Falsh stm32: simplify

* typo?
This commit is contained in:
Andrey G 2021-05-28 20:35:54 +03:00 committed by GitHub
parent 578256ec86
commit dbfe0e0fbc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
firmware/controllers/flash_main.cpp
View File

@ -94,7 +94,7 @@ int eraseAndFlashCopy(flashaddr_t storageAddress, const TStorage& data) {
auto err = intFlashErase(storageAddress, sizeof(TStorage));
if (FLASH_RETURN_SUCCESS != err) {
firmwareError(OBD_PCM_Processor_Fault, "Failed to erase flash at %#010x", storageAddress);
firmwareError(OBD_PCM_Processor_Fault, "Failed to erase flash at 0x%08x", storageAddress);
return err;
}

37
firmware/hw_layer/ports/stm32/flash_int.c
View File

@ -92,14 +92,18 @@ static bool isDualBank(void) {
#endif
int intFlashSectorErase(flashsector_t sector) {
uint8_t sectorRegIdx = sector;
#ifdef STM32F7XX
// On dual bank STM32F7, sector index doesn't match register value.
// High bit indicates bank, low 4 bits indicate sector within bank.
// Since each bank has 12 sectors, increment second-bank sector idx
// by 4 so that the first sector of the second bank (12) ends up with
// index 16 (0b10000)
if (isDualBank() && sector >= 12) {
sector += 4;
if (isDualBank() && sectorRegIdx >= 12) {
sectorRegIdx -= 12;
/* bit 4 defines bank.
* Sectors starting from 12 are in bank #2 */
sectorRegIdx |= 0x10;
}
#endif
@ -120,32 +124,15 @@ int intFlashSectorErase(flashsector_t sector) {
* 00001 sector 1
* ...
* 01011 sector 11 (the end of 1st bank, 1Mb border)
* 01100 sector 12 (start of 2nd bank)
* 10000 sector 12 (start of 2nd bank)
* ...
* 10111 sector 23 (the end of 2nd bank, 2Mb border)
* 11011 sector 23 (the end of 2nd bank, 2Mb border)
* others not allowed */
#ifndef FLASH_CR_SNB_3
FLASH_CR &= ~(FLASH_CR_SNB_0 | FLASH_CR_SNB_1 | FLASH_CR_SNB_2);
#elif !defined(FLASH_CR_SNB_4)
FLASH_CR &= ~(FLASH_CR_SNB_0 | FLASH_CR_SNB_1 | FLASH_CR_SNB_2 | FLASH_CR_SNB_3);
#else
FLASH_CR &= ~(FLASH_CR_SNB_0 | FLASH_CR_SNB_1 | FLASH_CR_SNB_2 | FLASH_CR_SNB_3 | FLASH_CR_SNB_4);
#endif
if (sector & 0x1)
FLASH_CR |= FLASH_CR_SNB_0;
if (sector & 0x2)
FLASH_CR |= FLASH_CR_SNB_1;
if (sector & 0x4)
FLASH_CR |= FLASH_CR_SNB_2;
#ifdef FLASH_CR_SNB_4
if (sector & 0x8)
FLASH_CR |= FLASH_CR_SNB_3;
#endif
#ifdef FLASH_CR_SNB_4
if (sector & 0x10)
FLASH_CR |= FLASH_CR_SNB_4;
#endif
FLASH_CR &= ~FLASH_CR_SNB_Msk;
FLASH_CR |= (sectorRegIdx << FLASH_CR_SNB_Pos) & FLASH_CR_SNB_Msk;
/* sector erase */
FLASH_CR |= FLASH_CR_SER;
/* start erase operation */
FLASH_CR |= FLASH_CR_STRT;
/* Wait until it's finished. */