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

Bootsplash change fails #1215

Open
SebastianCzapla opened this issue Jan 17, 2025 · 3 comments
Open

Bootsplash change fails #1215

SebastianCzapla opened this issue Jan 17, 2025 · 3 comments
Labels
bug Something isn't working firmware needs review needs testing Issue needs testing protectli_vault_ehl eg. VP2420

Comments

@SebastianCzapla
Copy link
Contributor

SebastianCzapla commented Jan 17, 2025

Component

Dasharo firmware

Device

Protectli VP2420

Dasharo version

1.2.1-rc3

Dasharo Tools Suite version

No response

Test case ID

DCU003.001

Brief summary

Changes to logo are not reflected in the final binary

How reproducible

100%

How to reproduce

Run DCU003.001 test case, or change the bootsplash manuall with DCU.

Expected behavior

New logo appears when booting platform

Actual behavior

Protectli logo does not change.

Test fails with:
'804dfe8796ced8081e725aa52c9f0c8f85257d365018df683c1753e825cff2db /sys/firmware/acpi/bgrt/image' does not contain 'f91fe017bef1f98ce292bde1c2c7c61edf7b51e9c96d25c33bfac90f50de4513'

Screenshots

No response

Additional context

No response

Solutions you've tried

Manual checks, checking that the sha256sum of the firmware file in variety of situations, the sha256sum changes after DCU, but image stays the same
No response

@Firminator
Copy link

I ran into this as well on a Z690 using DCU. The issue was that the resolution needs to be 544x432 or lower.
See Dasharo/dcu#19

@miczyg1
Copy link
Contributor

miczyg1 commented Jan 20, 2025

The resolution may matter, depending on the monitor resolution and other factors.

@miczyg1 miczyg1 added needs reproduction Issue has not yet been reproduced needs testing Issue needs testing and removed needs reproduction Issue has not yet been reproduced labels Feb 10, 2025
@miczyg1
Copy link
Contributor

miczyg1 commented Feb 10, 2025

The RC4 binary shoould have the issue fixed already by this commit: Dasharo/coreboot@44edcb5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working firmware needs review needs testing Issue needs testing protectli_vault_ehl eg. VP2420
Projects
None yet
Development

No branches or pull requests

3 participants