Skip to content

Fix lto #17

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 3 commits into from
Oct 24, 2022
Merged

Fix lto #17

merged 3 commits into from
Oct 24, 2022

Conversation

pennam
Copy link
Collaborator

@pennam pennam commented Sep 6, 2022

@facchinm i had to slightly change the custom.json file rto reach the same level of ROM optimization of lto.json otherwise the binary won't fit for portenta.

see 5c1c578

Could you give it a spin with your toolchain to check if it still works for you?

@pennam pennam requested a review from facchinm September 6, 2022 14:39
@facchinm
Copy link
Member

facchinm commented Sep 6, 2022

I get [ERROR] Attempt to override undefined parameter 'mcuboot.encrypt-scratch' in 'application[PORTENTA_H7_M7]' , maybe mbed-os.lib is outdated?

@pennam
Copy link
Collaborator Author

pennam commented Sep 6, 2022

I get [ERROR] Attempt to override undefined parameter 'mcuboot.encrypt-scratch' in 'application[PORTENTA_H7_M7]' , maybe mbed-os.lib is outdated?

🤔 it looks correct...

From a new folder this works for me:

git clone https://github.com/arduino/mcuboot-arduino-stm32h7.git
cd mcuboot-arduino-stm32h7/
mbed config root . && mbed deploy
git fetch origin
git checkout origin/fix-lto
mbed compile -m PORTENTA_H7_M7 -t GCC_ARM --profile=release --profile custom.json

@pennam pennam merged commit 5dbff35 into main Oct 24, 2022
@pennam pennam deleted the fix-lto branch March 21, 2023 09:19
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