-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update preserve_rom function to check if /tmp/cbfs-init.rom should be used. #10
Conversation
… used. Signed-off-by: Matthew Drobnak <[email protected]>
Closing in favor of linuxboot#1699 |
@mkopec the ideal fix would be to extend cbfs.c capabilities as discussed under linuxboot#1699 (comment). Thoughts? |
Re-opening here. This will fix any MSI users for the time being. |
@mkopec is upstreaming MSI boards under linuxboot/heads planned short term? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the fix! As we discussed elsewhere, ideally we'd add support for the extended BIOS window to flashtools and not use flashrom, but I have no problem merging this for now.
@macpijan needs upstreaming alongside MSI boards. |
linuxboot#1699 : put as draft. Please refer to that PR upstream with a PR that deprecates it. |
Opened Dasharo/dasharo-issues#897 to track |
On the MSI boards, the "internal" CBFS region, which is the default when calling cbfs without arguments, returns no files, or when called with -v, errors:
This patch is an extension of linuxboot@4d533b3 changes, which if configured uses flashrom to extract the image of the CBFS there, and here it uses that extracted image to add files to the image which is to be flashed.
This has been tested on my MSI Z790-P DDR4 board.