Skip to content

STM32 L0: Add Flash API support #4135

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

Merged
merged 1 commit into from
Apr 19, 2017

Conversation

LMESTM
Copy link
Contributor

@LMESTM LMESTM commented Apr 7, 2017

Description

This PR contains the flash API implementation for L0 STM32 family.

Status

READY

Steps to test or reproduce

Test report on L073RZ

+-----------------------+---------------+-----------------------------+-------------------------------+--------+--------+--------+--------------------+
| target                | platform_name | test suite                  | test case                     | passed | failed | result | elapsed_time (sec) |
+-----------------------+---------------+-----------------------------+-------------------------------+--------+--------+--------+--------------------+
| NUCLEO_L073RZ-GCC_ARM | NUCLEO_L073RZ | tests-mbed_drivers-flashiap | FlashIAP - init               | 1      | 0      | OK     | 0.05               |
| NUCLEO_L073RZ-GCC_ARM | NUCLEO_L073RZ | tests-mbed_drivers-flashiap | FlashIAP - program            | 1      | 0      | OK     | 3.53               |
| NUCLEO_L073RZ-GCC_ARM | NUCLEO_L073RZ | tests-mbed_drivers-flashiap | FlashIAP - program errors     | 1      | 0      | OK     | 0.06               |
| NUCLEO_L073RZ-GCC_ARM | NUCLEO_L073RZ | tests-mbed_hal-flash        | Flash - buffer alignment test | 1      | 0      | OK     | 0.59               |
| NUCLEO_L073RZ-GCC_ARM | NUCLEO_L073RZ | tests-mbed_hal-flash        | Flash - clock and cache test  | 0      | 1      | FAIL   | 0.37               |
| NUCLEO_L073RZ-GCC_ARM | NUCLEO_L073RZ | tests-mbed_hal-flash        | Flash - erase sector          | 1      | 0      | OK     | 0.16               |
| NUCLEO_L073RZ-GCC_ARM | NUCLEO_L073RZ | tests-mbed_hal-flash        | Flash - init                  | 1      | 0      | OK     | 0.3                |
| NUCLEO_L073RZ-GCC_ARM | NUCLEO_L073RZ | tests-mbed_hal-flash        | Flash - mapping alignment     | 1      | 0      | OK     | 0.06               |
| NUCLEO_L073RZ-GCC_ARM | NUCLEO_L073RZ | tests-mbed_hal-flash        | Flash - program page          | 1      | 0      | OK     | 0.52               |
+-----------------------+---------------+-----------------------------+-------------------------------+--------+--------+--------+--------------------+

Note: The tests-mbed_hal-flash | Flash - clock and cache test most of the time fails but this has nothing to do with Flash programming. There seems to be timer variations of 0.2% over time compared to 0.1% target, which needs to be investigated but also happens without flash programming.

L073RZ actually the only mbed-os5 target, so no tests can be run on other targets

This is the introduction of Flash API support for STM32 L0 family.
@0xc0170
Copy link
Contributor

0xc0170 commented Apr 7, 2017

Note: The tests-mbed_hal-flash | Flash - clock and cache test most of the time fails but this has nothing to do with Flash programming. There seems to be timer variations of 0.2% over time compared to 0.1% target, which needs to be investigated but also happens without flash programming.

Interesting, we might want o write more ticker tests :)

@sg-
Copy link
Contributor

sg- commented Apr 10, 2017

/morph test-nightly

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test-nightly

@bridadan
Copy link
Contributor

/morph test-nightly

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 1880

Test failed!

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 12, 2017

@LMESTM Don't forget to look at the timer issue mentioned above

@sg- sg- merged commit 92da0d9 into ARMmbed:mbed-os-workshop-17q2 Apr 19, 2017
@adustm
Copy link
Member

adustm commented May 16, 2017

Hello @0xc0170 @screamerbg @sg-
This PR has been merged 27 days ago, in the workshop branch.
Could you let us know when we will be able to see it in the master branch ?

Kind regards
Armelle

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants