-
-
Notifications
You must be signed in to change notification settings - Fork 200
Add Novacustom V54 (v540tu) #1913
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
Add Novacustom V54 (v540tu) #1913
Conversation
9e221c4 to
0c5197a
Compare
|
@filipleple board not present under .circleci/config.yml: please add so that CircleCI builds the board roms |
Note that this will be affected by #1911 and Dasharo/dasharo-issues#1216 (comment) since same coreboot commit as for v560tu which suffers from those issues. Is that what we want as a first release for v540tu? This won't pass QubesOS certification as noted under #1911 |
|
We do not want, we'll see what we will end up with. |
Signed-off-by: Filip Lewiński <[email protected]>
Signed-off-by: Filip Lewiński <[email protected]>
Signed-off-by: Filip Lewiński <[email protected]>
0c5197a to
f7f3b2f
Compare
|
Testing trace:
|
Plan is to resolve performance issues fixed under Dasharo-UEFI for Dasharo-Heads (coreboot fork) later: |
|
@tlaurion Do you have other comments to be resolved prior merging it, so we can proceed with the release (assuming we do not have the fix for performance yet Dasharo/dasharo-issues#1249 )? |
Unfixed known issue also affecting nv41 |
|
It won't be fixed right now. As explained in the linked issue, there is no such FSP option for MTL right now to expose this. We will recheck again with future releases, with newer FSP versions. How does NV41 affect this MTL release here? Yes, per (heads) coreboot config, early DMA is disabled for NV41 board. Maybe there was a reason for it, I do not recall it immediately. We can go back to the original MR bringing these Alder Lake boards to heads, maybe we can get some answers there. If this is needed, we can discuss it with the next NV41 releases, not sure how it affects this MR. |
As long as its in the known issues, and nv41 is marked affected in referred issue or duplicated for upstream tracking (coreboot side), I do not see this as a blocker to merge this PR @macpijan. Just need to point release notes PR and LGTM |
tlaurion
left a comment
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.
LGTM!
@macpijan for clarity: even if enabled, Dasharo/dasharo-issues#985 (comment) shows early dma protection cannot be applied. |
|
@macpijan so merge? |
Yes please |
So far we've confirmed this code boots on V54, including the recent Full HD revision. The
coreboot-dasharo_commit_hashis due to change to a more recent one if we come up with a proper fix for Dasharo/dasharo-issues#1243 soon enough. Otherwise, we might want to publish a release based on the same coreboot revision as for V56 and include the patches in the next release.