Skip to content

Commit 0ceb48d

Browse files
zackrsmb49
authored andcommitted
drm/vmwgfx: Sort primary plane formats by order of preference
BugLink: https://bugs.launchpad.net/bugs/2067959 commit d4c972b upstream. The table of primary plane formats wasn't sorted at all, leading to applications picking our least desirable formats by defaults. Sort the primary plane formats according to our order of preference. Nice side-effect of this change is that it makes IGT's kms_atomic plane-invalid-params pass because the test picks the first format which for vmwgfx was DRM_FORMAT_XRGB1555 and uses fb's with odd sizes which make Pixman, which IGT depends on assert due to the fact that our 16bpp formats aren't 32 bit aligned like Pixman requires all formats to be. Signed-off-by: Zack Rusin <zack.rusin@broadcom.com> Fixes: 36cc79b ("drm/vmwgfx: Add universal plane support") Cc: Broadcom internal kernel review list <bcm-kernel-feedback-list@broadcom.com> Cc: dri-devel@lists.freedesktop.org Cc: <stable@vger.kernel.org> # v4.12+ Acked-by: Pekka Paalanen <pekka.paalanen@collabora.com> Link: https://patchwork.freedesktop.org/patch/msgid/20240412025511.78553-6-zack.rusin@broadcom.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Portia Stephens <portia.stephens@canonical.com> Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
1 parent e9a24eb commit 0ceb48d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

drivers/gpu/drm/vmwgfx/vmwgfx_kms.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -246,10 +246,10 @@ struct vmw_framebuffer_bo {
246246

247247

248248
static const uint32_t __maybe_unused vmw_primary_plane_formats[] = {
249-
DRM_FORMAT_XRGB1555,
250-
DRM_FORMAT_RGB565,
251249
DRM_FORMAT_XRGB8888,
252250
DRM_FORMAT_ARGB8888,
251+
DRM_FORMAT_RGB565,
252+
DRM_FORMAT_XRGB1555,
253253
};
254254

255255
static const uint32_t __maybe_unused vmw_cursor_plane_formats[] = {

0 commit comments

Comments
 (0)