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.
This commit is contained in:
Tony Crisci 2017-07-27 15:57:29 -04:00
parent b1ec0dce3a
commit d822e0768b

View file

@ -304,6 +304,13 @@ static bool wlr_drm_output_set_cursor(struct wlr_output_state *output,
return true; 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; uint64_t bo_width, bo_height;
int ret; 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, 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]) { if (!output->cursor_bo[i]) {
wlr_log(L_ERROR, "Failed to create cursor bo"); wlr_log(L_ERROR, "Failed to create cursor bo");