Skip to content

Conversation

djiatsaf-st
Copy link
Contributor

This PR includes two related updates to improve flash testing and partition layout consistency

  • Updated CONFIG_TEST_DRIVER_FLASH_SIZE to match the actual flash size on NUCLEO_WB55RG
    and ensure flash tests pass.
  • Increased storage partition size on NUCLEO_H753ZI to support test_flash_copy.
  • Removed unnecessary scratch partition and reordered partition layout for clarity.

- Increase the size of the storage partition by two pages
because test_flash_copy needs 2 flash page.

Besides, delete obsolete scratch partition and reorder
partitions with appropriately

Signed-off-by: Fabrice DJIATSA <[email protected]>
update CONFIG_TEST_DRIVER_FLASH_SIZE to match actual flash size.
The test was previously failing due to a mismatch between the expected
flash size (827392 bytes) and the actual size reported by flash_get_size()
(1048576 bytes).
This update sets CONFIG_TEST_DRIVER_FLASH_SIZE to 1048576 to reflect the
correct size of the flash device as defined in the reference manual for
the NUCLEO_WB55RG board.
The increase in required flash size (216K) for the full stack M0 binary
also justifies aligning the test configuration with the actual hardware
capabilities.

This change ensures the test passes and accurately reflects the flash
memory layout.

Signed-off-by: Fabrice DJIATSA <[email protected]>
Copy link

@de-nordic de-nordic assigned erwango and unassigned de-nordic Oct 17, 2025
@de-nordic
Copy link
Contributor

Assigning to @erwango , STM stuff.

Copy link
Contributor

@etienne-lms etienne-lms left a comment

Choose a reason for hiding this comment

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

For commit ""For commit "boards: st: nucleo_h753zi: update and reorder flash partitions":
Moving slot0/slot1 partitions location do break update compatibility versus prior versions (LTS?). Ditto regarding the scratch partition if previous LTS mcuboot for this board used it.

At least a word or 2 in the migration guide is needed.
Maybe should we add a board variant DTS that provides this compatibility.

Why don't we use the full 2MB available in internal flash?

@djiatsaf-st
Copy link
Contributor Author

For commit ""For commit "boards: st: nucleo_h753zi: update and reorder flash partitions": Moving slot0/slot1 partitions location do break update compatibility versus prior versions (LTS?). Ditto regarding the scratch partition if previous LTS mcuboot for this board used it.

At least a word or 2 in the migration guide is needed. Maybe should we add a board variant DTS that provides this compatibility.

Why don't we use the full 2MB available in internal flash?

I understand your suggestion about backward compatibility; I think a solution within the scope of this PR would be to keep the scratch partition and simply move the offsets of the other partitions.

A discussion could be held on what strategy to adopt to standardize flash partition layouts in line with migration requirements.

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

Labels

area: Boards/SoCs area: Flash area: Tests Issues related to a particular existing or missing test platform: STM32 ST Micro STM32

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants