Skip to content

Revert "cmake: linker: arm: put RAM sections in RAM region" #59065

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

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

57300
Copy link
Collaborator

@57300 57300 commented Jun 8, 2023

This reverts commit f5eada5.

Fixes #57590.

In order to fix incorrect program headers with CMAKE_LINKER_GENERATOR, issue #59064 needs to be addressed first. Until then, revert to the status quo from several versions back.

tejlmand
tejlmand previously approved these changes Jun 9, 2023
nordicjm
nordicjm previously approved these changes Jun 12, 2023
@carlescufi carlescufi added the bug The issue is a bug, or the PR is fixing a bug label Jun 12, 2023
@carlescufi carlescufi added this to the v3.4.0 milestone Jun 12, 2023
galak
galak previously requested changes Jun 12, 2023
Copy link
Collaborator

@galak galak left a comment

Choose a reason for hiding this comment

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

Can we not fix the hex file issue while maintaining what commit f5eada5 was trying todo?

@57300
Copy link
Collaborator Author

57300 commented Jun 12, 2023

Can we not fix the hex file issue while maintaining what commit f5eada5 was trying todo?

I've noted that this is not possible until #59064 is fixed. If I leave .bss and .noinit in RAM, and try to move .data & others' LMA to flash, then the generated linker script is wrong.

@nashif nashif removed this from the v3.4.0 milestone Jun 16, 2023
@gmarull
Copy link
Member

gmarull commented Jul 31, 2023

Ping here, this resolves an important issue (linker generator unusable)

@gmarull gmarull requested a review from galak July 31, 2023 13:36
@carlescufi carlescufi dismissed galak’s stale review July 31, 2023 13:38

dismissing at this point since this is not fixable in any other way

@carlescufi
Copy link
Member

@57300 please rebase

This reverts commit f5eada5.

Fixes zephyrproject-rtos#57590.

In order to fix incorrect program headers with CMAKE_LINKER_GENERATOR,
issue zephyrproject-rtos#59064 needs to be addressed first. Until then, revert to the
status quo from several versions back.

Signed-off-by: Grzegorz Swiderski <[email protected]>
@57300 57300 dismissed stale reviews from nordicjm and tejlmand via b046e33 July 31, 2023 13:46
@57300 57300 force-pushed the fix-huge-zephyr-bin branch from f2e3a6f to b046e33 Compare July 31, 2023 13:46
@carlescufi carlescufi requested review from tejlmand and nordicjm July 31, 2023 13:46
@carlescufi carlescufi merged commit e4de669 into zephyrproject-rtos:main Aug 1, 2023
@57300 57300 deleted the fix-huge-zephyr-bin branch April 9, 2025 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Build System bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Huge bin/hex file when using CONFIG_CMAKE_LINKER_GENERATOR=y
8 participants