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

Fix STM32F7 ITCM Flash alias map #2077

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ALTracer
Copy link
Contributor

Detailed description

  • Not a new feature.
  • The existing problem is BMP/GDB Load failed for STM32F7 binaries linked to 0x0200_0000 ITCM flash bigger than 64 KiB.
  • This PR solves it by extending/copying AXIM flash map to ITCM flash alias map.

Tested on STM32F722RE with BMDA/blackpill-f411ce to allow loading 512 KiB binaries (erase, write, verify, in-firmware check). Testers with F76x/F77x (like STM32F777) are otherwise welcome.

I've changed the order of first bank stm32f4_add_flash() calls here, maybe it's unwanted. But the following code block will also want to append to ITCM dual-bank sectors. If mixed order in-BMP is a problem, please state so. Another worry is this will interact with future blank_check feature, forcing it to walk entire NVM twice. I'd like to treat this separately by tracking e.g. some "is-alias" flag, also applicable to OTP pages (of G0).
Like in PR2075, please clarify whether refman references are mandatory there or here.

Your checklist for this pull request

Closing issues

Closes #2006.

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

Successfully merging this pull request may close these issues.

[STM32F72x] Incorrect memory map
1 participant