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

Cherry-pick 98349bb367418866e008ba22ca75f4ab2984561a to kirkstone #1259

Merged
merged 1 commit into from
Jan 9, 2024

Conversation

leon-anavi
Copy link
Contributor

Especially with systemd its seen that psplash-start service starts before /dev/fb0 is created by kernel which results in

[FAILED] Failed to start Start psplash boot splash screen.

this is quite frequent race now with kernel 6.1

Add device dependency on sys-devices-platform-gpu-graphics-fb0.device via a unit file drop-in

- What I did

Fix psplash-start.service for branch kirkstone to avoid errors like:

Jan 01 00:00:09 raspberrypi4 systemd[1]: Starting Start psplash boot splash screen...
Jan 01 00:00:09 raspberrypi4 psplash[297]: Error opening /dev/fb0
Jan 01 00:00:09 raspberrypi4 systemd[1]: psplash-start.service: Main process exited, code=exited, status=255/EXCEPTION
Jan 01 00:00:09 raspberrypi4 systemd[1]: psplash-start.service: Failed with result 'exit-code'.
Jan 01 00:00:09 raspberrypi4 systemd[1]: Failed to start Start psplash boot splash screen.

Tested on Raspberry Pi 4.

- How I did it

Cherry-picked @kraj patch 98349bb to branch kirkstone. Considering this is Yocto LTS release until Apr. 2026 it makes sense to backport this particular patch.

Especially with systemd its seen that psplash-start service starts
before /dev/fb0 is created by kernel which results in

[FAILED] Failed to start Start psplash boot splash screen.

this is quite frequent race now with kernel 6.1

Add device dependency on sys-devices-platform-gpu-graphics-fb0.device
via a unit file drop-in

Signed-off-by: Khem Raj <[email protected]>
Signed-off-by: Leon Anavi <[email protected]>
Copy link
Owner

@agherzan agherzan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@agherzan agherzan merged commit 8bad47e into agherzan:kirkstone Jan 9, 2024
15 of 16 checks passed
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.

3 participants