From d822e0768bb90c959bc9e59aa0e2cb6945ab839a Mon Sep 17 00:00:00 2001 From: Tony Crisci Date: Thu, 27 Jul 2017 15:57:29 -0400 Subject: [PATCH] Use ARGB8888 pixel format for cursor bo The previous pixel format (RGBA8888) is not supported on all devices for cursors. Weston uses ARGB8888 exclusively for its cursor buffer object which leads me to believe this pixel format is better supported. --- backend/drm/drm.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/backend/drm/drm.c b/backend/drm/drm.c index bba3a390..6e969788 100644 --- a/backend/drm/drm.c +++ b/backend/drm/drm.c @@ -304,6 +304,13 @@ static bool wlr_drm_output_set_cursor(struct wlr_output_state *output, return true; } + if (!gbm_device_is_format_supported(state->renderer.gbm, + GBM_FORMAT_ARGB8888, GBM_BO_USE_CURSOR | GBM_BO_USE_WRITE)) { + wlr_log(L_ERROR, "Failed to create cursor bo: ARGB8888 pixel format is " + "unsupported on this device"); + return false; + } + uint64_t bo_width, bo_height; int ret; @@ -323,7 +330,7 @@ static bool wlr_drm_output_set_cursor(struct wlr_output_state *output, } output->cursor_bo[i] = gbm_bo_create(state->renderer.gbm, bo_width, bo_height, - GBM_FORMAT_RGBA8888, GBM_BO_USE_CURSOR | GBM_BO_USE_WRITE); + GBM_FORMAT_ARGB8888, GBM_BO_USE_CURSOR | GBM_BO_USE_WRITE); if (!output->cursor_bo[i]) { wlr_log(L_ERROR, "Failed to create cursor bo");