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

Add memory config for STM32F4 #268

Merged
merged 1 commit into from
Oct 5, 2014
Merged

Add memory config for STM32F4 #268

merged 1 commit into from
Oct 5, 2014

Conversation

vmayoral
Copy link
Contributor

The previous memory map didn't allow to debug the whole flash
memory while working with the STM32F4Discovery. A new
configuration has been added and should be used while working
with the STM32F4Discovery board.

Refer to ros2/ros2_embedded_nuttx#7.

@prattmic
Copy link
Contributor

If the memory map needs to be updated, could you please update the original map, rather than adding a new one and commenting out use of the original?

@texane
Copy link
Collaborator

texane commented Sep 29, 2014

Thanks for the patch, but I will wait before applying it. Can you do as
Michael
suggests ?

2014-09-29 23:35 GMT+02:00 Michael Pratt notifications@github.com:

If the memory map needs to be updated, could you please update the
original map, rather than adding a new one and commenting out use of the
original?


Reply to this email directly or view it on GitHub
#268 (comment).

@vmayoral
Copy link
Contributor Author

Done. Let me know if you guys need something else.

@prattmic
Copy link
Contributor

Taking a look at Table 5 in RM0090 (STM32F4 Reference Manual, Flash module organization (STM32F40x and STM32F41x)), it looks like the length should actually be 0xE0000 to cover all sectors. Otherwise, LGTM.

A whole other issue is supporting the 2MB flash in STM32F42x/3x.

The previous memory map didn't allow to debug the whole flash
memory. A new configuration has been added and should be
used while working with the STM32F4Discovery.
@vmayoral
Copy link
Contributor Author

@prattmic thanks for the catch. I actually didn't notice the "length" keyword before the value. Having noticed that i agree, 0xE0000 should do it for the STM32F4Discovery.

@prattmic
Copy link
Contributor

prattmic commented Oct 5, 2014

👍

texane added a commit that referenced this pull request Oct 5, 2014
Add memory config for STM32F4
@texane texane merged commit 452607b into stlink-org:master Oct 5, 2014
@Nightwalker-87 Nightwalker-87 added this to the v1.2.0 and older milestone Feb 26, 2020
@Nightwalker-87 Nightwalker-87 linked an issue Mar 27, 2020 that may be closed by this pull request
@stlink-org stlink-org locked as resolved and limited conversation to collaborators Apr 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

flashing: Impossible to flash large file on stm32f4
4 participants