Skip to content

Conversation

vermaete
Copy link
Contributor

The X11 was not working for my Raspberry Pi5 at master of meta-raspberrypi and meta-openembedded.

All credits for the the contributors of #1426

@kraj @agherzan Please review this PR careful please.

SERIAL_CONSOLES ?= "115200;ttyAMA10"

VC4DTBO ?= "vc4-kms-v3d"
VC4DTBO ?= "vc4-kms-v3d-pi5"
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is a common change that will affect wayland/weston or eglfs images as well. Can you see if core-image-weston boots for your case on pi5 ?

Copy link

@danielfdickinson danielfdickinson Jul 29, 2025

Choose a reason for hiding this comment

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

I haven't needed this with either a pi5 or cm5 with I/O board. When did you need it?

With cm5 I needed updated mesa, however.

Copy link
Owner

Choose a reason for hiding this comment

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

Also, this shouldn't be needed as per the overlay map. Unless there is something I'm missing, please drop.


do_deploy:append:raspberrypi5() {
echo "max_framebuffers=2" >> $CONFIG
echo "disable_fw_kms_setup=1" >> $CONFIG
Copy link
Collaborator

Choose a reason for hiding this comment

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

same here, it becomes effective for all kind of graphics stacks on top of pi5

Choose a reason for hiding this comment

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

As above. I haven't needed this with a pi5 or cm5 with I/O board. With cm5 I needed updated mesa however.

"

SRC_URI:append:raspberrypi5 = " \
file://99-vc4.conf \
Copy link
Collaborator

Choose a reason for hiding this comment

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

does it need to be pi5 specific ?

Choose a reason for hiding this comment

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

This is needed for (at least) both pi5 and cm5 with I/O board. Using this in something like raspberrypi5.conf allows to do SRC_URI:append:pi5 at least.

OVERRIDES:append = ":hwpi5:pi5"

However, from what I have read vc4 may apply to pi4/cm4-io (at least) as well. One could do something like

OVERRIDES:append = ":pi-vc4"

in an include file for any Pi board to which this applies (e.g. probably not the Zero/Zero W)
and SRC_URI:append:pi-vc4 here.

The official Pi hardware documentation might have the info needed to determine which models this is needed for.

do_install:append:raspberrypi5 () {
install -d ${D}/${sysconfdir}/X11/xorg.conf.d/
install -m 0644 ${S}/99-vc4.conf ${D}/${sysconfdir}/X11/xorg.conf.d/
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

same question as one above, can it be more generic ?

@marcinkowskip
Copy link

I went through the #1426, and nothing from here was needed for pure EGL KMS case - I use Qt6. I had the same issue on Qt6 and #1426 helped me to solve that issue.

The only thing was needed from #1426 was Mesa update (I based on the description from the issue). Another thing is that there is PR !1395 which bumps Mesa above 24.2.8 mentioned in the #1426. Question is if !1395 fixes the rendering issues or not.

Things which I was not needed:

@kraj
Copy link
Collaborator

kraj commented Jun 4, 2025

@vermaete please rebase it on latest master and push again.

@danielfdickinson
Copy link

danielfdickinson commented Jul 29, 2025

Similar to @marcinkowskip I needed a mesa update - but in my case only for cm5 with I/O board. I did, however, need the xorg.conf.d snippet.

I did not need the other suggested changes with either a pi5 or cm5 with I/O board.

SERIAL_CONSOLES ?= "115200;ttyAMA10"

VC4DTBO ?= "vc4-kms-v3d"
VC4DTBO ?= "vc4-kms-v3d-pi5"
Copy link
Owner

Choose a reason for hiding this comment

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

Also, this shouldn't be needed as per the overlay map. Unless there is something I'm missing, please drop.

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.

5 participants