Skip to content

Mbed os workshop 17q2 nordic flash api #4234

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

Closed

Conversation

kl-cruz
Copy link
Contributor

@kl-cruz kl-cruz commented Apr 27, 2017

Description

Adding support for flashiap for nRF52840 devices

Status

READY

Steps to test or reproduce

Tests passed:
mbed test -n "tests-mbed_hal-flash" -t GCC_ARM -m NRF52840_dk
and
mbed test -n "tests-mbed_drivers-flashiap" -t GCC_ARM -m NRF52840_dk

asmellby and others added 30 commits March 27, 2017 11:44
* Make memory sections configurable in linker files
* Dynamically determine vector location in flash for NVIC relocation
* Advertise bootloader support in targets.json
When using ARM Compiler 5, the RTX config hard-coded the heap and stack
sizes to specific values. This prevented the RTX HAL from dynamically
allocating unused memory as heap space.

Specifically, the HEAP_START define prevents this logic in RTX_CM_lib.h
from activating. The rest of the defines are also set in that header,
and should be removed from here.
The header is already included inside of flash_api.h if the target
uses CMSIS flash algos. If it doesn't, the file shouldn't be included,
since it doesn't exist.
* Using PinName as bitfield doesn't work without warnings, since NC
  needs all 32 bits to be represented.
* lp_ticker should not be freed when interrupt is disabled, since this
  will kill the timebase.
Signed-off-by: Mahadevan Mahesh <[email protected]>
*** Is alpha ***

I added options to tools/arm_pack_manager/pack_manager.py and
tools/flash_algo/extract.py to allow a user to specify a single pack on
disk to add to the cache. The syntax is as follows:
 - python tools/arm_pack_manager/pack_manager.py add-local-pack <local-pack>
 - cd tools/flash_algo; python extract.py --local-pack <local-pack>

Both of these methods will add your pack to the index and copy the pack
file into the location that arm_pack_manager would have placed it if it
was downloaded from the internet.
GetSector has been rewritten
Those defines are already available in STM32 L4 SDK and don't need to
be (re)defined here.
…support

[Silicon Labs] Add flash HAL API support
…p-with-armcc

[Silicon Labs] Enable dynamic heap with ARMCC
0xc0170 and others added 15 commits April 7, 2017 10:25
Flash API support using the MCUXpresso drivers
This is the introduction of Flash API support for STM32 L0 family.
K64F: Add MMCAU driver needed for mbedtls
…spi_sdcard

[NRF52840]: enabled SdBlockDevice capability
…op-17Q2

Disambiguate utest::v1 types and remove using statement
@kl-cruz
Copy link
Contributor Author

kl-cruz commented Apr 27, 2017

Added flashapi support for nRF52840 @nvlsianpu @0xc0170

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 28, 2017

@kl-cruz Can we push this to master (assuming there are no dependencies) ? Or what was reason for the workshop branch ?

@kl-cruz
Copy link
Contributor Author

kl-cruz commented Apr 28, 2017

Sure, we can push it to master. We were starting working on it at workshop branch.

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 28, 2017

Sure, we can push it to master. We were starting working on it at workshop branch.

Lets redirect it (rebase) to master, thank you

@theotherjimmy
Copy link
Contributor

@kl-cruz bump. Would you like to submit another PR? Alternatively, we could change the base branch of this PR and do appropriate rebasing.

@kl-cruz kl-cruz changed the base branch from mbed-os-workshop-17q2 to master May 5, 2017 07:10
@kl-cruz
Copy link
Contributor Author

kl-cruz commented May 8, 2017

I made another PR for flash api (Based on master with cherry picked commits from here): #4282
I'll close this PR. Now everything is up to date.

@kl-cruz kl-cruz closed this May 8, 2017
@sg- sg- removed the needs: work label May 8, 2017
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.