Skip to content

Conversation

@Vge0rge
Copy link
Contributor

@Vge0rge Vge0rge commented Nov 27, 2025

This reverts commit 57ca5ed which reverted the commit which changed the target for the startup files to be the platform_ns instead of tfm_ns.

This reapplies the commit and adds a fix to the build failures seen before.

@Vge0rge Vge0rge force-pushed the reapply_startup_target branch from b9a77e5 to 9d8aafb Compare November 27, 2025 13:33
@Vge0rge Vge0rge force-pushed the reapply_startup_target branch from 9d8aafb to a7d8d77 Compare November 28, 2025 16:15
@Vge0rge Vge0rge marked this pull request as draft November 28, 2025 16:16
@Vge0rge Vge0rge force-pushed the reapply_startup_target branch 2 times, most recently from 439d81e to 31cdb2b Compare November 28, 2025 19:48
This reverts commit dd98f7b.

Not needed anymore, it is now getting upstreamed.

Signed-off-by: Georgios Vasilakis <[email protected]>
…iles""

This reverts commit 57ca5ed.

The revert is not needed anymore because a fix is being upstreamed.

Signed-off-by: Georgios Vasilakis <[email protected]>
Update platform_ns to use correct names for startup files.
Add in required includes to work with new startup files.

Change the startup files to be linked as PRIVATE since
they don't need to be PUBLIC.

Make sure that the __Vectors symbol is retained when the
TF-M tests are being built because it is used by the
tfm_common_ns.ld linker script.

Change-Id: I34f28a5e5273819084bf7e95c8827d73472703b7
Signed-off-by: Georgios Vasilakis <[email protected]>
@Vge0rge Vge0rge force-pushed the reapply_startup_target branch from 31cdb2b to 255a0b2 Compare December 1, 2025 12:00
@Vge0rge Vge0rge marked this pull request as ready for review December 1, 2025 12:00
@Vge0rge Vge0rge requested review from degjorva and tomi-font December 1, 2025 12:00
Copy link
Collaborator

@tomi-font tomi-font left a comment

Choose a reason for hiding this comment

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

❤️

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.

2 participants