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

[MSI] Add STM defconfigs #431

Open
wants to merge 5 commits into
base: dasharo-4.21
Choose a base branch
from
Open

[MSI] Add STM defconfigs #431

wants to merge 5 commits into from

Conversation

mkopec
Copy link
Member

@mkopec mkopec commented Nov 3, 2023

Add STM defconfigs which use vboot in rwa configuration (no slot B) in order to make space for the STM binaries. Bootsplash region was moved as well to make the RW_A and WP_RO slots more equal in size. Make sure the ROMHOLE region is at the correct offset to ensure that BIOS flashback still works.

Change-Id: I973f411f53726952d159560c6810e3a22ac38c4e
Signed-off-by: Michał Kopeć <[email protected]>
Change-Id: I0365d58d48c84a5f21103fb31f6313200c93858c
Signed-off-by: Michał Kopeć <[email protected]>
Add defconfigs for STM. These configs use vboot rwa layout in order to
fit the STM binaries.

Change-Id: Ifb5c767283b4abfccfc14a459669bd83a2d84fe4
Signed-off-by: Michał Kopeć <[email protected]>
@macpijan
Copy link
Contributor

macpijan commented Nov 3, 2023

@mkopec thanks, I will ask Brian to try them out.

@mkopec mkopec force-pushed the msi_rwa branch 2 times, most recently from 9f22b73 to 30d3ffe Compare November 3, 2023 11:12
Change-Id: I9402b52f8f4c293ed0f1fd3e0e76307ca19b2b27
Signed-off-by: Michał Kopeć <[email protected]>
@pietrushnic
Copy link
Contributor

@bdelgado1995 cc

Change-Id: I62d059777fbc6d435f278517c0a13ff68d849893
Signed-off-by: Michał Kopeć <[email protected]>
@bdelgado1995
Copy link

bdelgado1995 commented Nov 5, 2023

Thanks! This solved the problem with fitting the stm.bin into the CBFS volume.

For the fully-working STM support, the default values in the configs files will need to be changed but I can advise on that once I conclude that testing. But for the concern of fitting the stm.bin image, these changes resolve the space constraint. Attaching a sample log from testing. (Can 'grep -i stm' for the STM-relevant portions.)

Also, recovery with MSI FlashBack still worked for me using this PR so that was good to see.

2.txt

@mkopec mkopec changed the base branch from dasharo to dasharo-4.21 June 3, 2024 08:48
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.

5 participants