Skip to content

Stm32h7rs flash restore flash size of 64KB #89894

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

FRASTM
Copy link
Collaborator

@FRASTM FRASTM commented May 13, 2025

  1. Fix the double definition of the FLASH_OPT_KEY1 and FLASH_OPT_KEY2 for the stm32h7RS serie
  2. following the flash: stm32h7x: fix implicit declaration for LL_GetFlashSize #89675 exact flash size for the stm32H7RS is fixed to 64KB

@FRASTM
Copy link
Collaborator Author

FRASTM commented May 13, 2025

removing the change for OPTKEY constant as already fixed by #89740

de-nordic
de-nordic previously approved these changes May 13, 2025
Copy link
Member

@erwango erwango left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not compatible with the changes done here: #89675
It seems github has no rebase problem on this but combination of both won't work.

Fix the flash size to 64KB for the stm32H7RS serie.
Restore the get_size API function for that serie.
This is a specificity of this stm32 mcu (RefMan RM0477)

Signed-off-by: Francois Ramu <[email protected]>
@FRASTM FRASTM force-pushed the stm32h7rs_flash branch from 7398f96 to 58aea05 Compare May 14, 2025 08:22
@FRASTM
Copy link
Collaborator Author

FRASTM commented May 14, 2025

rebase on 88b108d

@FRASTM FRASTM changed the title Stm32h7rs flash fix warning and flash size Stm32h7rs flash restore flash size of 64KB May 14, 2025
@FRASTM FRASTM requested a review from erwango May 14, 2025 08:24
@erwango erwango requested a review from de-nordic May 14, 2025 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants