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

b_u585i_iot02a_ns: Can't build TFM enabled samples (TF-M 1.7.0) #54236

Closed
erwango opened this issue Jan 30, 2023 · 5 comments
Closed

b_u585i_iot02a_ns: Can't build TFM enabled samples (TF-M 1.7.0) #54236

erwango opened this issue Jan 30, 2023 · 5 comments
Assignees
Labels
area: TF-M ARM Trusted Firmware-M (TF-M) bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32 priority: medium Medium impact/importance bug

Comments

@erwango
Copy link
Member

erwango commented Jan 30, 2023

Describe the bug
Since #53899, TFM samples compilation fails on b_u585i_iot02a_ns target:

Traceback (most recent call last):
File "/local/mcu/zephyrproject/zephyr/scripts/build/mergehex.py", line 28, in merge_hex_files
ih.merge(to_merge, overlap=overlap)
File "/local/home/frq07517/.local/lib/python3.8/site-packages/intelhex/init.py", line 858, in merge
raise AddressOverlapError(
intelhex.AddressOverlapError: Data overlapped at address 0xC094000

To Reproduce
Steps to reproduce the behavior:

  1. west build -p always -b b_u585i_iot02a_ns samples/tfm_integration/tfm_secure_partition
  2. See error

Expected behavior
Compilation should be successful

@erwango erwango added the bug The issue is a bug, or the PR is fixing a bug label Jan 30, 2023
@erwango erwango self-assigned this Jan 30, 2023
@erwango erwango added priority: medium Medium impact/importance bug area: TF-M ARM Trusted Firmware-M (TF-M) platform: STM32 ST Micro STM32 labels Jan 30, 2023
@erwango
Copy link
Member Author

erwango commented Jan 30, 2023

@microbuilder FYI.
I'm currently investigating. I might reach out to you for some support.

@erwango
Copy link
Member Author

erwango commented Jan 31, 2023

Issue spotted. Due to changes in flash partition in TF-M v1.7.0 on board b_u585i_iot02a, b_u585i_iot02a_ns flash partition should be updated to have NS primary slot starting at 0x98000 (instead of 0x74000 with v1.6.0).
I will check if other targets could be impacted and push an update.

@microbuilder
Copy link
Member

You might want to add this board to one of the samples (tfm_ipc, for example) so that CI catches these builds errors. I didn't notice it wasn't on the list: https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/tfm_integration/tfm_ipc/sample.yaml#L8

@erwango
Copy link
Member Author

erwango commented Jan 31, 2023

You might want to add this board to one of the samples (tfm_ipc, for example) so that CI catches these builds errors

Good point. Will do

@erwango
Copy link
Member Author

erwango commented Feb 1, 2023

Fixed by #54269

@erwango erwango closed this as completed Feb 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: TF-M ARM Trusted Firmware-M (TF-M) bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32 priority: medium Medium impact/importance bug
Projects
None yet
Development

No branches or pull requests

2 participants