Skip to content
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

NANO130: Support dynamic heap configuration on IAR #12061

Merged

Conversation

ccli8
Copy link
Contributor

@ccli8 ccli8 commented Dec 10, 2019

Summary of changes

This PR is continuation of #12047 and tries to fix static SRAM OOM further by supporting dynamic heap configuration on IAR. In this PR, for IAR toolchain, heap is configured to 1KiB at a minimum and expandable to available SRAM. This requires IAR 8.x.


Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[x] No Tests required for this change (E.g docs only update)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

Reviewers

@mprse @0xc0170


On IAR, configure heap to 1KiB at a minimum and expandable, dependent on available SRAM. This requires IAR 8.x.
@ciarmcom
Copy link
Member

@ccli8, thank you for your changes.
@mprse @0xc0170 @Ronny-Liu @ARMmbed/mbed-os-maintainers please review.

@0xc0170 0xc0170 added release-version: 6.0.0-alpha-1 First pre-release version of 6.0.0 needs: CI and removed needs: review labels Dec 10, 2019
@mbed-ci
Copy link

mbed-ci commented Dec 10, 2019

Test run: FAILED

Summary: 4 of 4 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

  • jenkins-ci/mbed-os-ci_unittests
  • jenkins-ci/mbed-os-ci_build-GCC_ARM
  • jenkins-ci/mbed-os-ci_build-ARM
  • jenkins-ci/mbed-os-ci_build-IAR

@0xc0170
Copy link
Contributor

0xc0170 commented Dec 11, 2019

Will restart the tests (internal fault)

@0xc0170
Copy link
Contributor

0xc0170 commented Dec 11, 2019

CI restarted

@mbed-ci
Copy link

mbed-ci commented Dec 11, 2019

Test run: SUCCESS

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

@0xc0170 0xc0170 removed the needs: CI label Dec 11, 2019
@0xc0170 0xc0170 merged commit 1d5bd75 into ARMmbed:master Dec 11, 2019
@cyliangtw cyliangtw deleted the nuvoton_nano130_iar_dynamic_heap branch March 9, 2023 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-version: 6.0.0-alpha-1 First pre-release version of 6.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants