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

STM32L432KC: increase RAM size from 48k to 64k #5171

Merged
merged 1 commit into from Sep 27, 2017

Conversation

Projects
None yet
5 participants
@jeromecoutant
Contributor

jeromecoutant commented Sep 22, 2017

Description

RAM size is increased from 48k to 64k for STM32L432KC as proposed in #4823

WARNING: you have to update ST-Link FW to V2J29M18
http://www.st.com/en/development-tools/stsw-link007.html

Status

READY

Tests

All OS2 and OS5 tests have been checked

@0xc0170 0xc0170 added the needs: CI label Sep 22, 2017

@0xc0170

What was the limitation before to use only 48k ? firmware?

@jeromecoutant

This comment has been minimized.

Contributor

jeromecoutant commented Sep 22, 2017

What was the limitation before to use only 48k ? firmware?

2 things:

  1. yes, FW: only V2J29M18 FW allows to use more than 48k

  2. As explained in #4823, SRAM1 and SRAM2 are not in the same area. But SRAM2 is mapped continuously to SRAM1. We didn't know that...

Thx @helmut64

@helmut64

This comment has been minimized.

Contributor

helmut64 commented Sep 22, 2017

Changes look good, thank you enhancing this. Helmut

@jeromecoutant

This comment has been minimized.

Contributor

jeromecoutant commented Sep 27, 2017

morph test

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Sep 27, 2017

/morph test

@mbed-bot

This comment has been minimized.

mbed-bot commented Sep 27, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1402

All builds and test passed!

@theotherjimmy theotherjimmy merged commit c06368a into ARMmbed:master Sep 27, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jeromecoutant jeromecoutant deleted the jeromecoutant:PR_L432_RAM branch Sep 28, 2017

@helmut64

This comment has been minimized.

Contributor

helmut64 commented Oct 1, 2017

@jeromecoutant
I verified the changes, it works great for me.
00:10:47.080943 MyHome: 1.0 (Oct 1 2017 16:36:04 mbed: v151)
00:10:47.081408 InitDefaults Done
00:10:47.083461 MemoryAvailable: 55 kB (56720 bytes)
Thank you, Helmut

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