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

psplash: udev rule to notify systemd of available framebuffers #1268

Merged
merged 1 commit into from
Aug 3, 2024

Conversation

Kiwi4Inovex
Copy link
Contributor

When using psplash in combination with systemd the splash screen is not shown. The dependency to sys-devices-platform-gpu-graphics-fb0.device will terminate psplash-start.service because systemd is not aware of the framebuffer device node.
See https://lists.yoctoproject.org/g/yocto/topic/91286438#57156

Adding a udev rule to notify systemd

@Kiwi4Inovex
Copy link
Contributor Author

Follow up of #1261 based on a new branch

@Kiwi4Inovex Kiwi4Inovex force-pushed the fix_psplash_timeout branch 2 times, most recently from 2b0742f to 5b3280a Compare January 30, 2024 15:00
Copy link
Collaborator

@kraj kraj left a comment

Choose a reason for hiding this comment

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

lgtm, without this patch we dont get splash with systemd. So we need this patch

@kraj
Copy link
Collaborator

kraj commented May 21, 2024

@Kiwi4Inovex can you rebase your branch on top of latest master and push this again so we can merge it with clean history.

@Kiwi4Inovex Kiwi4Inovex force-pushed the fix_psplash_timeout branch 3 times, most recently from 955d63c to 46efc9e Compare May 21, 2024 09:36
recipes-core/udev/udev-rules-rpi/fb.rules Outdated Show resolved Hide resolved
recipes-core/udev/udev-rules-rpi.bb Outdated Show resolved Hide resolved
@agherzan
Copy link
Owner

agherzan commented Jun 3, 2024

Also, could you please look into the branch conflict with master?

@Kiwi4Inovex Kiwi4Inovex force-pushed the fix_psplash_timeout branch 2 times, most recently from bbb2f50 to 7252172 Compare June 5, 2024 12:04
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.

Thanks again for this. Could you tweak your commit message to be in accordance with our contribution guidelines? You can find them here (https://meta-raspberrypi.readthedocs.io/en/latest/contributing.html), and you can also see examples in the history. In this specific case, there si no SoB footer.

@agherzan
Copy link
Owner

agherzan commented Jun 6, 2024

Another small comment: you are changing the "udev-rules-rpi" recipe, so I would use that in the subject line.

Add udev rule to notify systemd of available framebuffer

Using psplash in combination with systemd the splash screen is not
shown. The dependency to sys-devices-platform-gpu-graphics-fb0.device
will terminate psplash-start.service because systemd is not aware of the
existing framebuffer device node.
See https://lists.yoctoproject.org/g/yocto/topic/91286438#57156

Signed-off-by: Bastian Wanner <bastian.wanner@inovex.de>
@Kiwi4Inovex
Copy link
Contributor Author

Took me awhile sorry

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.

Thanks!

@agherzan agherzan enabled auto-merge (rebase) August 2, 2024 22:19
@agherzan agherzan merged commit e9e5efa into agherzan:master Aug 3, 2024
19 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