Skip to content

K22F: Add FlashIAP for storage #11282

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 23, 2019
Merged

K22F: Add FlashIAP for storage #11282

merged 1 commit into from
Aug 23, 2019

Conversation

mmahadevan108
Copy link
Contributor

Description

Enable Flash storage, test results below:

mbedgt: test suite results: 6 OK
mbedgt: test case report:

target platform_name test suite test case passed failed result elapsed_time (sec)
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-buffered_block_device BufferedBlockDevice functionality test 1 0 OK 0.26
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-flashsim_block_device FlashSimBlockDevice functionality test 1 0 OK 0.06
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-general_block_device DEFAULT Testing get type functionality 1 0 OK 0.1
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-general_block_device FLASHIAP Testing BlockDevice erase functionality 1 0 OK 0.44
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-general_block_device FLASHIAP Testing Deinit block device 1 0 OK 0.1
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-general_block_device FLASHIAP Testing Init block device 1 0 OK 0.1
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-general_block_device FLASHIAP Testing contiguous erase, write and read 1 0 OK 0.63
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-general_block_device FLASHIAP Testing multi threads erase program read 1 0 OK 3.75
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-general_block_device FLASHIAP Testing program read small data sizes 1 0 OK 0.18
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-general_block_device FLASHIAP Testing read write random blocks 1 0 OK 0.93
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-general_block_device FLASHIAP Testing unaligned erase blocks 1 0 OK 0.13
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-heap_block_device Testing get type functionality 1 0 OK 0.06
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-heap_block_device Testing read write random blocks 1 0 OK 1.94
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-mbr_block_device Testing formatting of master boot record 1 0 OK 0.07
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-mbr_block_device Testing mbr attributes 1 0 OK 0.62
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-mbr_block_device Testing mbr read write 1 0 OK 0.06
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-util_block_device Testing chaining of block devices 1 0 OK 0.06
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-util_block_device Testing profiling of block devices 1 0 OK 0.06
K22F-IAR K22F mbed-os-features-storage-tests-blockdevice-util_block_device Testing slicing of a block device 1 0 OK 0.08
mbedgt: test case results: 19 OK

Pull request type

[ ] Fix
[ ] Refactor
[X] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

@mmahadevan108
K22F: Add FlashIAP for storage
20aa1ec 
Signed-off-by: Mahesh Mahadevan <[email protected]>
@mmahadevan108
Copy link
Contributor Author

cc @0xc0170 @maclobdell

@ciarmcom ciarmcom requested a review from a team August 21, 2019 19:01
@ciarmcom
Copy link
Member

@mmahadevan108, thank you for your changes.
@ARMmbed/mbed-os-maintainers please review.

0xc0170
0xc0170 reviewed Aug 22, 2019
View reviewed changes
targets/targets.json
@@ -8574,7 +8575,7 @@
"bootloader_supported": true,
"mbed_rom_start": "0x10000000",
"mbed_rom_size": "0x100000",
"sectors": [[268435456, 512]],
"sectors": [[268435456, 512]],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this will conflict in your other PR I guess, lets see once we merge

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 22, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented Aug 22, 2019

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 2
Build artifacts

@0xc0170 0xc0170 merged commit b1aec79 into ARMmbed:master Aug 23, 2019
@mmahadevan108 mmahadevan108 deleted the K22_Add_Storage branch August 23, 2019 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xc0170 0xc0170 0xc0170 approved these changes

Assignees
No one assigned
Labels
release-version: 5.13.4
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mmahadevan108 @ciarmcom @0xc0170 @mbed-ci