Skip to content

Add FLASH support to TARGET_MAX32620C and brd targets #9316

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
Jan 18, 2019

Conversation

bentcooke
Copy link
Contributor

@bentcooke bentcooke commented Jan 9, 2019

Description

  1. Added FLASH IAP implementation for TARGET_MAX32620C MCU.
  2. Bumped heap size for IAR due to heap overflow for when doing page writes during testing.
  3. Added FLASH to device has of derivative platform targets MAX32620FTHR and SDT32620B.

Logs with passing GT flash tests:
flashtests-MAX32620FTHR-ARM.log
flashtests-MAX32620FTHR-IAR.log
flashtests-MAX32620FTHR-GCC_ARM.log

Pull request type

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

@bentcooke bentcooke force-pushed the add_flash_MAX32620FTHR branch from e14d562 to ec3f309 Compare January 9, 2019 20:57
@@ -16,7 +16,7 @@ define region RAM_region = mem:[from __region_RAM_start__ to __region_RAM

/* Stack and Heap */
define symbol __size_cstack__ = 0x1000;
define symbol __size_heap__ = 0x4000;
define symbol __size_heap__ = 0xF000;
Copy link
Contributor

Choose a reason for hiding this comment

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

That is quite the jump in heap allocation.

Though it looks like this target specific, so that should be fine.

Copy link
Contributor

@cmonr cmonr left a comment

Choose a reason for hiding this comment

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

LGTM

@cmonr cmonr requested review from a team January 10, 2019 06:43
@0xc0170
Copy link
Contributor

0xc0170 commented Jan 15, 2019

targets/TARGET_Maxim/TARGET_MAX32620C/device/TOOLCHAIN_IAR/MAX32620.icf

Please resolve the conflict via rebase

Copy link
Contributor

@davidsaada davidsaada left a comment

Choose a reason for hiding this comment

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

LGTM.

@bentcooke bentcooke force-pushed the add_flash_MAX32620FTHR branch from ec3f309 to a588dc3 Compare January 15, 2019 16:52
@bentcooke bentcooke force-pushed the add_flash_MAX32620FTHR branch from dfe5573 to 95a050b Compare January 15, 2019 17:00
@bentcooke
Copy link
Contributor Author

rebased to resolve conflict

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 17, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented Jan 17, 2019

Test run: SUCCESS

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

@cmonr cmonr merged commit 1afcbbf into ARMmbed:master Jan 18, 2019
@bentcooke bentcooke deleted the add_flash_MAX32620FTHR branch January 18, 2019 18:12
@adbridge
Copy link
Contributor

This is on top of #9092 scheduled for 5.12 so bumping this one also.

@adbridge
Copy link
Contributor

OK now hand patched across to mbed-os-5.11.3 release candidate

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

Successfully merging this pull request may close these issues.

6 participants