-
Notifications
You must be signed in to change notification settings - Fork 3k
Add bootloader support for the UBLOX_C030 platforms. #4590
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,184 @@ | ||
/* mbed Microcontroller Library | ||
******************************************************************************* | ||
* Copyright (c) 2017, STMicroelectronics | ||
* All rights reserved. | ||
* | ||
* Redistribution and use in source and binary forms, with or without | ||
* modification, are permitted provided that the following conditions are met: | ||
* | ||
* 1. Redistributions of source code must retain the above copyright notice, | ||
* this list of conditions and the following disclaimer. | ||
* 2. Redistributions in binary form must reproduce the above copyright notice, | ||
* this list of conditions and the following disclaimer in the documentation | ||
* and/or other materials provided with the distribution. | ||
* 3. Neither the name of STMicroelectronics nor the names of its contributors | ||
* may be used to endorse or promote products derived from this software | ||
* without specific prior written permission. | ||
* | ||
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" | ||
* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE | ||
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE | ||
* DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE | ||
* FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL | ||
* DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR | ||
* SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER | ||
* CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, | ||
* OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE | ||
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | ||
******************************************************************************* | ||
*/ | ||
|
||
#include "flash_api.h" | ||
#include "flash_data.h" | ||
#include "mbed_critical.h" | ||
|
||
#if DEVICE_FLASH | ||
|
||
#if defined (STM32F427xx) || defined (STM32F437xx) | ||
#define FLASH_SIZE (uint32_t) 0x100000 | ||
#endif | ||
|
||
static uint32_t GetSector(uint32_t Address); | ||
static uint32_t GetSectorSize(uint32_t Sector); | ||
|
||
int32_t flash_init(flash_t *obj) | ||
{ | ||
/* Allow Access to Flash control registers and user Falsh */ | ||
if (HAL_FLASH_Unlock()) { | ||
return -1; | ||
} else { | ||
return 0; | ||
} | ||
} | ||
int32_t flash_free(flash_t *obj) | ||
{ | ||
/* Disable the Flash option control register access (recommended to protect | ||
the option Bytes against possible unwanted operations) */ | ||
if (HAL_FLASH_Lock()) { | ||
return -1; | ||
} else { | ||
return 0; | ||
} | ||
} | ||
int32_t flash_erase_sector(flash_t *obj, uint32_t address) | ||
{ | ||
/*Variable used for Erase procedure*/ | ||
static FLASH_EraseInitTypeDef EraseInitStruct; | ||
uint32_t FirstSector; | ||
uint32_t SectorError = 0; | ||
|
||
if ((address >= (FLASH_BASE + FLASH_SIZE)) || (address < FLASH_BASE)) { | ||
|
||
return -1; | ||
} | ||
|
||
/* Get the 1st sector to erase */ | ||
FirstSector = GetSector(address); | ||
|
||
/* Fill EraseInit structure*/ | ||
EraseInitStruct.TypeErase = FLASH_TYPEERASE_SECTORS; | ||
EraseInitStruct.VoltageRange = FLASH_VOLTAGE_RANGE_3; | ||
EraseInitStruct.Sector = FirstSector; | ||
EraseInitStruct.NbSectors = 1; | ||
if(HAL_FLASHEx_Erase(&EraseInitStruct, &SectorError) != HAL_OK){ | ||
return -1; | ||
} else { | ||
return 0; | ||
} | ||
} | ||
|
||
int32_t flash_program_page(flash_t *obj, uint32_t address, const uint8_t *data, uint32_t size) | ||
{ | ||
|
||
if ((address >= (FLASH_BASE + FLASH_SIZE)) || (address < FLASH_BASE)) { | ||
return -1; | ||
} | ||
|
||
/* Note: If an erase operation in Flash memory also concerns data in the data or instruction cache, | ||
you have to make sure that these data are rewritten before they are accessed during code | ||
execution. If this cannot be done safely, it is recommended to flush the caches by setting the | ||
DCRST and ICRST bits in the FLASH_CR register. */ | ||
__HAL_FLASH_DATA_CACHE_DISABLE(); | ||
__HAL_FLASH_INSTRUCTION_CACHE_DISABLE(); | ||
|
||
__HAL_FLASH_DATA_CACHE_RESET(); | ||
__HAL_FLASH_INSTRUCTION_CACHE_RESET(); | ||
|
||
__HAL_FLASH_INSTRUCTION_CACHE_ENABLE(); | ||
__HAL_FLASH_DATA_CACHE_ENABLE(); | ||
|
||
while (size > 0) { | ||
if (HAL_FLASH_Program(FLASH_TYPEPROGRAM_BYTE, address, (uint64_t)*data) != HAL_OK) { | ||
return -1; | ||
} else { | ||
size--; | ||
address++; | ||
data++; | ||
} | ||
} | ||
return 0; | ||
} | ||
|
||
uint32_t flash_get_sector_size(const flash_t *obj, uint32_t address) | ||
{ | ||
|
||
if ((address >= (FLASH_BASE + FLASH_SIZE)) || (address < FLASH_BASE)) { | ||
return MBED_FLASH_INVALID_SIZE; | ||
} | ||
|
||
return (GetSectorSize(GetSector(address))); | ||
} | ||
|
||
uint32_t flash_get_page_size(const flash_t *obj) | ||
{ | ||
// not applicable for STM32F4 | ||
return (0x4000); // minimum sector size | ||
} | ||
uint32_t flash_get_start_address(const flash_t *obj) | ||
{ | ||
return FLASH_BASE; | ||
} | ||
uint32_t flash_get_size(const flash_t *obj) | ||
{ | ||
return FLASH_SIZE; | ||
} | ||
|
||
/** | ||
* @brief Gets the sector of a given address | ||
* @param None | ||
* @retval The sector of a given address | ||
*/ | ||
static uint32_t GetSector(uint32_t address) | ||
{ | ||
uint32_t sector = 0; | ||
uint32_t tmp = address - ADDR_FLASH_SECTOR_0; | ||
if (address < ADDR_FLASH_SECTOR_4) { // 16k sectorsize | ||
sector += tmp >>14; | ||
} else if (address < ADDR_FLASH_SECTOR_5) { //64k sector size | ||
sector += FLASH_SECTOR_4; | ||
} else { | ||
sector += 4 + (tmp >>17); | ||
} | ||
return sector; | ||
} | ||
|
||
/** | ||
* @brief Gets sector Size | ||
* @param None | ||
* @retval The size of a given sector | ||
*/ | ||
static uint32_t GetSectorSize(uint32_t Sector) | ||
{ | ||
uint32_t sectorsize = 0x00; | ||
if((Sector == FLASH_SECTOR_0) || (Sector == FLASH_SECTOR_1) || (Sector == FLASH_SECTOR_2) ||\ | ||
(Sector == FLASH_SECTOR_3)) { | ||
sectorsize = 16 * 1024; | ||
} else if((Sector == FLASH_SECTOR_4)){ | ||
sectorsize = 64 * 1024; | ||
} else { | ||
sectorsize = 128 * 1024; | ||
} | ||
return sectorsize; | ||
} | ||
|
||
#endif |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,56 @@ | ||
/* mbed Microcontroller Library | ||
******************************************************************************* | ||
* Copyright (c) 2016, STMicroelectronics | ||
* All rights reserved. | ||
* | ||
* Redistribution and use in source and binary forms, with or without | ||
* modification, are permitted provided that the following conditions are met: | ||
* | ||
* 1. Redistributions of source code must retain the above copyright notice, | ||
* this list of conditions and the following disclaimer. | ||
* 2. Redistributions in binary form must reproduce the above copyright notice, | ||
* this list of conditions and the following disclaimer in the documentation | ||
* and/or other materials provided with the distribution. | ||
* 3. Neither the name of STMicroelectronics nor the names of its contributors | ||
* may be used to endorse or promote products derived from this software | ||
* without specific prior written permission. | ||
* | ||
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" | ||
* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE | ||
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE | ||
* DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE | ||
* FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL | ||
* DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR | ||
* SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER | ||
* CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, | ||
* OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE | ||
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | ||
******************************************************************************* | ||
*/ | ||
#ifndef MBED_FLASH_DATA_H | ||
#define MBED_FLASH_DATA_H | ||
|
||
#include "device.h" | ||
#include <stdint.h> | ||
|
||
#if DEVICE_FLASH | ||
|
||
/* Exported types ------------------------------------------------------------*/ | ||
/* Exported constants --------------------------------------------------------*/ | ||
/* Exported macro ------------------------------------------------------------*/ | ||
/* Base address of the Flash sectors */ | ||
#define ADDR_FLASH_SECTOR_0 ((uint32_t)0x08000000) /* Base @ of Sector 0, 16 Kbytes */ | ||
#define ADDR_FLASH_SECTOR_1 ((uint32_t)0x08004000) /* Base @ of Sector 1, 16 Kbytes */ | ||
#define ADDR_FLASH_SECTOR_2 ((uint32_t)0x08008000) /* Base @ of Sector 2, 16 Kbytes */ | ||
#define ADDR_FLASH_SECTOR_3 ((uint32_t)0x0800C000) /* Base @ of Sector 3, 16 Kbytes */ | ||
#define ADDR_FLASH_SECTOR_4 ((uint32_t)0x08010000) /* Base @ of Sector 4, 64 Kbytes */ | ||
#define ADDR_FLASH_SECTOR_5 ((uint32_t)0x08020000) /* Base @ of Sector 5, 128 Kbytes */ | ||
#define ADDR_FLASH_SECTOR_6 ((uint32_t)0x08040000) /* Base @ of Sector 6, 128 Kbytes */ | ||
#define ADDR_FLASH_SECTOR_7 ((uint32_t)0x08060000) /* Base @ of Sector 7, 128 Kbytes */ | ||
#define ADDR_FLASH_SECTOR_8 ((uint32_t)0x08080000) /* Base @ of Sector 8, 128 Kbytes */ | ||
#define ADDR_FLASH_SECTOR_9 ((uint32_t)0x080A0000) /* Base @ of Sector 9, 128 Kbytes */ | ||
#define ADDR_FLASH_SECTOR_10 ((uint32_t)0x080C0000) /* Base @ of Sector 10, 128 Kbytes */ | ||
#define ADDR_FLASH_SECTOR_11 ((uint32_t)0x080E0000) /* Base @ of Sector 11, 128 Kbytes */ | ||
|
||
#endif | ||
#endif |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,40 @@ | ||
/* mbed Microcontroller Library | ||
* Copyright (c) 2017-2017 ARM Limited | ||
* | ||
* Licensed under the Apache License, Version 2.0 (the "License"); | ||
* you may not use this file except in compliance with the License. | ||
* You may obtain a copy of the License at | ||
* | ||
* http://www.apache.org/licenses/LICENSE-2.0 | ||
* | ||
* Unless required by applicable law or agreed to in writing, software | ||
* distributed under the License is distributed on an "AS IS" BASIS, | ||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
* See the License for the specific language governing permissions and | ||
* limitations under the License. | ||
*/ | ||
#ifndef NVIC_ADDR_H | ||
#define NVIC_ADDR_H | ||
|
||
#ifdef __cplusplus | ||
extern "C" { | ||
#endif | ||
|
||
#if defined(__ICCARM__) | ||
#pragma section=".intvec" | ||
#define NVIC_FLASH_VECTOR_ADDRESS ((uint32_t)__section_begin(".intvec")) | ||
#elif defined(__CC_ARM) | ||
extern uint32_t Load$$LR$$LR_IROM1$$Base[]; | ||
#define NVIC_FLASH_VECTOR_ADDRESS ((uint32_t)Load$$LR$$LR_IROM1$$Base) | ||
#elif defined(__GNUC__) | ||
extern uint32_t g_pfnVectors[]; | ||
#define NVIC_FLASH_VECTOR_ADDRESS ((uint32_t)g_pfnVectors) | ||
#else | ||
#error "Flash vector address not set for this toolchain" | ||
#endif | ||
|
||
#ifdef __cplusplus | ||
} | ||
#endif | ||
|
||
#endif |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1465,10 +1465,11 @@ | |
} | ||
}, | ||
"macros_add": ["RTC_LSI=1", "HSE_VALUE=12000000", "GNSSBAUD=9600"], | ||
"device_has_add": ["ANALOGOUT", "TRNG"], | ||
"device_has_add": ["ANALOGOUT", "SERIAL_FC", "TRNG", "FLASH"], | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @RobMeades Why did "SERIAL_FC" get added here? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @jeromecoutant's change to all the STM32F4 platforms, which I had to rebase against, dropped it, so I added it back in again. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I can put that change in another PR if you prefer but I really don't want to wait as we definitely need flow control on the serial ports. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Cool. Thanks for responding. |
||
"features": ["LWIP"], | ||
"public": false, | ||
"device_name": "STM32F437VG" | ||
"device_name": "STM32F437VG", | ||
"bootloader_supported": true | ||
}, | ||
"UBLOX_C030_U201": { | ||
"inherits": ["UBLOX_C030"], | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hi - just be aware that those lines seem to have side effects - see here #3958
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ooo, well spotted, the EmBitz exporter is not something I know anything about and I guess it is not tested by the automation system here. @0xc0170 : is this something I need to be concerned about?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
probably not blocking, but when a solution is found to #3958, let's make sure to report it here as well - either you or us ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@LMESTM If/when I find a solution to #3958, it should not affect this patch.