Skip to content

STM32L496xG: increase IAR heap size #9411

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
Jan 22, 2019
Merged

STM32L496xG: increase IAR heap size #9411

merged 1 commit into from
Jan 22, 2019

Conversation

jeromecoutant
Copy link
Collaborator

Description

STM32L496xG RAM size is 320k

So heap size default value should/can be increased in order to make all the mbed-os tests OK with IAR.

Pull request type

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

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 17, 2019

What tests were failing?

@jeromecoutant
Copy link
Collaborator Author

What tests were failing?

Some filesystem one, only with IAR.
They are OK with DISCO_L496AG and NUCLEO_L496ZG now.

@ciarmcom
Copy link
Member

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

@ciarmcom ciarmcom requested a review from a team January 17, 2019 14:00
@cmonr
Copy link
Contributor

cmonr commented Jan 17, 2019

NOTE: This PR has now been rebased.

If this was made in error, feel free to force-push your local branch to restore the PR.

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 21, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented Jan 21, 2019

Test run: FAILED

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

Failed test jobs:

  • jenkins-ci/mbed-os-ci_greentea-test

@alekla01
Copy link
Contributor

restarted ci: greentea-test

@0xc0170 0xc0170 merged commit 16e544d into ARMmbed:master Jan 22, 2019
@jeromecoutant jeromecoutant deleted the PR_L496 branch January 22, 2019 13:53
@adbridge
Copy link
Contributor

This is sitting on top of #9092 scheduled for 5.12

@jeromecoutant
Copy link
Collaborator Author

This is sitting on top of #9092 scheduled for 5.12

No... there is no link with #9092...

@adbridge
Copy link
Contributor

OK 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
Reviewers

@0xc0170 0xc0170 0xc0170 approved these changes

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

Successfully merging this pull request may close these issues.
7 participants
@jeromecoutant @0xc0170 @ciarmcom @cmonr @mbed-ci @alekla01 @adbridge