Skip to content

xv: change FOURCC_RGBA32 to AMD one#2176

Open
metux wants to merge 1 commit intomasterfrom
pr/xv-change-fourcc-rgba32-to-amd-one-_2026-04-13_11-47-43
Open

xv: change FOURCC_RGBA32 to AMD one#2176
metux wants to merge 1 commit intomasterfrom
pr/xv-change-fourcc-rgba32-to-amd-one-_2026-04-13_11-47-43

Conversation

@metux
Copy link
Copy Markdown
Contributor

@metux metux commented Apr 13, 2026

Initally we used value of DRM_FORMAT_ARGB8888, but it seems than
xf86-video-ati has established other value for years. So, change it to
well established one.

Because we define a different values, this breaks compilation of xf86-video-ati.
I am unsure, who is right in this situation, but I guess better to use
an established value, to make AMD xorg driver compile.

I do not aware about any of the side effects of this change.

Fixes: 15412e7 - glamor: xv: add rgba32 format
Signed-off-by: Konstantin ria.freelander@gmail.com
Part-of: https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1321

@metux metux self-assigned this Apr 13, 2026
@metux metux requested a review from a team April 13, 2026 09:47
Initally we used value of DRM_FORMAT_ARGB8888, but it seems than
xf86-video-ati has established other value for years. So, change it to
well established one.

Because we define a different values, this breaks compilation of xf86-video-ati.
I am unsure, who is right in this situation, but I guess better to use
an established value, to make AMD xorg driver compile.

I do not aware about any of the side effects of this change.

Fixes: 15412e7 - glamor: xv: add rgba32 format
Signed-off-by: Konstantin <ria.freelander@gmail.com>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1321>
@cepelinas9000
Copy link
Copy Markdown
Contributor

This one is duplicate of #2128

metux pushed a commit that referenced this pull request Apr 13, 2026
metux pushed a commit that referenced this pull request Apr 13, 2026
@metux
Copy link
Copy Markdown
Contributor Author

metux commented Apr 13, 2026

This one is duplicate of #2128

okay, then let's take @stefan11111's one.
(keeping this open until it becomes empty by rebase)

metux pushed a commit that referenced this pull request Apr 13, 2026
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