Skip to content

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